bug-global
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Patch for gtags.el problems


From: Thomas Viehweger
Subject: Patch for gtags.el problems
Date: Thu, 2 Aug 2012 22:29:30 +0200
User-agent: KMail/1.13.7 (Linux/3.4.7; KDE/4.8.4; x86_64; ; )

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

Attachment: gtags.el.diff
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]