Code review comment for ~ahasenack/ubuntu/+source/squid3:xenial-squid-reload-1738412

Christian Ehrhardt  (paelzer) wrote :

The changes LGTM +1 for those.

I read and thought about the regression potential twice, but I like the "explicitly broken" much more than "silently broken". I think it is good that you outlined it in the bug, but I'd want the change and not see it as a bad thing.

review: Approve

« Back to merge proposal