Bug: Org leaves whitespace around :LOGBOOK: [8.2.10 (release_8.2.10 @ /home/jsynacek/.local/share/emacs/24.4/lisp/org/)]

Bug: Org leaves whitespace around :LOGBOOK: [8.2.10 (release_8.2.10 @ /home/jsynacek/.local/share/emacs/24.4/lisp/org/)]

[Jan Synacek]

Hello,

I'm not sure if this is by design or not, but it sure looks like a bug
to me and is quite annoying.

Reproducer:
1) emacs -Q
2) eval the following:
(require 'org)
(setq org-log-done 'time)
(setq org-log-into-drawer t)
(setq org-todo-keywords
      '((sequence "TODO(t)" "WAITING(w@/!)" "|" "DONE(d!)")))
3) create a temporary org-mode buffer, insert an item in it, and C-c C-t
   on the item (set the item to the WAITING state)
4) notice the whitespace after :LOGBOOK: and :END:, creating sort of a box

Emacs  : GNU Emacs 24.4.2 (x86_64-unknown-linux-gnu, GTK+ Version 3.10.9)
 of 2014-10-22 on jsynacek-ntb-work.brq.redhat.com
Package: Org-mode version 8.2.10 (release_8.2.10 @ /home/jsynacek/.local/share/emacs/24.4/lisp/org/)

current state:
==============
(setq
 org-tab-first-hook '(org-hide-block-toggle-maybe org-src-native-tab-command-maybe
		      org-babel-hide-result-toggle-maybe org-babel-header-arg-expand)
 org-speed-command-hook '(org-speed-command-default-hook org-babel-speed-command-hook)
 org-occur-hook '(org-first-headline-recenter)
 org-metaup-hook '(org-babel-load-in-session-maybe)
 org-log-done 'time
 org-confirm-shell-link-function 'yes-or-no-p
 org-after-todo-state-change-hook '(org-clock-out-if-current)
 org-src-mode-hook '(org-src-babel-configure-edit-buffer org-src-mode-configure-edit-buffer)
 org-agenda-before-write-hook '(org-agenda-add-entry-text)
 org-babel-pre-tangle-hook '(save-buffer)
 org-mode-hook '(#[nil "\300\301\302\303\304$\207"
		   [org-add-hook change-major-mode-hook org-show-block-all append local] 5]
		 #[nil "\300\301\302\303\304$\207"
		   [org-add-hook change-major-mode-hook org-babel-show-result-all append local] 5]
		 org-babel-result-hide-spec org-babel-hide-all-hashes)
 org-ctrl-c-ctrl-c-hook '(org-babel-hash-at-point org-babel-execute-safely-maybe)
 org-cycle-hook '(org-cycle-hide-archived-subtrees org-cycle-hide-drawers
		  org-cycle-hide-inline-tasks org-cycle-show-empty-lines
		  org-optimize-window-after-visibility-change)
 org-todo-keywords '((sequence "TODO(t)" "WAITING(w@/!)" "|" "DONE(d!)"))
 org-confirm-elisp-link-function 'yes-or-no-p
 org-metadown-hook '(org-babel-pop-to-session-maybe)
 org-log-into-drawer t
 org-clock-out-hook '(org-clock-remove-empty-clock-drawer)
 )

Cheers,
-- 
Jan Synacek
Software Engineer, Red Hat

Re: Bug: Org leaves whitespace around :LOGBOOK: [8.2.10 (release_8.2.10 @ /home/jsynacek/.local/share/emacs/24.4/lisp/org/)]

[Nicolas Goaziou]

Hello,

Jan Synacek <jsynacek@redhat.com> writes:

> I'm not sure if this is by design or not, but it sure looks like a bug
> to me and is quite annoying.
>
> Reproducer:
> 1) emacs -Q
> 2) eval the following:
> (require 'org)
> (setq org-log-done 'time)
> (setq org-log-into-drawer t)
> (setq org-todo-keywords
>       '((sequence "TODO(t)" "WAITING(w@/!)" "|" "DONE(d!)")))
> 3) create a temporary org-mode buffer, insert an item in it, and C-c C-t
>    on the item (set the item to the WAITING state)
> 4) notice the whitespace after :LOGBOOK: and :END:, creating sort of a box
>
> Emacs  : GNU Emacs 24.4.2 (x86_64-unknown-linux-gnu, GTK+ Version 3.10.9)
>  of 2014-10-22 on jsynacek-ntb-work.brq.redhat.com
> Package: Org-mode version 8.2.10 (release_8.2.10 @ /home/jsynacek/.local/share/emacs/24.4/lisp/org/)

FWIW, I cannot reproduce it on development version (8.3_beta).


Regards,

-- 
Nicolas Goaziou

Re: Bug: Org leaves whitespace around :LOGBOOK: [8.2.10 (release_8.2.10 @ /home/jsynacek/.local/share/emacs/24.4/lisp/org/)]

[Jan Synacek]

[-- Attachment #1: Type: text/plain, Size: 1795 bytes --]

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:
> Hello,
>
> Jan Synacek <jsynacek@redhat.com> writes:
>
>> I'm not sure if this is by design or not, but it sure looks like a bug
>> to me and is quite annoying.
>>
>> Reproducer:
>> 1) emacs -Q
>> 2) eval the following:
>> (require 'org)
>> (setq org-log-done 'time)
>> (setq org-log-into-drawer t)
>> (setq org-todo-keywords
>>       '((sequence "TODO(t)" "WAITING(w@/!)" "|" "DONE(d!)")))
>> 3) create a temporary org-mode buffer, insert an item in it, and C-c C-t
>>    on the item (set the item to the WAITING state)
>> 4) notice the whitespace after :LOGBOOK: and :END:, creating sort of a box
>>
>> Emacs  : GNU Emacs 24.4.2 (x86_64-unknown-linux-gnu, GTK+ Version 3.10.9)
>>  of 2014-10-22 on jsynacek-ntb-work.brq.redhat.com
>> Package: Org-mode version 8.2.10 (release_8.2.10 @ /home/jsynacek/.local/share/emacs/24.4/lisp/org/)
>
> FWIW, I cannot reproduce it on development version (8.3_beta).

Git bisect log:

bad: [3f484a52775c8873a3b1b44e8264e01369850a8a] Fix headline insertion after an empty headline
good: [e112f3c0df49264998d4ae8df545c9940692f68f] ox-texinfo: Fix corner case when fixing a node name
good: [409913b253de2de1b49f9468a4b77b9292e090c4] Fix `org-promote' error
bad: [dc16617f46d64073e4d06308f7150c46d992a408] orgguide.texi: Small fixes
good: [2e5b3dede103bba0071144ec7b7fc250471c1463] org-element: Interpret headlines according to `org-o;dd-levels-only'
bad: [2de0c1c3da1e7f9859b82d8af0167c60545c5c5f] Fix 37bf05
bad: [37bf0576f2f2894c6e37239ee8db63a3ef21a840] org-element: Make properties parsing more robust
first bad commit: [37bf0576f2f2894c6e37239ee8db63a3ef21a840] org-element: Make properties parsing mor;e robust

Cheers,
-- 
Jan Synacek
Software Engineer, Red Hat

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 818 bytes --]

Re: Bug: Org leaves whitespace around :LOGBOOK: [8.2.10 (release_8.2.10 @ /home/jsynacek/.local/share/emacs/24.4/lisp/org/)]

[Nicolas Goaziou]

Jan Synacek <jsynacek@redhat.com> writes:

> Git bisect log:
>
> bad: [3f484a52775c8873a3b1b44e8264e01369850a8a] Fix headline insertion after an empty headline
> good: [e112f3c0df49264998d4ae8df545c9940692f68f] ox-texinfo: Fix corner case when fixing a node name
> good: [409913b253de2de1b49f9468a4b77b9292e090c4] Fix `org-promote' error
> bad: [dc16617f46d64073e4d06308f7150c46d992a408] orgguide.texi: Small fixes
> good: [2e5b3dede103bba0071144ec7b7fc250471c1463] org-element: Interpret headlines according to `org-o;dd-levels-only'
> bad: [2de0c1c3da1e7f9859b82d8af0167c60545c5c5f] Fix 37bf05
> bad: [37bf0576f2f2894c6e37239ee8db63a3ef21a840] org-element: Make properties parsing more robust
> first bad commit: [37bf0576f2f2894c6e37239ee8db63a3ef21a840] org-element: Make properties parsing mor;e robust

I think it's good enough if it is fixed in Org 8.3.

Regards,