Re: Particular Regex Appears Region Broken in 7.4

2013-08-23 Fir de Conversatie Amadeus Demarzi
I can confirm that Patch 2 fixes this! Thanks so much! Seriously! On Wednesday, August 21, 2013 11:45:52 AM UTC-7, Amadeus Demarzi wrote: Just curious if any headway has been made on this lookbehind regex issue? I only ask because it breaks a pretty popular JS syntax plugin. On Tuesday,

Re: Particular Regex Appears Region Broken in 7.4

2013-08-21 Fir de Conversatie Amadeus Demarzi
Just curious if any headway has been made on this lookbehind regex issue? I only ask because it breaks a pretty popular JS syntax plugin. On Tuesday, August 13, 2013 5:34:52 PM UTC-7, Amadeus Demarzi wrote: Alright, so I am admittedly terrible with Regexes, but I think I did find something.

Particular Regex Appears Region Broken in 7.4

2013-08-13 Fir de Conversatie Amadeus Demarzi
With the update to vim 7.4, it seems as if this particular regex just doesn't match in particular cases in 7.4, but it worked fine in 7.3. syntax region jsRegexpString start=+\(\(\(return\|case\)\s\+\)\@=\|\(\([)\]']\|\d\|\w\)\s*\)\@!\)/\(\*\|/\)\@!+ skip=+\|\\/+ end=+/[gimy]\{,4}+

Re: Particular Regex Appears Region Broken in 7.4

2013-08-13 Fir de Conversatie Bram Moolenaar
Amadeus Demarzi wrote: With the update to vim 7.4, it seems as if this particular regex just doesn't match in particular cases in 7.4, but it worked fine in 7.3. syntax region jsRegexpString start=+\(\(\(return\|case\)\s\+\)\@=\|\(\([)\]']\|\d\|\w\)\s*\)\@!\)/\(\*\|/\)\@!+

Re: Particular Regex Appears Region Broken in 7.4

2013-08-13 Fir de Conversatie Amadeus Demarzi
I am pretty bad with regex stuff, but I will try to figure out how to isolate it. I posted it here quickly just in case there was something silly I was missing in that snippet. For what it's worth, it's used in this syntax plugin: http://github.com/pangloss/vim-javascript And setting the

Re: Particular Regex Appears Region Broken in 7.4

2013-08-13 Fir de Conversatie Amadeus Demarzi
Alright, so I am admittedly terrible with Regexes, but I think I did find something. Using this file for the regex test: https://gist.github.com/amadeus/82d0288a9b8b2e3ff2a9 And using this regex in search, with hlsearch on: \(\(\(return\|case\)\s\+\)\@=\|\(\([)\]']\|\d\|\w\)\s*\)\@!\)\/ Both