RE: Keybinding Patch gtags.el

[Stan Moore]

 From: Shigio YAMAGUCHI [mailto:address@hidden
> Sent: Thursday, September 12, 2013 9:21 PM
> To: Stan Moore
> Cc: address@hidden
> Subject: Re: Keybinding Patch gtags.el
> 
> On Thu, 12 Sep 2013 18:55:18 -0400
> "Stan Moore" <address@hidden> wrote:
> 
> > As I mentioned in my first post, I have a hard time calling it an
> > outright bug but the feature doesn't work as expected.
> 
> I cannot approve of calling it an bug.

As noted, we don't disagree here.

> > The original code only reads the variable when gtags.el is  autoloaded
> > so setting the variable in .emacs won't work.
> 
> You set a defcustom variable in a hook after '(gtags-mode t)'.
> It is failure. It should be set statically in .eamcs.

That's the point; setting it statically in .emacs is broken. Try it. When
you attempt to set the variable the gtags.el is loaded by autoload before
setting the variable. Since the variable is only checked during loading any
static user setting is ignored. Using the current gtags.el, it's not
possible for a user to set the variable in .emacs; the patch allows a user
to set the variable in .emacs. Without the patch the variable is of no use
and should be removed.  
 
> > My change reads the variable when the mode is activated in the
> > customary method and allows the user to customize gtags-mode in .emacs.
> 
> I'm sorry but I hesitate to call that it is useful.
> I don't understand a use.

I don't mean this to sound disrespectful but have you tried setting this
variable? In any case I can live with your decision so we can let this drop
if that's what you want.

Stan