Hi Peter, You could probably move the “// Response body handling” part inside the first if, then you don’t need the return in the else and the flow will be clearer. Yes, (more...)

Hi Petr, The tests look good! You could probably move the "// Response body handling" part inside the first if, then you don't need the return in the else and the (more...)

Hi Peter, thanks for comments! Http::Header and its derived classes should implement a Value Object, so we should provide operator==(). The operator()== implementation compating the raw representation of the Header objects (more...)

Hi Petr, I just pushed a revision based on your latest with the tests reworked. I think the tests are a good demonstration for the "Tell, Don't Ask" principle. I've removed the (more...)