|
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] |