Hello! 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 http://nginx.org/ _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-devel
