Mike, thanks for bringing this up! Huge +1 from me. For posterity, here’s the bug I filed almost a year ago (time flies!): https://bugzilla.mozilla.org/show_bug.cgi?id=957564 <https://bugzilla.mozilla.org/show_bug.cgi?id=957564>
Cheers, Another Mike. > On 17 Jun 2015, at 12:53, Mike Hommey <m...@glandium.org> wrote: > > Hi, > > The following post visible on planet, prompted a discussion on a french > irc channel. > http://www.otsukare.info/2015/06/17/mozilla-central-sublimetext > > What we currently have in the tree is essentially modelines for vim and > emacs. But: > - most vim installations have modelines disabled by default because of > the security implications: https://lwn.net/Articles/20249/. > - emacs has the same security implication, and AIUI, opening a file with > a modeline makes emacs ask questions to the user. Or ignore them in > batch mode. > - they add 2 or 3 lines of boilerplate to every file in the tree. > - they are not being added to every file in the tree. > > Once you realize all the above, you wonder what good those modelines are > doing, and if they are worth keeping. > > OTOH, there is http://editorconfig.org/ which works per directory > (recursively, so if everything is setup uniformly, you only need one > file at the top source tree), is supported by default by a few editors, > has plugins for many other editors (including sublime text, vim, and > emacs), and doesn't have the security issues that make modelines ignored > in most cases. > > So how about removing modelines and adding editorconfig files? > > Mike > _______________________________________________ > dev-platform mailing list > dev-platform@lists.mozilla.org > https://lists.mozilla.org/listinfo/dev-platform _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
