Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modify function-call cleanup #3961

Closed
wants to merge 5 commits into from
Closed
Diff settings

Always

Just for now

Modify "made_copy" limit: base on pagesize * 1024

  • Loading branch information...
ichizok committed Feb 13, 2019
commit b5591572b8bca872ca61a2ab6124ee82f7e73ae6
Copy path View file
@@ -699,7 +699,10 @@ cleanup_function_call(funccall_T *fc)
fc->caller = previous_funccal;
previous_funccal = fc;

if (++made_copy == 10000)
if (want_garbage_collect)
// If garbage collector is ready, clear count.
made_copy = 0;
else if (++made_copy >= (4096 * 1024) / sizeof(*fc))
{
// We have made a lot of copies. This can happen when
// repetitively calling a function that creates a reference to
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.