search

Shougo / neosnippet.vim

neo-snippet plugin
Other
1.12k stars 108 forks source link

Use execute() instead of :redir #482

Closed tyru closed 4 years ago

tyru commented 4 years ago

In my environment, this function is called from execute().

Error detected while processing function neosnippet#mappings#_clear_select_mode_mappings:
line    5:
E930: Cannot use :redir inside execute()

I didn't check in detail though, I confirmed that execute() is implemented before Vim 8.0. Generally speaking, execute() is better than :redir.

tyru commented 4 years ago

vimlint shows wrong errors on CI (false-positive)

autoload/neosnippet/mappings.vim:36:18:Error: E118: Too many arguments for function: execute
Shougo commented 4 years ago

OK. I will fix it.