Alex Efros wrote:
> On Thu, Apr 24, 2014 at 12:43:01PM +0200, Bram Moolenaar wrote:
> > > Me, too. The following used to work:
> > >
> > > fun! Foo()
> > > endfun
> > > let g:Foo = function('Foo')
> > >
> > > After 7.4.264, I get:
> > > " E705: Variable name conflicts with existing function: g:Foo
> >
> > That is correct. Previously Foo() and g:Foo were different names, which
> > is wrong. Both refer to the same function.
>
> So, what is the right way to implement this, compatible with pre-patch-260
> and current one? I wanna use global variable to define user-configurable
> handler function, and I've in my plugin code exactly as shown above.
> My plugin uses exists() and type() for testing value of that global
> variable, and they return different results for g:Foo in cases
> fun g:Foo()
> endfun
> and
> fun Foo()
> endfun
> let g:Foo = function('Foo')
> That's why such let operation was added.
>
> My guess right solution should be
> fun s:Foo()
> endfun
> let g:Foo = function('s:Foo')
> but I'm not 100% sure.
That should be the right way. The more things are script-local the
better. But check that it actually works.
> Also in my plugin I allow users to use symbolic funcref for their custom
> handlers:
> fun MyHandler()
> endfun
> let g:Handler = 'MyHandler'
> by using type() to detect is variable contains text and then converting it
> to real funcref using exists('*MyHandler') and then function('MyHandler) -
> is this logic still correct after patch 265? Real code is here:
> https://code.google.com/p/vim-plugin-viewdoc/source/browse/plugin/viewdoc.vim#181
>
>
> P.S. One more difference between fun g:Foo and let g:Foo=function() is
> while auto-complete 'Foo<Tab>' in command line - first case shown as
> 'Foo(', second as 'Foo' - maybe it makes sense to detect variables with
> funcref values and show them with '(' in autocomplete too?
--
You can't have everything. Where would you put it?
-- Steven Wright
/// 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.
