org-clock.el (org-clock-in): Don't forward by one character when setting the marker...
authorBastien Guerry <bzg@altern.org>
Wed, 24 Apr 2013 08:47:16 +0000 (10:47 +0200)
committerBastien Guerry <bzg@altern.org>
Wed, 24 Apr 2013 08:47:16 +0000 (10:47 +0200)
* org-clock.el (org-clock-in): Don't forward by one character
when setting the marker in the clock history.

This was done to prevent the marker from being broken when a user
manually insert some text right one it.  I cannot reproduce this
manual marker modification problem, so let's remove this ugly
exception, as it is not consistent with other clock markers.

Thanks to Bernt Hansen for pointing this out.

lisp/org-clock.el

index d96e034..591d0d3 100644 (file)
@@ -1199,13 +1199,8 @@ make this the default behavior.)"
            (goto-char target-pos)
            (org-back-to-heading t)
            (or interrupting (move-marker org-clock-interrupted-task nil))
-           (save-excursion
-             (forward-char) ;; make sure the marker is not at the
-             ;; beginning of the heading, since the
-             ;; user is liking to insert stuff here
-             ;; manually
-             (run-hooks 'org-clock-in-prepare-hook)
-             (org-clock-history-push))
+           (run-hooks 'org-clock-in-prepare-hook)
+           (org-clock-history-push)
            (setq org-clock-current-task (nth 4 (org-heading-components)))
            (cond ((functionp org-clock-in-switch-to-state)
                   (looking-at org-complex-heading-regexp)