From mboxrd@z Thu Jan  1 00:00:00 1970
From: Oleh <ohwoeowho@gmail.com>
Subject: Re: [PATCH] org.el (org-delete-property): Remove bogus properties
Date: Thu, 13 Mar 2014 21:57:50 +0100
Message-ID: <CAA01p3qEM=s3RNTeMr0DaN758W7sYgb0v2xPYqa9LrF0BWbzmw@mail.gmail.com>
References: <CAA01p3rMKvbnmnF6hs-7pZ0JBeKdpb+H8TB7n_gh7V7eN5mDZg@mail.gmail.com>
	<878usfcm5q.fsf@bzg.ath.cx>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Return-path: <emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:52466)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <ohwoeowho@gmail.com>) id 1WOChN-0005WE-Ei
	for emacs-orgmode@gnu.org; Thu, 13 Mar 2014 16:57:54 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <ohwoeowho@gmail.com>) id 1WOChM-0004d9-KP
	for emacs-orgmode@gnu.org; Thu, 13 Mar 2014 16:57:53 -0400
In-Reply-To: <878usfcm5q.fsf@bzg.ath.cx>
List-Id: "General discussions about Org-mode." <emacs-orgmode.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/emacs-orgmode>
List-Post: <mailto:emacs-orgmode@gnu.org>
List-Help: <mailto:emacs-orgmode-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=subscribe>
Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org
Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org
To: Bastien <bzg@gnu.org>
Cc: org mode <emacs-orgmode@gnu.org>

Hi Bastien,

I've pushed a bugfix related to the patch.
`org-entry-delete' was being called with 3 args, although it takes 2.
I hope I understood it correctly. Could you please check?

regards,
Oleh

On Wed, Mar 12, 2014 at 7:22 PM, Bastien <bzg@gnu.org> wrote:
> Hi Oleh,
>
> Oleh <ohwoeowho@gmail.com> writes:
>
>> Can someone have a look at this patch and check it in?
>
> I applied a different patch, things are a bit more complicate:
> (org-get-category) always returns a non-empty string (at least
> after `org-refresh-category-properties' as been called once),
> while (org-entry-get (point) "CATEGORY") is the real check we
> need to perform.
>
> Thanks for raising this!
>
> --
>  Bastien
>