GNU bug report logs -
#5600
23.1; etags don't tag (defvar foo) declaration
Previous Next
Reported by: Kevin Ryde <user42 <at> zip.com.au>
Date: Fri, 19 Feb 2010 00:03:01 UTC
Severity: normal
Tags: patch
Done: Chong Yidong <cyd <at> gnu.org>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
Your bug report
#5600: 23.1; etags don't tag (defvar foo) declaration
which was filed against the emacs package, has been closed.
The explanation is attached below, along with your original report.
If you require more details, please reply to 5600 <at> debbugs.gnu.org.
--
5600: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=5600
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
Kevin Ryde <user42 <at> zip.com.au> writes:
> Sounds likely. I didn't know about that option, new diff below.
> I suppose it could tag `declare-function' and the like too, but start
> with untagging the defvar ones.
I've now committed this patch to trunk. Thanks.
[Message part 3 (message/rfc822, inline)]
[Message part 4 (text/plain, inline)]
It'd be good if etags didn't tag defvar declaration forms, only actual
variable definitions. For example it picks up
(defvar message-indent-citation-function)
in longlines.el ahead of that variable's actual definition in
message.el, so M-. of that var goes to longlines.el first.
Perhaps something along the lines below.
I found the tripe-negative of while(!noninname) hard to follow and
thought a skip_name() could be clearer. It might be shared by
Perl_functions, PHP_functions and Makefile_functions.
2010-02-18 Kevin Ryde <user42 <at> zip.com.au>
* etags.c (Lisp_functions): Don't tag "(defvar foo)" declarations.
(skip_name): New helper.
[etags.c.defvar.diff (text/x-diff, attachment)]
[Message part 6 (text/plain, inline)]
In GNU Emacs 23.1.1 (i486-pc-linux-gnu, GTK+ Version 2.16.5)
of 2009-09-14 on raven, modified by Debian
configured using `configure '--build=i486-linux-gnu' '--host=i486-linux-gnu' '--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib' '--localstatedir=/var/lib' '--infodir=/usr/share/info' '--mandir=/usr/share/man' '--with-pop=yes' '--enable-locallisppath=/etc/emacs23:/etc/emacs:/usr/local/share/emacs/23.1/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/23.1/site-lisp:/usr/share/emacs/site-lisp:/usr/share/emacs/23.1/leim' '--with-x=yes' '--with-x-toolkit=gtk' '--with-toolkit-scroll-bars' 'build_alias=i486-linux-gnu' 'host_alias=i486-linux-gnu' 'CFLAGS=-DDEBIAN -g -O2' 'LDFLAGS=-g' 'CPPFLAGS=''
This bug report was last modified 12 years and 224 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.