On 08/24/2016 09:20 AM, Amos Jeffries wrote:

> in src/HttpReply.h:
> * please use doxygen syntax "\returns" instead of "returns" in the
> comment text.

No objection, but please note that the "returns..." phrase in isolation
does not fully describe what the method returns in this case. Only the
method description taken as a whole describes that.


> in src/client_side_reply.cc:
> * please remove the "handleIMSReply: " bit from touched debugs messages.

If Eduard does that, he should remove that bit from all related debugs
messages to keep them all consistent IMO.


Thank you,

Alex.

_______________________________________________
squid-dev mailing list
squid-dev@lists.squid-cache.org
http://lists.squid-cache.org/listinfo/squid-dev

Reply via email to