On Mon, Mar 6, 2017 at 7:44 AM, Michel Dänzer <[email protected]> wrote: > This doesn't seem consistent with existing .dir-locals.el files in the > tree. If any of those files match the Linux kernel coding style as well, > it would be good to make them all consistent one way or another.
Hi Michel, you're right, I will copy one of the existing .dir-locals.el to keep it consistent and update the ones that appear to follow a Linux kernel style rather than the Stroustrup style. Also, I've found that setting those variables for cc-mode rather than the more generic prog-mode prevents Emacs from warning about potentially unsafe values whenever opening a file. Lastly, I couldn't help but notice that .dir-locals.el is in .gitignore requiring a forced add into git. So I was wondering if that should be removed as well or is that for people to keep their own customizations outside of the repo? > Please add a corresponding .editorconfig file as well. Ok, I'll copy the one from the radeonsi driver that sets indents to tabs. Thanks, -Damien _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
