On Fr, 22 Mär 2019, Christian Brabandt wrote:

> 
> On Fr, 22 Mär 2019, Bram Moolenaar wrote:
> 
> > How about with patch 1033 included?
> 
> This came to late, but the problem test 1033 fixed, did not appear on 
> the appveyor build. And the build with patch 1032 worked perfectly 
> again:
> https://github.com/vim/vim-win32-installer/releases/tag/v8.1.1032

I still see occasional failures. This has come up here:
https://ci.appveyor.com/project/chrisbra/vim-win32-installer/builds/23310233/job/fvn456ubjt25km7q

,----
| From test_memory_usage.vim:
| Found errors in Test_memory_func_capture_vargs():
| function RunTheTest[40]..Test_memory_func_capture_vargs line 22: Expected 
range 17712 - 35424, but got 17476
| TEST FAILURE
`----

and here:
https://ci.appveyor.com/project/chrisbra/vim-win32-installer/build/job/wfn13cxfi40bdpxi

,----
| From test_memory_usage.vim:
| Found errors in Test_memory_func_capture_vargs():
| function RunTheTest[40]..Test_memory_func_capture_vargs line 22: Expected 
range 17624 - 35248, but got 17540
| TEST FAILURE 
`----

Since this is a less than 2 % deviation of the lower limit, allow for 2% 
fluctuation of the lower bounds in Test_memory_func_capture_vargs:

diff --git a/src/testdir/test_memory_usage.vim 
b/src/testdir/test_memory_usage.vim
index 565a11554..e802f7ab1 100644
--- a/src/testdir/test_memory_usage.vim
+++ b/src/testdir/test_memory_usage.vim
@@ -97,9 +97,10 @@ func Test_memory_func_capture_vargs()
   let after = s:monitor_memory_usage(vim.pid)

   " Estimate the limit of max usage as 2x initial usage.
-  call assert_inrange(before, 2 * before, after.max)
-  " In this case, garbage collecting is not needed.  The value might fluctuate
-  " a bit, allow for 3% tolerance.
+  " Allow for a bit of fluctuation for the lower range
+  call assert_inrange(before * 98 / 100, 2 * before, after.max)
+  " In this case, garbage collecting is not needed.
+  " The value might fluctuate a bit, allow for 3% tolerance.
   let lower = after.last * 97 / 100
   let upper = after.last * 103 / 100
   call assert_inrange(lower, upper, after.max)


Best,
Christian
-- 
Die Philosophen sind wie Zahnärzte, die Löcher aufbohren, ohne sie
füllen zu können.
                -- Giovanni Guareschi

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Raspunde prin e-mail lui