Hi,
2016/2/1 Mon 18:13:39 UTC+9 mattn wrote:
> On Monday, February 1, 2016 at 2:45:55 PM UTC+9, Gary Johnson wrote:
> > On 2016-01-31, mattn wrote:
> > > Hi list and Dr.Chip.
> > >
> > > This is a small fixing guard for loading script. You may want to
> > > add logipat_version = 'v3'. Please take care of this and update
> > > bunded one in vim repository.
> >
> > What is the problem with setting g:loaded_LogiPat to "v3"? It
> > doesn't matter what the value is since the test of that variable is
> > simply for its existence. Also, if you look at Dr. Chip's other
> > plugins, you'll see that they all follow the convention of setting
> > the loaded_<plugin> variable to the version.
> >
> > Regards,
> > Gary
>
> Try below
>
> :let g:loaded_LogiPat = "v3"
> :echo exists("loaded_logipat")
>
> This is guard to avoid duplicate-loading. But always return 0.
The problem is that the cases of the variables are different. So there's
no need to change the value "v3" to 1.
It seems that Dr. Chip's guards are based on the filenames. So using
"loaded_logiPat" might be consistent.
--- a/runtime/plugin/logiPat.vim
+++ b/runtime/plugin/logiPat.vim
@@ -36,10 +36,10 @@
" ---------------------------------------------------------------------
" Load Once: {{{1
-if &cp || exists("loaded_logipat")
+if &cp || exists("loaded_logiPat")
finish
endif
-let g:loaded_LogiPat = "v3"
+let g:loaded_logiPat = "v3"
let s:keepcpo = &cpo
set cpo&vim
"DechoRemOn
Regards,
Ken Takata
--
--
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.
