[PATCH] Contrib: vim syntax, support block region

Maxim Dounin mdounin at mdounin.ru
Fri Apr 21 13:46:45 UTC 2017


On Fri, Apr 21, 2017 at 04:39:03PM +0800, OOO wrote:

> Actually nginx.vim supports on and off[1] before.
> And some option also been highlighted because there are directives
> with the same name, ex: ssl.
> So if we complete remove this.
> It will looks like the function is missing.

Yes, but this highlighting is not directive-specific.  Nothing 
stops from highlighting some parameters in all directives, and 
this is what I did in the PoC to preserve compatibility with the 
previous versions of highlighting.

On the other hand, I really think that highlighting of "on / off" 
in directives which do not handle these parameters is a 
misfeature, and in many cases it misleads people to think that a 
given directive does support on / off parameters.

Maxim Dounin

More information about the nginx-devel mailing list