GNU bug report logs - #8158
Definition of auto-mode-alist

Previous Next

Package: emacs;

Reported by: Reuben Thomas <rrt <at> sc3d.org>

Date: Wed, 2 Mar 2011 22:03:01 UTC

Severity: wishlist

Fixed in version 29.1

Done: Stefan Kangas <stefan <at> marxist.se>

Bug is archived. No further changes may be made.

Full log


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

From: Reuben Thomas <rrt <at> sc3d.org>
To: Lennart Borgman <lennart.borgman <at> gmail.com>
Cc: 8158 <at> debbugs.gnu.org
Subject: Re: bug#8158: Definition of auto-mode-alist
Date: Wed, 2 Mar 2011 22:22:47 +0000
On 2 March 2011 22:18, Lennart Borgman <lennart.borgman <at> gmail.com> wrote:
> On Wed, Mar 2, 2011 at 11:02 PM, Reuben Thomas <rrt <at> sc3d.org> wrote:
>> A comment in files.el says:
>>
>>  ;; Note: The entries for the modes defined in cc-mode.el (c-mode,
>>  ;; c++-mode, java-mode and more) are added through autoload
>>  ;; directives in that file.  That way is discouraged since it
>>  ;; spreads out the definition of the initial value.
>>
>> Isn't this a bit unmodular as Emacs continues to grow, and given loaddefs.el?
>>
>> If the maintainers agree, then the last sentence should be changed to
>> encourage the removal of the initial values back into the relevant
>> mode files.
>
> I think I disagree. This sort of information must be coordinated so it
> need to be in a central place.

Why does it have to be coordinated? The most obvious reason seems to
me "to avoid clashes", but this is detectable by parsing
auto-mode-alist. Generating a warning when there are clashing settings
for the same suffix would also be handy for 3rd party modes, which
cannot integrate their information in this way.

For modes that are part of Emacs, this system is fragile, as it's easy
to forget that part of the mode is in files.el.

-- 
http://rrt.sc3d.org




This bug report was last modified 3 years and 119 days ago.

Previous Next


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