changeset 9713:058e5ea0400e v7.4.2132

commit https://github.com/vim/vim/commit/31440a1f2b6ae8a2f7ffdd4a8a3661eadc4df23c Author: Bram Moolenaar <Bram@vim.org> Date: Sat Jul 30 23:14:28 2016 +0200 patch 7.4.2132 Problem: test_partial has memory leaks reported. Solution: Add a note about why this happens.
author Christian Brabandt <cb@256bit.org>
date Sat, 30 Jul 2016 23:15:06 +0200
parents 3677deb0fc62
children 7cccb8851a96
files src/testdir/test_partial.vim src/version.c
diffstat 2 files changed, 24 insertions(+), 19 deletions(-) [+]
line wrap: on
line diff
--- a/src/testdir/test_partial.vim
+++ b/src/testdir/test_partial.vim
@@ -1,4 +1,26 @@
 " Test binding arguments to a Funcref.
+ 
+" NOTE: This function may cause memory leaks to be reported.
+" That is because when fork/exec fails memory is not freed.  Since the process
+" exists right away it's not a real leak.
+func Test_job_start_fails()
+  if has('job')
+    let job = job_start('axdfxsdf')
+    for i in range(100)
+      let status = job_status(job)
+      if status == 'dead' || status == 'fail'
+	break
+      endif
+      sleep 10m
+    endfor
+    if has('unix')
+      call assert_equal('dead', job_status(job))
+    else
+      call assert_equal('fail', job_status(job))
+    endif
+    unlet job
+  endif
+endfunc
 
 func MyFunc(arg1, arg2, arg3)
   return a:arg1 . '/' . a:arg2 . '/' . a:arg3
@@ -271,25 +293,6 @@ func Test_cycle_partial_job()
   endif
 endfunc
 
-func Test_job_start_fails()
-  if has('job')
-    let job = job_start('axdfxsdf')
-    for i in range(100)
-      let status = job_status(job)
-      if status == 'dead' || status == 'fail'
-	break
-      endif
-      sleep 10m
-    endfor
-    if has('unix')
-      call assert_equal('dead', job_status(job))
-    else
-      call assert_equal('fail', job_status(job))
-    endif
-    unlet job
-  endif
-endfunc
-
 func Test_ref_job_partial_dict()
   if has('job')
     let g:ref_job = job_start('echo')
--- a/src/version.c
+++ b/src/version.c
@@ -764,6 +764,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    2132,
+/**/
     2131,
 /**/
     2130,