emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: files.el: Patch to make project-settings optional/customizable


From: Dan Nicolaescu
Subject: Re: files.el: Patch to make project-settings optional/customizable
Date: Sat, 22 Nov 2008 11:40:22 -0800 (PST)

Juri Linkov <address@hidden> writes:

  > >> So let's get rid of "project" in the function names and use the common
  > >> prefix dir-local-variables by analogy with file-local-variables.
  > >
  > > Sounds OK.
  > >
  > >> Since the only term the user have to remember will be "directory-local
  > >> variables" this suggests using a file name derived from it.  Since
  > >> the file name should be short a good candidate is ".dir-locals.el".
  > >
  > > Fine by me,
  > 
  > Done.

Thanks for doing this!

  > Since seeing constant file names repeated in several places hurts my eyes,
  > I also created defconst for the file name, as Richard suggested - with the
  > docstring saying why it is constant.

Since dir-locals-file is used in exactly one function, it would be
better to just let bind it there.  
And defconsts are not exactly constants in elisp, so if someone does a
(setq dir-locals-file BLAH), the problem that started this whole
discussion still happens.  More, we don't have defconsts for 
"Local Variables"  or "-*-" which serve a similar purpose but at file
level.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]