From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Hyatt Subject: Re: [BUG] Infinite loop in org-agenda-show-new-time Date: Wed, 5 Feb 2020 11:34:35 -0500 Message-ID: References: <87zjswb0vq.fsf@fastmail.fm> <87siyojc2c.fsf@gmail.com> <87fvun6gku.fsf@fastmail.fm> <87ob9bkd4c.fsf@gmail.com> <87txj2lo0j.fsf@fastmail.fm> <87mwopinzv.fsf@gmail.com> <87fvucuwva.fsf@fastmail.fm> <87vc38ff84.fsf@gmail.com> <3E07FCDB-2619-4BFE-BC68-C5181717152B@gmail.com> <87haegy5i9.fsf@gmail.com> <87blqiodo6.fsf@gnu.org> <87o8ufo5l8.fsf@gnu.org> <87zhdx3otz.fsf@gnu.org> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="000000000000072282059dd6bcac" Return-path: Received: from eggs.gnu.org ([2001:470:142:3::10]:40026) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1izNdh-0003A9-HT for emacs-orgmode@gnu.org; Wed, 05 Feb 2020 11:34:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1izNdg-0006Bm-9u for emacs-orgmode@gnu.org; Wed, 05 Feb 2020 11:34:57 -0500 In-Reply-To: <87zhdx3otz.fsf@gnu.org> 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-mx.org@gnu.org Sender: "Emacs-orgmode" To: Bastien Cc: Nick Dokos , "emacs-orgmode@gnu.org" --000000000000072282059dd6bcac Content-Type: text/plain; charset="UTF-8" It is fixed, but now the new time that's supposed to be displayed via text-properties does not show up. Let me spend some time and get a small reproducible case, which will help us test this. On Tue, Feb 4, 2020 at 6:38 PM Bastien wrote: > Hi Andrew, > > thanks again. > > Andrew Hyatt writes: > > > Removing the (beginning-of-line 1) at the end of the time display > > code in that function, and substituting (org-agenda-previous-line) > > seems to fix it. I'm not sure if that's the right approach - the > > previous code didn't use that function for a reason, but I don't know > > what that reason was. > > I think this approach is correct is it will move over visible lines. > > I've pushed a patch, please let me know if it is fixed. > > Best, > > -- > Bastien > --000000000000072282059dd6bcac Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
It is fixed, but now the new time that's supposed to b= e displayed via text-properties does not show up.

Let me= spend some time and get a small reproducible case, which will help us test= this.

On Tue, Feb 4, 2020 at 6:38 PM Bastien <bzg@gnu.org> wrote:
Hi Andrew,

thanks again.

Andrew Hyatt <ahya= tt@gmail.com> writes:

> Removing the (beginning-of-line 1) at the end of the time display
> code in that function, and substituting (org-agenda-previous-line)
> seems to fix it.=C2=A0 I'm not sure if that's the right approa= ch - the
> previous code didn't use that function for a reason, but I don'= ;t know
> what that reason was.

I think this approach is correct is it will move over visible lines.

I've pushed a patch, please let me know if it is fixed.

Best,

--
=C2=A0Bastien
--000000000000072282059dd6bcac--