nbalkanas at gmail.com
Mon Mar 2 08:46:56 CET 2009
I agree that this is the right solution. However, i don't think it is worth
it for a simple compiler warning. If we start claterring the code with too
many preprocessor options for each system, quickly the code will become
My idea from the start was a simple straight thing. If it doesn't break
other systems, implement it. Else forget it.
----- Original Message -----
From: "Martin Conte Mac Donell" <reflejo at gmail.com>
To: "Nikos Balkanas" <nbalkanas at gmail.com>
Cc: <devel at kannel.org>
Sent: Monday, March 02, 2009 5:59 AM
Subject: Re: gwlib/charset patch
> On Mon, Mar 2, 2009 at 1:11 AM, Nikos Balkanas <nbalkanas at gmail.com>
>> The only valid argument against the patch would be that (const char **)
>> causes warnings to other systems. If it does not, why not apply it to
>> everybody, including Solaris and OSX, happy?
>> It is not an alternative to change libiconv sources or configuration,
>> especially if it doesn't make a difference for kannel.
> I'm not saying that we need to change libiconv. What i'm saying is
> that the "right way" to get over this warning is using a compiler
> conditional inside kannel. See my attached example.
More information about the devel