Re: [PATCH] org-get-cursor-date regexp patch

[No Wayman]

> Makes sense.  IMO it'd be nice to see something along the lines 
> of this
> explanation in the commit message itself.
>
> The function name is missing here:
>
>     * lisp/org.el (org-get-cursor-date): ...

> To my eyes, the new variable doesn't add any clarity over 
> keeping it
> inline.

Addressed in attached patch.

> Also, I very much like rx and I'm okay if you want to stick with 
> it
> here, but in this particular case I find it less readable than
>
>     \\([0-9]?[0-9]\\):\\([0-9][0-9]\\)
>
> or the stricter
>
>     \\([0-2]?[0-9]\\):\\([0-5][0-9]\\)

Fair enough. My preference for how the regular expression is 
notated does not matter so long as it is the correct expression.
I've inlined your second, stricter regular expression.

0001-org.el-org-get-cursor-date-Fix-regular-expression.patch
Description: org-get-cursor-date-regexp-revised