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


View this message in rfc822 format

From: Chong Yidong <cyd <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Paul Eggert <eggert <at> cs.ucla.edu>, 11617 <at> debbugs.gnu.org
Subject: bug#11617: Don't use 'register'; these days it's not needed.
Date: Sun, 03 Jun 2012 22:50:02 +0800
Eli Zaretskii <eliz <at> gnu.org> writes:

>> Attached is a patch to remove 'register' from the mainline
>> Emacs source code.
>
> FWIW, I see no compelling reason for this change, even after I've read
> the explanations.  This keyword is a cue to the compiler, so it in
> general conveys non-trivial information.  If today's compilers mostly
> ignore that, it doesn't mean this will always be the state of affairs.
> However, by removing this, we lose those cues forever.

I wonder how many of those register variables were actually added after
profiling to see if they make any difference.  My guess is none,
i.e. those keywords are bogus.




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.