To: vim_dev@googlegroups.com Subject: Patch 7.4.2132 Fcc: outbox From: Bram Moolenaar Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ------------ Patch 7.4.2132 Problem: test_partial has memory leaks reported. Solution: Add a note about why this happens. Files: src/testdir/test_partial.vim *** ../vim-7.4.2131/src/testdir/test_partial.vim 2016-07-04 22:29:22.079959892 +0200 --- src/testdir/test_partial.vim 2016-07-30 23:12:47.636301103 +0200 *************** *** 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 *************** *** 270,294 **** unlet 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') --- 292,297 ---- *** ../vim-7.4.2131/src/version.c 2016-07-30 23:05:04.724678964 +0200 --- src/version.c 2016-07-30 23:13:57.875637229 +0200 *************** *** 765,766 **** --- 765,768 ---- { /* Add new patch number below this line */ + /**/ + 2132, /**/ -- ARTHUR: If you do not open these doors, we will take this castle by force ... [A bucket of slops land on ARTHUR. He tries to retain his dignity.] "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD /// Bram Moolenaar -- Bram@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 ///