emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [O] make orgtbl-ascii-plot easier to install


From: Thierry Banel
Subject: Re: [O] make orgtbl-ascii-plot easier to install
Date: Thu, 28 Aug 2014 22:46:19 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.0

Hi Nicolas.

This new patch takes into account all your feedback.
Thanks again for the time spent !

Thierry


Le 28/08/2014 01:28, Nicolas Goaziou a écrit :
>
> You forgot a space at the end of the sentence. Actually, this is the
> case in all your docstrings.
Right. Fixed.

>
> Use "\\[universal-argument]" instead of "C-u".
I didn't knew this macro. Applied.

> Just to be sure to catch the following (odd) table
>
>  |--------|
>  |--------|
>  | header |
>  |--------|
>  | values |
>
> I suggest the following
>
>   (while (eq (car table) 'hline) (setq table (cdr table)))
>   (dolist (x (or (cdr (memq 'hline table)) table))
This is a strengthened algorithm, and it is shorter than the previous
version. Applied!

> The second sentence should start a new line.
Right. Fixed.

Attachment: 0001-Add-ascii-plotting-in-tables.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]