Re: [Orgmode] org-show-following-heading shows even when nil

[Samuel Wales]

i.e. this:

diff --git a/lisp/org-agenda.el b/lisp/org-agenda.el
index 93334b6..4bbaefc 100644
--- a/lisp/org-agenda.el
+++ b/lisp/org-agenda.el
@@ -2764,6 +2764,7 @@ bind it in the options section.")
         (org-set-local 'org-agenda-name name)))
   (setq buffer-read-only nil))

+(eval-when-compile (defvar org-agenda-overriding-columns-format))
 (defun org-finalize-agenda ()
   "Finishing touch for the agenda buffer, called just before displaying it."
   (unless org-agenda-multi
@@ -6129,10 +6130,7 @@ and by additional input from the age of a
schedules or deadline entry."
     (push-mark)
     (goto-char pos)
     (when (org-mode-p)
-      (org-show-context 'agenda)
-      (save-excursion
-       (and (outline-next-heading)
-            (org-flag-heading nil)))) ; show the next heading
+      (org-show-context 'agenda))
     (recenter (/ (window-height) 2))
     (run-hooks 'org-agenda-after-show-hook)
     (and highlight (org-highlight (point-at-bol) (point-at-eol)))))
@@ -6315,10 +6313,7 @@ at the text of the entry itself."
       (widen)
       (goto-char pos)
       (when (org-mode-p)
-       (org-show-context 'agenda)
-       (save-excursion
-         (and (outline-next-heading)
-              (org-flag-heading nil)))))))  ; show the next heading
+       (org-show-context 'agenda)))))

 (defun org-agenda-goto-mouse (ev)
   "Go to the Org-mode file which contains the item at the mouse click."



On 2010-05-22, Samuel Wales <address@hidden> wrote:
> Hi Carsten,
>
> Could it be the following in org-agenda-switch-to?
>
>       (save-excursion
>         (and (outline-next-heading)
>              (org-flag-heading nil)))))))  ; show the next heading
>
> I wonder if it will cause any problems to remove that whole sexp.
>
> Thanks.
>
> Samuel
>
> On 2010-05-19, Carsten Dominik <address@hidden> wrote:
>> Hi Samuel,
>>
>> thanks for your feedback - I do not know what else could be wrong
>> here.....
>>
>> - Carsten
>>
>> On May 18, 2010, at 6:17 AM, Samuel Wales wrote:
>>
>>> Hi Carsten,
>>>
>>> On 2010-05-14, Carsten Dominik <address@hidden> wrote:
>>>> for some reason I do not remember, I enforce showing the next
>>>> headline
>>>> in this special case.  Please try the following patch and run with if
>>>> for a while, to see if it causes any problems.
>>>
>>> The patch did not apply for some reason, but I manually applied it,
>>> and it seemed in trivial testing to work perfectly.  Strangely,
>>> however, today the behavior is as if I did not apply the patch.  I did
>>> eval the function, so it must be some other piece of code that is
>>> involved.  I don't get it.  (Not physically able to investigate now,
>>> unfortunately.)
>>>
>>> Samuel
>>>
>>> --
>>> Q: How many CDC "scientists" does it take to change a lightbulb?
>>> A: "You only think it's dark." [CDC has denied a deadly disease for
>>> 25 years]
>>> ==========
>>> Retrovirus: http://www.wpinstitute.org/xmrv/index.html
>>
>> - Carsten
>>
>>
>>
>>
>
>
> --
> Q: How many CDC "scientists" does it take to change a lightbulb?
> A: "You only think it's dark." [CDC has denied a deadly disease for 25
> years]
> ==========
> Retrovirus: http://www.wpinstitute.org/xmrv/index.html
>


-- 
Q: How many CDC "scientists" does it take to change a lightbulb?
A: "You only think it's dark." [CDC has denied a deadly disease for 25 years]
==========
Retrovirus: http://www.wpinstitute.org/xmrv/index.html