|
| From: | Rüdiger Sonderfeld |
| Subject: | Re: [O] [PATCH 2/3] org-datetree.el: Add support for ISO week trees. |
| Date: | Thu, 03 Sep 2015 01:14:43 +0100 |
| User-agent: | KMail/4.13.3 (Linux/3.13.0-57-generic; KDE/4.13.3; x86_64; ; ) |
On Wednesday 02 September 2015 21:58:17 Nicolas Goaziou wrote:
> Rüdiger Sonderfeld <address@hidden> writes:
> > + (let ((prop (org-find-property "DATE_WEEK_TREE")))
>
> I don't think we need to introduce a new property for that. DATE_TREE is
> enough.
Since DATE_TREE and DATE_WEEK_TREE (or WEEK_TREE instead?) are structured
differently it might make sense to keep the property separated.
> > + ;; ISO 8601 week format is %G-W%V(-%u)
> > + (org-datetree--find-create "^\\*+[
> > \t]+\\([12][0-9]\\{3\\}\\)\\(\\s-*?\\([
> > \t]:[[:alnum:]:address@hidden:\\)?\\s-*$\\)"
> Isn't this line too long?
What's the limit? Because if it's 80 char then I'd need to do some `concat'
ugliness because the regex is over 80 char long.
I've fixed the rest and will send updated patches.
Cheers,
Rüdiger
| [Prev in Thread] | Current Thread | [Next in Thread] |