[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] org-agenda.el: customise outline path in echo area
From: |
Mikhail Skorzhinskii |
Subject: |
Re: [PATCH] org-agenda.el: customise outline path in echo area |
Date: |
Tue, 28 Dec 2021 13:01:48 +0100 |
User-agent: |
Evolution 3.40.4-1 |
Hi Ihor,
Thank you for your comments once again. I've changed string= to eq and
it appears to be working OK.
I've also renamed "title" variable to "title-prop" for better
readability. The -prop suffix should show the reader that it was
extracted from the file, and thus cost some CPU/IO.
I've attached an updated version to this email. Would love to hear what
you think.
Thanks,
Mikhail
On Sun, 2021-12-26 at 21:44 +0800, Ihor Radchenko wrote:
> Mikhail Skorzhinskii <mskorzhinskiy@eml.cc> writes:
>
> Thanks for the patch! The addition looks reasonable to me.
>
> > + (title (when (and file-or-title (string= file-or-title
> > 'title))
> > ...
> > + (and file-or-title bfn (concat (if (and (string= file-
> > or-
> > title 'title) title)
>
> (string= file-or-title 'title) will match FILE-OR-TITLE values
> "title"
> and 'title. I am not sure if it is what you intended to achieve.
> Probably, a simple (eq file-or-title 'title) would be sufficient.
>
> Best,
> Ihor
0001-org-agenda.el-customize-outline-path-in-echo-area.patch
Description: Text Data