Horst H. von Brand, Sun, Jan 14, 2007 01:42:57 +0100: > Alex Riesen <fork0@t-online.de> wrote: > > Junio C Hamano, Sat, Jan 13, 2007 02:31:35 +0100: > > > +/* High bit set, or ISO-2022-INT */ > > > +static int non_ascii(int ch) > > > +{ > > > + ch = (ch & 0xff); > > > + return ((ch & 0x80) || (ch == 0x1b)); > > > +} > > > + > > > > "return (ch & 0x0x80) || (ch & 0xff) == 0x1b;" :) > ^^ Oops :) > Is the same, if ch == 0x9b, it will match the first part anyway. So it should. 0x9b isn't ASCII. > The outer parentesis can (should?) go. It's "question of style", I'm afraid :) - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Mon Jan 15 06:26:58 2007
This archive was generated by hypermail 2.1.8 : 2007-01-15 06:28:23 EST