Re: Patch 8.1.1027

2019-03-21 Fir de Conversatie Bram Moolenaar


Elimar Riesebieter wrote:

> * Bram Moolenaar  [2019-03-20 22:45 +0100]:
> 
> > 
> > Patch 8.1.1027
> > Problem:Memory usage test sometimes fails.
> > Solution:   Use 80% of before.last as the lower limit. (Christian Brabandt)
> > Files:  src/testdir/test_memory_usage.vim
> 
> Building 8.1.1027 on linux i686 I get:
> >From test_memory_usage.vim:
> Found errors in Test_memory_func_capture_lvars():
> function RunTheTest[40]..Test_memory_func_capture_lvars line 31: Expected 
> range 10115 - 27580, but got 27596
> TEST FAILURE

Just a tiny bit over the limit.  We can tolerate 1%.

-- 
>From "know your smileys":
 :-&Eating spaghetti

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: Patch 8.1.1027

2019-03-21 Fir de Conversatie Christian Brabandt


On Do, 21 Mär 2019, Elimar Riesebieter wrote:

> * Bram Moolenaar  [2019-03-20 22:45 +0100]:
> 
> > 
> > Patch 8.1.1027
> > Problem:Memory usage test sometimes fails.
> > Solution:   Use 80% of before.last as the lower limit. (Christian Brabandt)
> > Files:  src/testdir/test_memory_usage.vim
> 
> Building 8.1.1027 on linux i686 I get:
> From test_memory_usage.vim:
> Found errors in Test_memory_func_capture_lvars():
> function RunTheTest[40]..Test_memory_func_capture_lvars line 31: Expected 
> range 10115 - 27580, but got 27596
> TEST FAILURE

So in your case the upper limit is not correct.

Best,
Christian
-- 
Das Erlebte weiß jeder zu schätzen, am meisten der Denkende und
Nachsinnende im Alter; er fühlt mit Zuversicht und Behaglichkeit,
dass ihm das niemand rauben kann.
-- Goethe, Maximen und Reflektionen, Nr. 10

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: Patch 8.1.1027

2019-03-21 Fir de Conversatie Elimar Riesebieter
* Elimar Riesebieter  [2019-03-21 16:59 +0100]:

> * Bram Moolenaar  [2019-03-20 22:45 +0100]:
> 
> > 
> > Patch 8.1.1027
> > Problem:Memory usage test sometimes fails.
> > Solution:   Use 80% of before.last as the lower limit. (Christian Brabandt)
> > Files:  src/testdir/test_memory_usage.vim
> 
> Building 8.1.1027 on linux i686 I get:
> >From test_memory_usage.vim:
> Found errors in Test_memory_func_capture_lvars():
> function RunTheTest[40]..Test_memory_func_capture_lvars line 31: Expected 
> range 10115 - 27580, but got 27596
> TEST FAILURE

ppc32 and amd64 builds fine, though

Elimar
-- 
  "Talking much about oneself can also
   be a means to conceal oneself."
 -Friedrich Nietzsche

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: Patch 8.1.1027

2019-03-21 Fir de Conversatie Elimar Riesebieter
* Bram Moolenaar  [2019-03-20 22:45 +0100]:

> 
> Patch 8.1.1027
> Problem:Memory usage test sometimes fails.
> Solution:   Use 80% of before.last as the lower limit. (Christian Brabandt)
> Files:src/testdir/test_memory_usage.vim

Building 8.1.1027 on linux i686 I get:
>From test_memory_usage.vim:
Found errors in Test_memory_func_capture_lvars():
function RunTheTest[40]..Test_memory_func_capture_lvars line 31: Expected range 
10115 - 27580, but got 27596
TEST FAILURE

Elimar
-- 
  From The Collaborative International Dictionary of English v.0.48 [gcide]:
  .
  arsehole \arse"hole`\ ([aum]rs"h[=o]l`), n.
 1. execretory opening at the end of the alimentary canal.

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Patch 8.1.1027

2019-03-20 Fir de Conversatie Bram Moolenaar


Patch 8.1.1027
Problem:Memory usage test sometimes fails.
Solution:   Use 80% of before.last as the lower limit. (Christian Brabandt)
Files:  src/testdir/test_memory_usage.vim


*** ../vim-8.1.1026/src/testdir/test_memory_usage.vim   2019-03-14 
13:42:57.169435481 +0100
--- src/testdir/test_memory_usage.vim   2019-03-20 22:43:43.280574636 +0100
***
*** 137,143 
  let last = s:monitor_memory_usage(vim.pid).last
endfor
  
!   call assert_inrange(before, after.max + (after.last - before), last)
  
call vim.stop()
call delete(testfile)
--- 137,145 
  let last = s:monitor_memory_usage(vim.pid).last
endfor
  
!   " The usage may be a bit less than the last value 
!   let lower = before * 8 / 10
!   call assert_inrange(lower, after.max + (after.last - before), last)
  
call vim.stop()
call delete(testfile)
*** ../vim-8.1.1026/src/version.c   2019-03-20 21:42:16.832737656 +0100
--- src/version.c   2019-03-20 22:36:25.828348477 +0100
***
*** 781,782 
--- 781,784 
  {   /* Add new patch number below this line */
+ /**/
+ 1027,
  /**/

-- 
>From "know your smileys":
 8-O"Omigod!!" (done "rm -rf *" ?)

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org///
 \\\help me help AIDS victims -- http://ICCF-Holland.org///

-- 
-- 
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 vim_dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.