Fix error "E1502: Lua failed to grow stack to 4001" #1075
Merged
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.
The
neoVimBuffer
function was updated in #1015 to used a lua script throughnvimExecLua
. The script returns the dictionary retrieved from thegetbufinfo
vim functions. The issue is that this can grow very large, especially thevariables
entry, and can cause the lua stack to grow too big and error.In the end the
neoVimBuffer
function only needs a small handful of the entries fromgetbufinfo
, and so the lua script has been updated to return a dictionary with only those entries.At the same time the
hasDirtyBuffers
function was found to also return thegetbufinfo
dictionary and was similarly fixed. It was also noticed that the argument passed intogetbuinfo
here was a vim style dictionary and not valid lua. This madehasDirtyBuffers
fail and caused an issue inMainWindow.windowShouldClose
where selecting `FileFixes #1044