From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leonard Randall Subject: Re: (org-insert-headline '(4)) should insert new headline before point Date: Sun, 4 May 2014 03:37:54 +0100 Message-ID: References: <87a9bk9guf.fsf@bzg.ath.cx> <87zjjdafa3.fsf@bzg.ath.cx> <87eh0lg940.fsf@bzg.ath.cx> <8738gwwbxb.fsf@bzg.ath.cx> <87siowqks8.fsf@bzg.ath.cx> <87fvkvfbqb.fsf@bzg.ath.cx> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary=047d7b472800e5e22404f889e6f2 Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:54863) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WgmJS-0005II-04 for emacs-orgmode@gnu.org; Sat, 03 May 2014 22:37:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WgmJQ-0006n2-Vo for emacs-orgmode@gnu.org; Sat, 03 May 2014 22:37:57 -0400 In-Reply-To: List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-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 Cc: York Zhao , emacs-orgmode --047d7b472800e5e22404f889e6f2 Content-Type: text/plain; charset=UTF-8 Hi Bastien, Just to follow up on the error, reverting commit 0ca4092 fixes the bug that I mentioned in my last email and reverting the fourth hunk of commit b8c85a1 should fix the bug that you tried to fix with that commit. Carsten had added an 'unless (bolp)' condition to the function that controls inserting new lines outside of headlines to avoid deleting whitespace. However, so far as I can tell, the other measures Carsten put in place with that commit made adding this condition unnecessary. Outside of the contexts where it produces the first bug I mentioned it does not seem to affect headline insertion behaviour. Anyway, reverting those commits should get headline insertion to work as advertised. All best Leonard --047d7b472800e5e22404f889e6f2 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
Hi Bastien,

Just to follow up on th= e error, reverting commit 0ca4092 fixes the bug that I mentioned in my last= email and reverting the fourth hunk of commit b8c85a1 should fix the bug t= hat you tried to fix with that commit.=C2=A0 Carsten had added an 'unle= ss (bolp)' condition to the function that=20 controls inserting new lines outside of headlines to avoid deleting=20 whitespace. However, so far as I can tell, the other measures Carsten=20 put in place with that commit made adding this condition unnecessary. Outsi= de of the=20 contexts where it produces the first bug I mentioned it does not seem to affect headline insertion behaviour.

Anyway, reverting those commits should get headline insertion to work a= s advertised.

All best

Leonard


--047d7b472800e5e22404f889e6f2--