[PATCH xkbcomp 2/2] Fix incorrect processing of char(0xff) in comments
Michel Dänzer
michel at daenzer.net
Mon Jan 17 01:40:18 PST 2011
On Son, 2011-01-16 at 12:27 +0100, Dirk Wallenstein wrote:
> On Sun, Jan 16, 2011 at 01:26:10AM +0500, Alexandr Shadchin wrote:
> > If type readBuf is char, then scanchar() return -1 for char(0xFF) and EOF.
> > If type readBuf is unsigned char, then scanchar() return 0xFF for char(0xFF)
> > and -1 for EOF.
>
> NAK: I assume you want to correct the case when returning EOF stored
> inside of readBuf? This would now return EOF as 0x00ff but EOF is a
> negative value. All parts that access readBuf work with signed values.
FYI, 'char' on its own may be signed or unsigned depending on the
implementation.
--
Earthling Michel Dänzer | http://www.vmware.com
Libre software enthusiast | Debian, X and DRI developer
More information about the xorg-devel
mailing list