[PATCH] Proper assignment and copying of HttpHeader

From: Alex Rousskov <rousskov_at_measurement-factory.com>
Date: Thu, 17 Feb 2011 22:08:24 -0700

Code cleanup: Proper assignment and copying of HttpHeader.

Besides being the Right Thing, having correct assignment operator and
copy constructor helps classes that have HttpHeader data members to
avoid defining explicit assignment operators and copy constructors.

Also adds forgotten reset of "len" in the clean() method. Perhaps that
data member is not needed at all?

Other polishing touches. HttpHeader::reset() is now a tiny bit faster.

No runtime changes expected.

Thank you,

Alex.
P.S. I stumbled upon this ancient mess while enhancing adaptation
History classes. This patch will remove a few dozen lines from those
classes, but those changes will be submitted separately.

Received on Fri Feb 18 2011 - 05:08:32 MST

This archive was generated by hypermail 2.2.0 : Fri Feb 18 2011 - 12:00:05 MST