[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] Custom timestamps in date ranges
From: |
Nicolas Goaziou |
Subject: |
Re: [O] Custom timestamps in date ranges |
Date: |
Wed, 05 Jul 2017 23:19:10 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Hello,
jeeger <address@hidden> writes:
> I have a single-line patch to contribute. In my org-mode version (Org
> mode version 9.0.9 (9.0.9-30-g796a78-elpa)), custom timestamps do not
> correctly apply to date ranges. Only the second date is replaced in
> the custom date format. I've tracked this down to org-activate-dates.
> The line
>
> (if (match-end 3)
> (org-display-custom-time (match-beginning 3) (match-end 3))
> (org-display-custom-time (match-beginning 1) (match-end 1)))
>
> leads to only the second of the date range being fontified. The
> corrected version is:
>
> (when (match-end 3)
> (org-display-custom-time (match-beginning 3) (match-end 3)))
> (org-display-custom-time (match-beginning 1) (match-end 1))
>
> This fontifies the first date correctly as well.
Thank you.
Would it makes sense to add a comment above the `when' explaining this
is about date ranges?
Also, could you use git format-patch and add an appropriate commit
message so I can apply your changes?
Regards,
--
Nicolas Goaziou