Re: [patch-2.3.99pre6-5] Include File Patches

From: Jens Axboe (axboe@suse.de)
Date: Sun Apr 23 2000 - 09:50:04 EST


On Sat, Apr 22 2000, Horst von Brand wrote:
> Bill Wendling <wendling@ganymede.isdn.uiuc.edu> said:
> > Here are some minor patches to the include files. I tried to compile the
> > kernel with -pedantic (and failed). I got several warnings about newlines
> > in strings being invalid in ANSI C. There were some text after an #endif
> > in the include/linux/raid files which should be comments. One of the
> > strangest errors was this:
> >
> > __u8 short slot_tablelen;
> >
> > in include/linux/cdrom.h. __u8 is typedef'ed as unsigned char. I'm
> > assuming this was meant to be unsigned short instead?

Oops, I don't know how that got in there. That is supposed to be a
__16, thanks for spotting this.

> I'd assume __u16 is meant here, i.e.. 16 bits worth of unsigned, not plain
> short.

Definately!

-- 
*  Jens Axboe <axboe@suse.de>
*  Linux CD/DVD-ROM, SuSE Labs
*  http://kernel.dk

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Apr 23 2000 - 21:00:22 EST