Re: Patch for gtags.el problems

[Shigio YAMAGUCHI]

Hi,
Thank you for the patch.
I will check later.

> Hi,
> 
> I built a Debian package for global 6.2.4 based on Ron Lees existing package. 
> But as XEmacs user I had some problems with your gtags.el.
> 
> 1) Mouse bindings do not work ([mouse-3] vs 'button3)
> 
> 2) Debugger entered--Lisp error: (wrong-type-argument characterp nil)
>   char-equal(nil ?C)
>   gtags-goto-tag("tag" "" nil)
> According to the documentation of char-equal in XEmacs: Both arguments must 
> be 
> characters
> 
> 3) Byte-compiling covers some old code:
> Emacs23:
> In gtags-get-rootpath:
> gtags.el:447:14:Warning: assignment to free variable `n'
> gtags.el:447:14:Warning: reference to free variable `n'
> 
> In gtags-decode-pathname:
> gtags.el:460:36:Warning: `string-to-int' is an obsolete function (as of Emacs
>     22.1); use `string-to-number' instead.
> 
> In gtags-select-it:
> gtags.el:814:13:Warning: `goto-line' used from Lisp code
> That command is designed for interactive use only
> Wrote /usr/share/emacs23/site-lisp/global/gtags.elc
> 
> XEmacs21:
> While compiling gtags-get-rootpath in file /usr/share/xemacs21/site-
> lisp/global/gtags.el:
>   ** assignment to free variable n
>   ** reference to free variable n
> While compiling gtags-visit-rootdir:
>   ** variable n bound but not referenced
> While compiling gtags-find-tag-from-here:
>   ** variable flag bound but not referenced
> While compiling gtags-pop-stack:
>   ** variable buffer bound but not referenced
> While compiling the end of the data:
>   ** The following functions are not known to be defined: 
>     posn-point, event-start, posn-window, event-end
> 
> Attached is a patch which also replaces the Windows path delimiter (\\) 
> by / which is understood by global.
> 
> -- 
> Regards
> Thomas Viehweger
--
Shigio YAMAGUCHI <address@hidden>
PGP fingerprint: D1CB 0B89 B346 4AB6 5663  C4B6 3CA5 BBB3 57BE DDA3