Messages in this thread | | | From | Blaisorblade <> | Subject | Re: [patch 7/7] uml: resolve symbols in back-traces | Date | Thu, 28 Oct 2004 23:02:48 +0200 |
| |
On Thursday 28 October 2004 21:28, Chris Wedgwood wrote: > On Thu, Oct 28, 2004 at 08:34:21PM +0200, Blaisorblade wrote: > > And removing the final Emacs comment is not welcome (I don't care, > > but Jeff does. If that should be removed, that's a separate > > problem).
> the emacs comments are gratuitous and completely pointless, they serve > no useful purpose. fwiw in my .emacs i have:
> (defun cw-linux-c-mode () > "C mode with adjusted defaults for use with the Linux kernel." > (interactive) > (c-mode) > (c-set-style "linux")) > (setq auto-mode-alist > (append '(("wk/linux.*/.*\\.[ch]$" . cw-linux-c-mode)) > '(("/usr/src/linux.*/.*\\.[ch]$" . cw-linux-c-mode)) > auto-mode-alist)) > > which actually could be cleaned up a bit (it's been hacked over the > years and never cleaned up suitably) but the idea is pretty simple
I mostly agree on this, and thought it myself (I *never* use Emacs, only Vim, but no flames here).
However, let's be kind with Jeff, since he's the UML maintainer. You can be right, but there's also courtesy.
If Jeff says "OK", then someone can submit a separate patch removing all the final comments.
Until then, please don't.
And never mix such unrelated changes in a patch. -- Paolo Giarrusso, aka Blaisorblade Linux registered user n. 292729
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |