lkml.org 
[lkml]   [2005]   [Jul]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH] signed char fixes for scripts
    From
    Date
    On Thu, 2005-07-28 at 11:02 +0100, Paulo Marques wrote:
    > 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?

    "unsigned char *" is simply the wrong type for mere text strings. "char
    *" ist the corrrect one. These are BTW two completely different types
    (yes, "char" can be promoted into an "unsigned char" but essentially
    these are two completely different types like "int" and "long long *").

    > Is my compiler version the problem (3.3.2), or are you testing with the

    Compiler version - zse gcc-4.*.

    Bernd
    --
    Firmix Software GmbH http://www.firmix.at/
    mobil: +43 664 4416156 fax: +43 1 7890849-55
    Embedded Linux Development and Services

    -
    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-28 12:24    [W:4.101 / U:0.368 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site