emacs-devel
[Top][All Lists]
Advanced

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

Re: Updated project-specific settings patch


From: David Kastrup
Subject: Re: Updated project-specific settings patch
Date: Mon, 19 May 2008 22:29:24 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux)

Stefan Monnier <address@hidden> writes:

> Better use the same buffer name *Local Variables*.
>
>> +       ((file-exists-p (concat dir ".dir-settings.el"))
>> +    (setq result (concat dir ".dir-settings.el")))
>
> In most cases, we'd prefer (expand-file-name ".dir-settings.el" dir)
> over the use of `concat'.  The only exception I know is when `dir' is
> relative and it's important for the result also be relative.

If the file name or directory has been read directly from disk and thus
may contain "~" or "~USER" as valid literal components, you don't want
to have those expanded.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum




reply via email to

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