Re: [O] [PATCH 3/9] org-clock: fix `org-clock-time%'

2015-11-04 Thread Jan Malakhovski
Aaron Ecay writes: > This could be converted to dolist while you’re here (I realize you > didn’t touch this line). Ok. >> - (if (string-match "\\([0-9]+\\):\\([0-9]+\\)" s) >> - (throw 'exit >> - (/ (* 100.0 (+ (string-to-number (match-string 2 s)) >> -

Re: [O] [PATCH 3/9] org-clock: fix `org-clock-time%'

2015-11-04 Thread Aaron Ecay
Hi Jan, A couple stylistic comments. 2015ko azaroak 3an, Jan Malakhovski-ek idatzi zuen: > > * lisp/org-clock.el (org-clock-time%): Respect org-effort-durations. > > This also fixes a bug with time percents looking pretty much random and adding > to a number that is less than 100% when a clock

[O] [PATCH 3/9] org-clock: fix `org-clock-time%'

2015-11-03 Thread Jan Malakhovski
* lisp/org-clock.el (org-clock-time%): Respect org-effort-durations. This also fixes a bug with time percents looking pretty much random and adding to a number that is less than 100% when a clock report has long intervals (e.g. days). --- lisp/org-clock.el | 29 +++-- 1 fi