Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] Contrib: vim syntax, support block region

OOO
April 19, 2017 05:54AM
Hi Dounin

I have looked into your POC.
I use this approach in yajs.vim too.
The major problem of this approach is: It creates (sort of) redundant
syntax group.
For the listen directive:

> syn keyword ngxDirectiveImportantListen listen
> \ nextgroup=@ngxListenParams skipwhite skipempty
> syn match ngxListenParam '\([^;{ \t\\]\|\\.\)\+'
> \ contained
> \ nextgroup=@ngxListenParams skipwhite skipempty
> syn region ngxListenString start=+\z(["']\)+ end=+\z1+ skip=+\\\\\|\\\z1+
> \ contained
> \ nextgroup=@ngxListenParams skipwhite skipempty
> syn match ngxListenComment '#.*$'
> \ contained
> \ nextgroup=@ngxListenParams skipwhite skipempty
> syn keyword ngxListenOptions contained
> \ default_server ssl http2 spdy proxy_protocol
> \ setfib fastopen backlog rcvbuf sndbuf accept_filter deferred bind
> \ ipv6only reuseport so_keepalive keepidle
> \ nextgroup=@ngxListenParams skipwhite skipempty
> syn cluster ngxListenParams
> \ contains=ngxListenParam,ngxListenString,ngxListenComment
> \ add=ngxListenOptions

The ngxListenParam, ngxListenString, ngxListenComment and
ngxListenParams are the redundant groups.
For every directive we want to highlight params.
We will have to add four more syntax group like this one.
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH 1 of 4] Contrib: vim syntax, allow multiline config by skipempty

othree 702 March 03, 2017 05:58AM

[PATCH 3 of 4] Contrib: vim syntax, highlight HTTP status code

othree 345 March 03, 2017 05:58AM

[PATCH 2 of 4] Contrib: vim syntax, highlight rewrite flag

othree 320 March 03, 2017 05:58AM

[PATCH 4 of 4] Contrib: vim syntax, remove unecessary ngxBlock

othree 293 March 03, 2017 05:58AM

Re: [PATCH 4 of 4] Contrib: vim syntax, remove unecessary ngxBlock

Maxim Dounin 286 March 03, 2017 09:06AM

Re: [PATCH 4 of 4] Contrib: vim syntax, remove unecessary ngxBlock

OOO 338 March 03, 2017 10:40AM

Re: [PATCH 4 of 4] Contrib: vim syntax, remove unecessary ngxBlock

Maxim Dounin 336 March 03, 2017 12:26PM

[PATCH] Contrib: vim syntax, support block region

othree 345 March 04, 2017 07:00AM

Re: [PATCH] Contrib: vim syntax, support block region

OOO 342 March 10, 2017 11:46AM

Re: [PATCH] Contrib: vim syntax, support block region

Maxim Dounin 320 March 10, 2017 11:54AM

Re: [PATCH] Contrib: vim syntax, support block region

OOO 293 March 10, 2017 11:58AM

Re: [PATCH] Contrib: vim syntax, support block region

OOO 286 April 19, 2017 05:54AM

Re: [PATCH] Contrib: vim syntax, support block region

Maxim Dounin 249 April 19, 2017 11:12AM

Re: [PATCH] Contrib: vim syntax, support block region

OOO 258 April 21, 2017 04:40AM

Re: [PATCH] Contrib: vim syntax, support block region

Maxim Dounin 326 April 21, 2017 09:48AM

Re: [PATCH 4 of 4] Contrib: vim syntax, remove unecessary ngxBlock

OOO 353 March 04, 2017 07:20AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 133
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 500 on July 15, 2024
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready