lkml.org 
[lkml]   [2005]   [Jul]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] signed char fixes for scripts
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 ;)

Thanks.
Can you please resend with proper changelog and signed-off-by.
Diff should be done on top of latest -linus preferable.
Also this patch seems relative small compared to the others floating
around to cure signed warnings in scripts/
Does this really fix all of them or only a subset of the warnings?

I do not have gcc4 present but maybe thats easy - running gentoo?

> --- linux-2.6.12-jam7/scripts/kallsyms.c.orig 2005-07-06 00:16:39.000000000 +0200
> +++ linux-2.6.12-jam7/scripts/kallsyms.c 2005-07-06 00:42:24.000000000 +0200
> @@ -166,9 +166,9 @@
> * move then they may get dropped in pass 2, which breaks the
> * kallsyms rules.
> */
> - if ((s->addr == _etext && strcmp(s->sym + offset, "_etext")) ||
> - (s->addr == _einittext && strcmp(s->sym + offset, "_einittext")) ||
> - (s->addr == _eextratext && strcmp(s->sym + offset, "_eextratext")))
> + if ((s->addr == _etext && strcmp((char*)s->sym + offset, "_etext")) ||
> + (s->addr == _einittext && strcmp((char*)s->sym + offset, "_einittext")) ||
> + (s->addr == _eextratext && strcmp((char*)s->sym + offset, "_eextratext")))
> return 0;
> }
Can we have a local variable so we do not have all the casts in the if
condition?

Sam
-
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/

\
 
 \ /
  Last update: 2005-07-16 10:23    [W:0.098 / U:2.720 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site