[lkml]   [2002]   [Jul]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [patch] Input cleanups for 2.5.29 [2/2]
    On Tue, Jul 30, 2002 at 02:20:36PM -0700, Linus Torvalds wrote:

    > On Tue, 30 Jul 2002, Greg KH wrote:
    > > On Tue, Jul 30, 2002 at 03:23:42PM +0200, Vojtech Pavlik wrote:
    > > > -#include <asm/types.h>
    > > > +#include <stdint.h>
    > >
    > > Why? I thought we were not including any glibc (or any other libc)
    > > header files when building the kernel?
    > Indeed. This is unacceptable.
    > Especially as the standard types are total crap, and the u8 etc are a lot
    > more readable. People should realize:
    > - the "int" is superfluous. Of _course_ it's an integer. If it was a
    > floating point number, it would be fp16/fp32/fp64/fp80/whatever.
    > - the "_t" is there only for namespace collisions, sane people can chose
    > to ignore it.
    > What do you have left after you have removed the crap? Yup. u8, u16, etc.
    > And if you want to share with user space, there's the long-accepted
    > namespace collision avoidance of prepending two underscores.
    > Fix it, Vojtech.
    > Linus

    I will, and will do so happily. I don't like the uint*_t types as well.
    This change was pushed very heavily for by Brad Hards, based on a
    conclusion of a rather lengthy discussion (I think on linux-usb) on
    which types should be used.

    Now the question remaining is how to fix that? You can just skip the
    patch. I've tried a 'bk undo', but that complains about unmerged leaves
    in that case (though really nothing depends on those changes). Or should
    I just make another cset on top of all the previous?

    Vojtech Pavlik
    SuSE Labs
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:27    [W:0.023 / U:4.728 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site