Re: [PATCH] signed char fixes for scripts

[Paulo Marques <pmarques@grupopie.com>]

J.A. Magallon wrote:
> On 07.27, Sam Ravnborg wrote:
> 
>>On Fri, Jul 15, 2005 at 10:14:43PM +0000, J.A. Magallon wrote:
>>
>>>On 07.16, J.A. Magallon wrote:
>>>
>>>>On 07.15, Andrew Morton wrote:
>>>>
>>>>>ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.13-rc3/2.6.13-rc3-mm1/
>>>
>>>This time I did not break anything... and they shut up gcc4 ;)
>>
>>I have applied it to my tree. There still is a lot left when I compile
>>with -Wsign-compare.
> 
> All the problems are born here:
> 
> struct sym_entry {
>     unsigned long long addr;
>     unsigned int len;
>     unsigned char *sym;
> };

What are you guys talking about?

I've just compiled the current version in -mm with -Wsign-compare and it 
doesn't give me a single warning.

Is my compiler version the problem (3.3.2), or are you testing with the 
old version of kallsyms?

-- 
Paulo Marques - www.grupopie.com

It is a mistake to think you can solve any major problems
just with potatoes.
Douglas Adams