On Fri, 30 Nov 2018, Ben Greear wrote:

>> This is a broken compiler that warns about this. I really prefer the
>> existing one-line version to this 4-line replacement.
>
> Well yes, it would be great if the compiler wasn't broken, but still nice to
> allow curl to compile (with -Werror) on weird/broken-ish compilers.
>
> I think a few lines of extra code is a small price to pay.

Everything's a ballance of course. What's the price/benefit trade-off?

We would gain an ancient compiler to build warning-free at the price of
slightly less readable code (which of course is debatable and we could argue
how much of a problem it would be, but I also don't want to open the flood
gates for fixing warnings when it is so obviously wrong in the compiler).

In this case, it seems mostly one or two persons will gain from fixing the
build for the ancient compiler (that even could be updated) while more than
one or two persons will potentially risk something by needlessly complicating
the code.

So my stance on this patch is still: thanks, but no thanks.

-- 
  / daniel.haxx.se
-------------------------------------------------------------------
Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library
Etiquette:   https://curl.haxx.se/mail/etiquette.html