Do not change after-change-functions when jedi:install-imenu is nil. #235
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.
If I understand the code correctly, the purpose of
jedi:after-change-handler
is to change the value ofjedi:defined-names--cache
.And this value is used for only functions which is related to
jedi
andimenu
.So if
jedi:install-imenu
isnil
,jedi:after-change-handler
should not be added toafter-change-functions
. I think this commit will fix #234.