Björn Kettunen <[email protected]> writes: >> Any particular reason why you don't use the code I suggested? > > Mostly because I didn't see your reply until I posted the updated patch. > > But after reading the suggested code now it's also that it didn't seem > obvious to me that it can be shortened that much essentially you do the > same thin as I'm already doing but without the check if a table contains > any data.
Is that check necessary though? org-clock-get-table-data always returns non-empty list. > Also I think it's easier to read wrap the table merging and the > expanding of files with their archives into separate functions. > The only thing that bothers me that I don't know how to call the > get-table-data-with-archives function without the use of a lambda when > passing both the parameters and the file. I am not sure why using lambda is a problem. > What's the difference between using nthcdr 2 and cddr? The latter is slightly faster. That's it. > Instead of car and then nthcdr or cddr I could probably also do cddar. Sure, you can. -- Ihor Radchenko // yantar92, Org mode maintainer, Learn more about Org mode at <https://orgmode.org/>. Support Org development at <https://liberapay.com/org-mode>, or support my work at <https://liberapay.com/yantar92>
