Hi all

I think this version of TestHttpHeader is much better than the previous one. All the test code is much simplified and just contain the  assertions… Fixtures help a lot here.

I removed some duplication in BDecoderTest too… I removed the  length typeid and instead have put in the testing of the pointer value… so the typeid tests are still there but the type is instead checked by the dynamic_cast.

It is amazing how removing two lines of duplication simplifies the code a lot ;-) I still don’t like that separate hasReadAll() and hasReadPassEnd() — shouldn’t it be hasReadPastEnd ;-)

Let me know what you all think… and happy weekend to you all ;-)
—- cheerio atul