Re: [PATCH] Revert "Fix :VISIBILITY: handling of nested "folded" properties"

From: Carsten Dominik <carsten.dominik@gmail.com>
To: Matt Lundin <mdl@imapmail.org>
Cc: Org Mode <emacs-orgmode@gnu.org>
Subject: Re: [PATCH] Revert "Fix :VISIBILITY: handling of nested "folded" properties"
Date: Mon, 6 Dec 2010 10:07:23 +0100	[thread overview]
Message-ID: <614CA58B-9D41-45EB-BB27-0C8EA77991EF@gmail.com> (raw)
In-Reply-To: <871v5v1pd8.fsf@fastmail.fm>

Applied, thanks.

- Carsten

On Dec 6, 2010, at 5:45 AM, Matt Lundin wrote:

> This reverts commit 383802d063a9f2dd959d5574b226fa8ec7f8be41.
>
> The commit had org-mode process the headlines from bottom to top,  
> which
> meant that any changes to the visibility of lower headlines were
> overridden/modified by changes higher up the tree. Reverting the  
> commit
> causes VISIBILITY to work correctly.
> ---
> lisp/org.el |    4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lisp/org.el b/lisp/org.el
> index 66514a2..4b39c9c 100644
> --- a/lisp/org.el
> +++ b/lisp/org.el
> @@ -6073,8 +6073,8 @@ With a numeric prefix, show all headlines up  
> to that level."
>   (interactive)
>   (let (org-show-entry-below state)
>     (save-excursion
> -      (goto-char (point-max))
> -      (while (re-search-backward
> +      (goto-char (point-min))
> +      (while (re-search-forward
> 	      "^[ \t]*:VISIBILITY:[ \t]+\\([a-z]+\\)"
> 	      nil t)
> 	(setq state (match-string 1))
> -- 
> 1.7.3.2
>
>
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

- Carsten