[#154] avoid `execute()` in `capture_highlight`

preservim / vim-indent-guides

A Vim plugin for visually displaying indent levels in code

MIT License

2.62k stars 162 forks source link

[#154] avoid `execute()` in `capture_highlight` #158

Open Boolean263 opened 1 year ago

Boolean263 commented 1 year ago

Instead of using execute() to get the foreground and background colours of a colour scheme, use synIDattr() and related functions. This prevents an E12 error when using this plugin in neovim to edit a file which has a modeline.

Tested in neovim 0.9.0 and vim 8.2.

Closes #154

huyz commented 3 months ago

Thanks @Boolean263, this PR fixes my problem whenever I open help in neovim 0.10.0:

Error detected while processing modelines[272]..OptionSet Autocommands for "tabstop"..function indent_guides#process_autoc
mds[2]..indent_guides#enable[8]..indent_guides#init_script_vars[7]..indent_guides#capture_highlight:
line    1:
E12: Command not allowed in secure mode in current dir or tag search
E12: Command not allowed in secure mode in current dir or tag search