fix hightlighting issue when working with ack options

[mannih]

I came across this bug today: set g:ackhighlight = 1 and do
:Ack -i something

This will result in every "w" being highlighted in your current window
and in the quick fix list. I "fixed" this by trying to strip out any
options before the actual search string.

The code is a bad hack and if you happen to also specifiy directories
for ack to search in, they will still be highlighted. But the code kind
of works and I didn't want to simply open an issue.

[kassio]

First of all, thank's for your contribution. But I guess this is a little complicated.

I guess we could solve this only changing the regex to ignore the words that are prefixed with -, something like that:

let @/ = matchstr(a:args, "\\v[^-]\<\\zs\\w+\>|['\"]\\zs[^\"]+\\ze['\"]")

WDYT?

[mannih]

Nice approach. Much better than mine. It does work for the case where I provide options to ack, but it fails for the option-less case.

I'll play with this for a bit. Thank you!

[kassio]

option-less case? I didn't get it?

[mannih]

Sorry.

This works:
:Ack -i test
"test" gets highlighted.

But this doesn't work:
:Ack test

[kassio]

My bad, try this one:

let @/ = matchstr(a:args, "\\v[^-]\\zs\\w+\>|['\"]\\zs[^\"]+\\ze['\"]")

[mannih]

Much better now, but a little bit of the bug still remains:

:Ack test

will only highlight est and skip the t.

:Ack -i test

works just fine.

This matchstr seems to fix it completely:

let @/ = matchstr(a:args, "\\v\\zs[^- ]\\w+\>|['\"]\\zs[^\"]+\\ze['\"]")

I pulled the \\zs to the front and included a space in the character class that should not be matched.