This is the mail archive of the libstdc++@sources.redhat.com mailing list for the libstdc++ project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: V3 PATCH: Fix const-correctness in use of iconv


>>>>> "Andreas" == Andreas Jaeger <aj@suse.de> writes:

    Andreas> And that's the way to go according to the open group - we
    Andreas> should really write a fixincludes for this.

It's clear we need to do something.

I think it may actually be better to autoconf this.  It's a little
funky to change the type of this function; that may break code written
for the out-of-date/incorrect definitions that all these OS's used.
In general, I bet lots of systems have some weirdnesses in their
headers; traditionally we only fixincluded the bare minimum that was
necessary to get the headers to be parsable by GCC.

But, I have no strong opinion, and am happy to leave the autoconf
vs. fixincludes decision to people who know more than I -- both about
`iconv' and also about the philosophy of fixincludes.

Patches welcome!

--
Mark Mitchell                   mark@codesourcery.com
CodeSourcery, LLC               http://www.codesourcery.com

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]