Re: stringng-cherrypick r12764

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Fri, 27 Sep 2013 21:55:25 +1200

On 27/09/2013 9:47 p.m., Kinkie wrote:
>> If you recall that side project I was doing ever now and again to improve
>> the compiler warnings situation. -Wshadow got done but -Wcasting and
>> -Wconversion and a few others are still very much TODO.
>>
>> The situation as it stands for conversion is that *certain* very specific
>> dangerous conversions are highlighted as errors by the compilers I refer to
>> in commit messages as "strict" (clang, ICC and Sun Studio mainly). However
>> the vast majority of conversions are considered safe by compilers and they
>> all only produce warnings which we are *not yet* catching with Squid compile
>> flags.
>> If you build with "-Wconversion -Wsign-conversion -Wtraditional-conversion"
>> on a recent GCC you will see a list of what needs investigating still.
>> (sorry have not yet made bugs to track them).
> How about proceeding like this: merge as-is and trust the callers to
> know what they're doing, so that I can spend some time on implementing
> these flags? It'll be tedious work, but it shouldn't require too much
> brainpower. The alternative (blocking StringNG until the conversions
> work is done) would stall StringNg even more :(

I would go with trusting the callers for now.

Amos
Received on Fri Sep 27 2013 - 09:55:35 MDT

This archive was generated by hypermail 2.2.0 : Fri Sep 27 2013 - 12:00:11 MDT