emacs-orgmode
[Top][All Lists]
Advanced

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

Re: [O] [PATCH] Make the point visible when jumping to the mark


From: Sebastien Vauban
Subject: Re: [O] [PATCH] Make the point visible when jumping to the mark
Date: Thu, 22 May 2014 10:56:30 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.91 (windows-nt)

Bastien wrote:
> Hi Ian,
>
> Please send me your public key so that I can give you access to the
> org-mode and worg repositories.
>
> Ian Kelling <address@hidden> writes:
>
>> From 9191e4a364e251119cf8b7c72e41f6c0d09583f2 Mon Sep 17 00:00:00 2001
>> Message-ID: <address@hidden>
>> MIME-Version: 1.0
>> Content-Type: text/plain
>>
>> *lisp/org.el: Advise commands which jump to the mark
>
> The formatting of the Changelog message not good.
>
> It should be
>
> * org.el: Advise commands which jump to the mark.
>
> See http://orgmode.org/worg/org-contribute.html and example in
> the git logs.
>
>> +(eval-after-load "simple"
>> +  '(defadvice set-mark-command (after org-make-visible activate)
>> +     "Make the point visible with `org-show-context'."
>> +     (org-mark-jump-unhide)))
>>
>> +(eval-after-load "simple"
>> +  '(defadvice exchange-point-and-mark (after org-make-visible activate)
>> +     "Make the point visible with `org-show-context'."
>> +     (org-mark-jump-unhide)))
>
>> +(eval-after-load "simple"
>> +  '(defadvice pop-global-mark (after org-make-visible activate)
>> +     "Make the point visible with `org-show-context'."
>> +     (org-mark-jump-unhide)))
>
> This two ones are good, yes.

Even if the goal is desirable, I thought that we may not add defadvice
in Emacs sources.

(... a couple of minutes later ...)

Well, Googling returns this :

  ╭──── 
http://www.gnu.org/software/emacs/manual/html_node/elisp/Advising-Functions.html
  │
  │ Emacs's own source files should not put advice on functions in
  │ Emacs. There are currently a few exceptions to this convention, but we
  │ aim to correct them.
  ╰────

So, it seems this should be avoided as long as Org does go into Emacs.

Best regards,
  Seb

-- 
Sebastien Vauban




reply via email to

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