On Wednesday, May 28, 2014 5:23:53 PM UTC+2, Christian Wellenbrock wrote:
> On Wednesday, May 28, 2014 4:47:35 PM UTC+2, Bram Moolenaar wrote:
> > I wrote:
> >
> > > Patch 7.4.310
> > > Problem: getpos()/setpos() don't include curswant.
> > > Solution: Add a fifth number when getting/setting the cursor.
> > > Files: src/eval.c, src/testdir/test_eval.in,
> > > src/testdir/test_eval.ok,
> > > runtime/doc/eval.txt
> >
> > Please let me know if you spot an incompatibility because of this patch.
> > Hopefully the extra list item doesn't cause trouble anywhere, but you
> > never know.
>
> I was using list assignment like this in an earlier version of a Vim plugin
> [1]:
>
> let [a, b, c, d] = getpos('.')
>
> I'm using something like this now, which should work with the new version:
>
> let [b, c] = getpos('.')[1:2]
>
> This patch would have broken my earlier version, so I wouldn't be surprised
> if other plugins are affected.
>
> E687: Less targets than List items
>
> [1]
> https://github.com/wellle/targets.vim/blob/ae25356de27adffaa07ccac3899d671aa45ed61c/autoload/targets.vim#L299
> [2]
> https://github.com/wellle/targets.vim/blob/56a4d5842f7f1ed72d57d8bc37c13428bc3a4751/autoload/targets.vim#L399
Same here, the change in getpos() breaks list unpacking in my comment
text object plugin.
--
David
--
--
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.
