From: Shigio YAMAGUCHI [mailto:[email protected]] > Sent: Thursday, September 12, 2013 9:21 PM > To: Stan Moore > Cc: [email protected] > Subject: Re: Keybinding Patch gtags.el > > On Thu, 12 Sep 2013 18:55:18 -0400 > "Stan Moore" <[email protected]> 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 _______________________________________________ Bug-global mailing list [email protected] https://lists.gnu.org/mailman/listinfo/bug-global
