curl / Mailing Lists / curl-library / Single Mail


Re: Travis CI proposal

From: David Chapman <>
Date: Thu, 7 Sep 2017 00:53:53 -0700

On 9/6/2017 11:27 PM, Daniel Stenberg wrote:
> Googling about this issue, I found that others have ran into this
> problem before [1], as the standard [2] says it should be const, but
> the glibc maintainer with a "strong personality" dismissed that pretty
> hard [3] back in the day.
> Maybe we should just start out with removing the 'const' there and
> then wait for people to report when *that* causes warnings and then we
> can add #ifdefs for it then... ?
> [1] =
> [2] =
> [3] =

The response in [3] is kind of snippy because [2] is out of date.
Standard iconv() [4] now uses "char ** restrict" for the second
argument.  Since I am out of my comfort zone here, I won't try to figure
out when the change was made official, but it appears to have been 2004
[5] or earlier.  I can't help with patching either; I am a mere mortal
in libcurl world.

So yeah, I'd remove the "const" and wait for complaints.  Hopefully they
will be more polite than the developer in [3]...

[4] =
[5] =

     David Chapman
     Chapman Consulting -- San Jose, CA
     Software Development Done Right.
Received on 2017-09-07