[squid-dev] [PATCH] remove String.cci

Alex Rousskov rousskov at measurement-factory.com
Thu Jan 5 15:32:44 UTC 2017


On 01/04/2017 07:42 PM, Amos Jeffries wrote:
> This patch removes String.cci file, moving the simple methods inline to
> the SquidString.h and more complicated others to String.cc.

Thank you for doing this long-overdue cleanup.


> +    /// throws when size() > INT_MAX
> +    int psize() const {
> +        Must(size() < INT_MAX);
> +        return size();

The code does not match its description: The code also throws when
size() is exactly INT_MAX for some reason.

Please commit when you are comfortable with the code.

Alex.





More information about the squid-dev mailing list