From mboxrd@z Thu Jan 1 00:00:00 1970 From: Carsten Dominik Subject: [Accepted] [Orgmode] Removed unecessary invocations of org-agenda-show. Date: Mon, 2 May 2011 10:17:01 +0200 (CEST) Message-ID: <20110502081701.6CCF8436F83@u016822.science.uva.nl> References: <87zkvly3m3.fsf@archdesk.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from eggs.gnu.org ([140.186.70.92]:55179) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QGoJU-0000f6-Mf for emacs-orgmode@gnu.org; Mon, 02 May 2011 04:17:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QGoJT-0006jX-3B for emacs-orgmode@gnu.org; Mon, 02 May 2011 04:17:04 -0400 Received: from u016822.science.uva.nl ([146.50.39.34]:50275) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QGoJS-0006j7-S6 for emacs-orgmode@gnu.org; Mon, 02 May 2011 04:17:03 -0400 List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: emacs-orgmode@gnu.org Patch 271 (http://patchwork.newartisans.com/patch/271/) is now "Accepted". Maintainer comment: none This relates to the following submission: http://mid.gmane.org/%3C87zkvly3m3.fsf%40archdesk.localdomain%3E Here is the original message containing the patch: > Content-Type: text/plain; charset="utf-8" > MIME-Version: 1.0 > Content-Transfer-Encoding: 7bit > Subject: [Orgmode] Removed unecessary invocations of org-agenda-show. > Date: Mon, 13 Sep 2010 20:48:27 -0000 > From: Matt Lundin > X-Patchwork-Id: 271 > Message-Id: <87zkvly3m3.fsf@archdesk.localdomain> > To: Org Mode > > lisp/org-agenda.el (org-agenda-set-tags): Remove org-agenda-show to prevent > disrupting windows and changing point in original buffer. > (org-agenda-set-property): Same > (org-agenda-set-effort): Same > (org-agenda-toggle-archive-tag): Same > > When setting a tag in the agenda, org-mode displays the corresponding > entry in the original org buffer by calling org-agenda-show. This has > the unwelcome side-effect of disrupting the current window arrangement > and changing the position of the point in the original buffer. This > behavior is inconsistent with the that of org-agenda-todo, which makes > all its changes "silently." > > Here is the offending line (6799) in org-agenda-set-tags: > > --8<---------------cut here---------------start------------->8--- > (org-agenda-show) ;;; FIXME This is a stupid hack and should not be needed > --8<---------------cut here---------------end--------------->8--- > > The same line occurs in org-agenda-set-property, org-agenda-set-effort, > and org-agenda-toggle-archive tag. > > --- > lisp/org-agenda.el | 4 ---- > 1 files changed, 0 insertions(+), 4 deletions(-) > > diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el > index 32c65db..784ba6a 100644 > --- a/lisp/org-agenda.el > +++ b/lisp/org-agenda.el > @@ -6796,7 +6796,6 @@ the same tree node, and the headline of the tree node in the Org-mode file." > (org-agenda-check-no-diary) > (if (and (org-region-active-p) (interactive-p)) > (call-interactively 'org-change-tag-in-region) > - (org-agenda-show) ;;; FIXME This is a stupid hack and should not be needed > (let* ((hdmarker (or (org-get-at-bol 'org-hd-marker) > (org-agenda-error))) > (buffer (marker-buffer hdmarker)) > @@ -6825,7 +6824,6 @@ the same tree node, and the headline of the tree node in the Org-mode file." > "Set a property for the current headline." > (interactive) > (org-agenda-check-no-diary) > - (org-agenda-show) ;;; FIXME This is a stupid hack and should not be needed > (let* ((hdmarker (or (org-get-at-bol 'org-hd-marker) > (org-agenda-error))) > (buffer (marker-buffer hdmarker)) > @@ -6848,7 +6846,6 @@ the same tree node, and the headline of the tree node in the Org-mode file." > "Set the effort property for the current headline." > (interactive) > (org-agenda-check-no-diary) > - (org-agenda-show) ;;; FIXME This is a stupid hack and should not be needed > (let* ((hdmarker (or (org-get-at-bol 'org-hd-marker) > (org-agenda-error))) > (buffer (marker-buffer hdmarker)) > @@ -6872,7 +6869,6 @@ the same tree node, and the headline of the tree node in the Org-mode file." > "Toggle the archive tag for the current entry." > (interactive) > (org-agenda-check-no-diary) > - (org-agenda-show) ;;; FIXME This is a stupid hack and should not be needed > (let* ((hdmarker (or (org-get-at-bol 'org-hd-marker) > (org-agenda-error))) > (buffer (marker-buffer hdmarker)) >