GNU bug report logs - #11617
Don't use 'register'; these days it's not needed.

Previous Next

Package: emacs;

Reported by: Paul Eggert <eggert <at> cs.ucla.edu>

Date: Sun, 3 Jun 2012 06:10:01 UTC

Severity: wishlist

Tags: patch, wontfix

Done: Chong Yidong <cyd <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #23 received at 11617 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 11617 <at> debbugs.gnu.org
Subject: Re: bug#11617: Don't use 'register'; these days it's not needed.
Date: Mon, 04 Jun 2012 09:06:42 -0400
>> While `register' has been unnecessary for a while
>> now, I don't see why it would hurt.
> It hurts because it wastes developers' time, and has done so
> for many years.

No it doesn't and hasn't.

>      /* If we use `register' here, gcc-4.0.2 on amd64 using
>        -DUSE_LISP_UNION_TYPE complains further down that we're getting the
>        address of `force'.  Go figure.  */

Right, compilers have bugs, and we sometimes bump into them.  If it's
not with "register" it's with something else.

> This stuff is all bogus and our lives would be a tiny bit simpler
> if it vanished.

In my world, the keyword "register" is just another kind of whitespace.
So just like I don't want to install a patch that "cleans up the
whitespace" I'd rather not install your patch.  OTOH, just like for
whitespace, it's perfectly fine for regular patches to remove "register"
in the code they touch.


        Stefan




This bug report was last modified 12 years and 327 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.