gwlib/charset patch

Nikos Balkanas nbalkanas at
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>
To: "Nikos Balkanas" <nbalkanas at>
Cc: <devel at>
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> 
> wrote:
>> Martin,
>> 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 
>> keep
>> 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.
>> BR,
>> Nikos
> 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.
> M.

More information about the devel mailing list