GNU bug report logs - #5666
23.1; please define push-tag-mark in etags.el

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 5666 in the body.
You can then email your comments to 5666 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Richard Kim <Richard.Kim1 <at> synopsys.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 23.1; please define push-tag-mark in etags.el
Date: Mon, 01 Mar 2010 11:27:05 -0800

GNU/Emacs defines pop-tag-mark command to go along with find-tag
command for traversing back in cursor movement history.  However
GNU/Emacs does not define matching push-tag-mark command.  I would
like to request that something like the following be added to etags.el

  (defun push-tag-mark ()
    "Push the current position to the ring of markers so that
    \\[pop-tag-mark] can be used to come back to current position."
    (interactive)
    (ring-insert find-tag-marker-ring (point-marker)))

I make this request for the following reasons:

- This is a useful command by itself which I use often

- This would be useful for use by tools other than etags, e.g.,
  forthcoming cedet tool or even for existing tools, e.g.,

    (defadvice imenu
      (before use-etags-stack activate compile)
      "Push point onto stack used by etags so that \\[pop-tag-mark] can be used."
      (push-tag-mark))

    (defadvice cscope-call
      (before use-etags-stack activate compile)
      "Push point onto stack used by etags so that \\[pop-tag-mark] can be used."
      (push-tag-mark))

- Since xemacs defines push-tag-mark already, it makes it easier for
  tool developers, so that we don't have to add code such as the
  following in our own packages:

  (unless (fboundp 'push-tag-mark)
    (defun push-tag-mark ()
      (interactive)
      (ring-insert find-tag-marker-ring (point-marker))))

Thanks.

Message #12 received at 5666 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Richard Kim <Richard.Kim1 <at> synopsys.com>
Cc: 5666 <at> debbugs.gnu.org
Subject: Re: bug#5666: 23.1; please define push-tag-mark in etags.el
Date: Tue, 01 Oct 2019 17:41:52 +0200

Richard Kim <Richard.Kim1 <at> synopsys.com>, Richard.Kim1 <at> synopsys.com
writes:

> GNU/Emacs defines pop-tag-mark command to go along with find-tag
> command for traversing back in cursor movement history.  However
> GNU/Emacs does not define matching push-tag-mark command.  I would
> like to request that something like the following be added to etags.el
>
>   (defun push-tag-mark ()
>     "Push the current position to the ring of markers so that
>     \\[pop-tag-mark] can be used to come back to current position."
>     (interactive)
>     (ring-insert find-tag-marker-ring (point-marker)))

(I'm going through old bug reports that unfortunately didn't get any
response at the time.)

Emacs now has the xref framework for this stuff, and it has
`xref-push-marker-stack', so I think this issue should now be solved,
and I'm closing this bug report.  If this is still something that should
be done, please reopen.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.