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
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/