From: Warren Young
Date: November 27 2006 11:58pm
Subject: Re: Patch to fix char signedness issue on ppc
List-Archive: http://lists.mysql.com/plusplus/6153
Message-Id: <456B7BB9.2000704@etr-usa.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Michael Hanselmann wrote:
> 
> On Linux/ppc, gcc uses unsigned char's by default. query.cpp constructs
> an object of SQLParseElement and gives it a value of -1 for the "char"
> parameter (n). Later this value is compared to -1, which will be 255
> when using unsigned chars. The result is a wrongly thrown exception. The
> attached patch changes the type to "short int" where the number is used.

Unfortunately, that will change the ABI of the library, which would 
force this change to wait for v3.0.  Instead, why not just make the 
required signedness explicit?  Then the library interface stays the 
same, so I can apply it immediately.