Patch: disallow encoding in modeline.

2008-12-09 Fir de Conversatie Patrick Texier
Attached patch disallows 'encoding' in a modeline but not in secure
mode.
Perharps some others options could use this flag.

 
-- 
Patrick Texier

--~--~-~--~~~---~--~~
You received this message from the vim_dev maillist.
For more information, visit http://www.vim.org/maillist.php
-~--~~~~--~~--~--~---



enc.patch
Description: 'encoding' patch


Re: Patch: disallow encoding in modeline.

2008-12-09 Fir de Conversatie Matt Wozniski

On Tue, Dec 9, 2008 at 3:40 AM, Patrick Texier wrote:
 Attached patch disallows 'encoding' in a modeline but not in secure
 mode.

Looks like a good idea to me, especially since the vim help explicitly
says that 'encoding' should either be set in ~/.vimrc or not at all...
 In fact, I think it shouldn't be usable in a modeline even if not in
secure mode...

~Matt

--~--~-~--~~~---~--~~
You received this message from the vim_dev maillist.
For more information, visit http://www.vim.org/maillist.php
-~--~~~~--~~--~--~---



Re: Patch: disallow encoding in modeline.

2008-12-09 Fir de Conversatie Patrick Texier

On Tue, 9 Dec 2008 04:31:08 -0500, Matt Wozniski wrote:

 On Tue, Dec 9, 2008 at 3:40 AM, Patrick Texier wrote:
  Attached patch disallows 'encoding' in a modeline but not in secure
  mode.
 
 Looks like a good idea to me, especially since the vim help explicitly
 says that 'encoding' should either be set in ~/.vimrc or not at all...
  In fact, I think it shouldn't be usable in a modeline even if not in
 secure mode...

No, you can never set encoding in a modeline but, if you're in secure
mode you can set encoding using ex command-line.

In Vim 7.2.65, if you block an option, you block it for modelines and
secure mode ex command-line.
-- 
Patrick Texier

--~--~-~--~~~---~--~~
You received this message from the vim_dev maillist.
For more information, visit http://www.vim.org/maillist.php
-~--~~~~--~~--~--~---