Nikolay Pavlov wrote: > 2016-05-24 23:14 GMT+03:00 Christian Brabandt <[email protected]>: > > Hi Bram! > > > > On Di, 24 Mai 2016, Bram Moolenaar wrote: > > > >> > >> Patch 7.4.1836 > >> Problem: When using a partial on a dictionary it always gets bound to > >> that > >> dictionary. > >> Solution: Make a difference between binding a function to a dictionary > >> explicitly or automatically. > >> Files: src/structs.h, src/eval.c, src/testdir/test_partial.vim, > >> runtime/doc/eval.txt > > > > Starting with this patch, I see sporadic failures on appveyor: > > https://ci.appveyor.com/project/chrisbra/vim/build/1234/job/wlpgqc4vnyvrdgqo > > https://ci.appveyor.com/project/chrisbra/vim/build/1239/job/2p6y8dru0sv216dv#L2073 > > https://ci.appveyor.com/project/chrisbra/vim/build/1238/job/q4e5huq9x6opj6id#L2073 > > > > > > test86: > > ..\gvim -u dos.vim -U NONE --noplugin --not-a-term "+set ff=unix|f > > test.out|wq" dostmp\test86.out > > 522c522 > > < psa3(self={"20": 1}): !result: [['abcArgsPSA3'], {'abcSelfPSA3': > > 'abcSelfPSA3Val'}] > > --- > >> psa3(self={"20": 1}): !result: [['abcArgsPSA3'], {'20': 1} > > Uninitialized memory: set_partial() in if_py_both.h is not setting new > attribute.
Ah, that's much more work than I expected. Thanks! To avoid this flaky behavior I'll clear the partial_T in FunctionCall(). Would be nice to have a test fail when adding another field, but that isn't possible. I only noticed valgrind errors when running the test, but they didn't pinpoint the problem. -- hundred-and-one symptoms of being an internet addict: 1. You actually wore a blue ribbon to protest the Communications Decency Act. /// Bram Moolenaar -- [email protected] -- 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 [email protected]. For more options, visit https://groups.google.com/d/optout.
