Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Don't use
exists("g:...")
then==
to lookup globals, use oneget()
. This is faster.exec
to set known variablesSetting variables using
exec
only makes sense if the variable name comes from the value of another variable, not how it was being used. Aside from omitting dangerous behavior (exec
=~eval
), it's slightly faster.&background
as this can cause vim to re-load the color scheme.According to the documentation, this causes the color scheme to be
loaded TWICE.
Essentially, color schemes should not set the value of &background.
I tested these changes on vim 8.0.1453 and they appeared to work with no syntax errors, and color scheme loaded correctly.