]> granicus.if.org Git - vim/commitdiff
patch 7.4.2132 v7.4.2132
authorBram Moolenaar <Bram@vim.org>
Sat, 30 Jul 2016 21:14:28 +0000 (23:14 +0200)
committerBram Moolenaar <Bram@vim.org>
Sat, 30 Jul 2016 21:14:28 +0000 (23:14 +0200)
Problem:    test_partial has memory leaks reported.
Solution:   Add a note about why this happens.

src/testdir/test_partial.vim
src/version.c

index b43f34902c9786cfeeb586bc28e19828d22c01f7..6b098c2ddd863eb012b89a61766796f92f50e58d 100644 (file)
@@ -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')
index c28c38eb1e90f5783b981c2afa9d61302fda4118..ea7c7b9297f654358963e16e667f5f7486cde115 100644 (file)
@@ -763,6 +763,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    2132,
 /**/
     2131,
 /**/