[Mesa-dev] [PATCH] radv: add .dir-locals.el to make radv code more readable in Emacs

Damien Grassart damien at grassart.com
Mon Mar 6 11:58:21 UTC 2017


On Mon, Mar 6, 2017 at 7:44 AM, Michel Dänzer <michel at daenzer.net> 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


More information about the mesa-dev mailing list