You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm writing a plugin for Vim and while I'm not absolutely sure that this is the problem, I believe I've tracked it down to an interaction with the autocommands that the notes plugin adds (especially the CursorHold ones).
Is there a reason why all the autocommands in plugin/notes.vim are added for every file? Wouldn't it be more prudent to add them only for files with the 'notes' filetype (and then make them buffer-local with <buffer>)? That way all the functionality is preserved while the already crowded autocommand space is now less crowded. :)
This is also more efficient since the commands don't have to execute for irrelevant files. For instance, I doubt that it's necessary to call xolox#notes#refresh_syntax() on every CursorHold event when I'm editing a C++ file.
I love your plugins, keep up the great work!
The text was updated successfully, but these errors were encountered:
I'm writing a plugin for Vim and while I'm not absolutely sure that this is the problem, I believe I've tracked it down to an interaction with the autocommands that the notes plugin adds (especially the CursorHold ones).
Is there a reason why all the autocommands in plugin/notes.vim are added for every file? Wouldn't it be more prudent to add them only for files with the 'notes' filetype (and then make them buffer-local with
<buffer>
)? That way all the functionality is preserved while the already crowded autocommand space is now less crowded. :)This is also more efficient since the commands don't have to execute for irrelevant files. For instance, I doubt that it's necessary to call xolox#notes#refresh_syntax() on every CursorHold event when I'm editing a C++ file.
I love your plugins, keep up the great work!
The text was updated successfully, but these errors were encountered: