Messages in this thread | | | Date | Fri, 4 Jul 2008 13:41:59 -0600 | From | Jonathan Corbet <> | Subject | Re: [PATCH -mm] documentation: update CodingStyle tips for Emacs users |
| |
On Fri, 04 Jul 2008 19:52:16 +0200 Johannes Weiner <hannes@saeurebad.de> wrote:
> +(add-hook 'c-mode-hook > + (lambda () > + (let ((filename (buffer-file-name))) > + ;; Enable kernel mode for the appropriate files > + (when (and file > + (string-match "/usr/src/linux" filename))
Shouldn't that be "(and filename ..." rather than "file"?
A test for /usr/src/linux seems misplaced, though; I don't think a whole lot of people keep their kernel trees there anymore. There must be a better heuristic one can use to figure out whether a kernel source file is being edited.
jon
| |