From: Gabriel Paubert <paubert@iram.es>
To: viro@parcelfarce.linux.theplanet.co.uk
Cc: tom st denis <tomstdenis@yahoo.com>,
Christoph Hellwig <hch@infradead.org>,
linux-kernel@vger.kernel.org
Subject: Re: [OT] Re: 0xdeadbeef vs 0xdeadbeefL
Date: Thu, 8 Jul 2004 13:55:11 +0200 [thread overview]
Message-ID: <20040708115511.GA4391@iram.es> (raw)
In-Reply-To: <20040708111521.GK12308@parcelfarce.linux.theplanet.co.uk>
On Thu, Jul 08, 2004 at 12:15:21PM +0100, viro@parcelfarce.linux.theplanet.co.uk wrote:
> On Thu, Jul 08, 2004 at 11:32:50AM +0200, Gabriel Paubert wrote:
>
> > Yes, I know and I use BK. But given the fact that you insult me for
> > better knowing C rules than you, I'm seriously considering switch
> > to subversion or arch instead.
> >
> > Argh, I've mentioned BK. There should be a Goldwin's law equivalent
> > for BitKeeper on lkml ;-)
>
> Godwin, surely?
Right, should have looked the name up.
>
> Anyway, if you think that suckversion authors knew C... Try to read their
> decoder/parser/whatever you call the code handling the data stream obtained
> from other end of connection. _Especially_ when it comes to signedness
> (of integers, mostly).
I did not read it, neither did I read BitKeeper's code for
obvious reasons.
>
> > - the aforementioned fgetc/getc/getchar issues.
>
> ... have nothing to do with char; getc() has more legitimate return values
> than char can represent.
Only one more, unless I missed something.
> No matter whether it's signed or unsigned, if
> you have
> ... char c;
> ...
> c = getc();
> you have a bug - Dirichlet Principle bites you anyway.
Indeed, but unfortunately you don't hit the problem with
pure ASCII on x86. That's one of the reasons for which
a lot of code ported to archs where char is unsigned by
default is simply compiled with -fsigned-char instead of
correcting the bugs.
The remaining case (char 0xff) is infrequent, at least
for Latin-[19] encodings in the languages I know, and
never happens with UTF-8 AFAICT.
Heck, even the 2.4 ppc kernel is compiled with
-fsigned-char, for $DEITY's sake.
Regards,
Gabriel
next prev parent reply other threads:[~2004-07-08 12:00 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-07-06 21:56 0xdeadbeef vs 0xdeadbeefL David Eger
2004-07-07 0:06 ` tom st denis
2004-07-07 3:00 ` viro
2004-07-07 11:10 ` tom st denis
2004-07-07 11:18 ` Prohibited attachment type (was 0xdeadbeef) Richard B. Johnson
2004-07-07 11:48 ` tom st denis
2004-07-07 12:29 ` Jakub Jelinek
2004-07-08 5:52 ` Pavel Machek
2004-07-08 14:03 ` Jakub Jelinek
2004-07-07 12:13 ` R. J. Wysocki
2004-07-07 14:22 ` 0xdeadbeef vs 0xdeadbeefL viro
2004-07-07 18:47 ` tom st denis
2004-07-07 16:30 ` Gabriel Paubert
2004-07-07 18:41 ` tom st denis
2004-07-07 18:47 ` Christoph Hellwig
2004-07-07 18:53 ` tom st denis
2004-07-07 23:17 ` Harald Arnesen
2004-07-08 6:15 ` David Weinehall
2004-07-08 9:32 ` [OT] " Gabriel Paubert
2004-07-08 11:15 ` viro
2004-07-08 11:55 ` Gabriel Paubert [this message]
2004-07-08 16:41 ` Andries Brouwer
2004-07-08 17:13 ` Michael Driscoll
2004-07-08 17:16 ` Horst von Brand
2004-07-10 1:52 ` Andrew Rodland
2004-07-07 0:38 ` Richard B. Johnson
2004-07-07 4:52 ` David Eger
2004-07-07 11:40 ` Richard B. Johnson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20040708115511.GA4391@iram.es \
--to=paubert@iram.es \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tomstdenis@yahoo.com \
--cc=viro@parcelfarce.linux.theplanet.co.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.