end-of-defun odd behaviour in org mode

end-of-defun odd behaviour in org mode

[Leo Liu]

Hi there,

While adding support for org-mode in easy-kill¹ I have discovered C-M-a
and C-M-e work asymmetrically. C-M-e moves forward 2 headlines while
C-M-a move back 1.

Cheers,
Leo

Footnotes: 
¹  https://github.com/leoliu/easy-kill

Re: end-of-defun odd behaviour in org mode

[Bastien]

Hi Leo,

Leo Liu <sdl.web@gmail.com> writes:

> While adding support for org-mode in easy-kill¹ I have discovered C-M-a
> and C-M-e work asymmetrically. C-M-e moves forward 2 headlines while
> C-M-a move back 1.

Commit 76fa97922 changed C-M-a and C-M-e to `org-backward-element'
and `org-forward-element' but I know see it was wrong, so I partially
reverted it, the old behavior makes more sense.

Thanks for reporting this,

http://orgmode.org/cgit.cgi/org-mode.git/commit/?id=76fa97922

-- 
 Bastien

Re: end-of-defun odd behaviour in org mode

[Leo Liu]

On 2014-04-21 19:26 +0200, Bastien wrote:
> Commit 76fa97922 changed C-M-a and C-M-e to `org-backward-element'
> and `org-forward-element' but I know see it was wrong, so I partially
> reverted it, the old behavior makes more sense.

Thanks.

> Thanks for reporting this,
>
> http://orgmode.org/cgit.cgi/org-mode.git/commit/?id=76fa97922

Seems there is another bug after this change:

   C-M-e moves forward 1 and C-M-a moves back 0.

org-back-to-heading does not fully satisfy the protocol of
beginning-of-defun-function.

Leo

Re: end-of-defun odd behaviour in org mode

[Bastien]

Hi Leo,

Leo Liu <sdl.web@gmail.com> writes:

> Seems there is another bug after this change:
>
>    C-M-e moves forward 1 and C-M-a moves back 0.
>
> org-back-to-heading does not fully satisfy the protocol of
> beginning-of-defun-function.

Yeah -- I pushed a fix, relying back again on `org-forward-element'
and `org-backward-element'.

Thanks,

-- 
 Bastien

Re: end-of-defun odd behaviour in org mode

[Leo Liu]

On 2014-04-22 10:51 +0200, Bastien wrote:
> Yeah -- I pushed a fix, relying back again on `org-forward-element'
> and `org-backward-element'.

Thanks. seems to work fine now.

Leo