fix bug in ascii plot

fix bug in ascii plot

[Thierry Banel]

[-- Attachment #1: Type: text/plain, Size: 409 bytes --]

The 16c7594 commit (found by bisecting) revealeda bug in ascii-plot.
Attached is a fix.

To test, consider this table:
| 1 |
| 2 |
| 3 |

The cursor being in the table, type C-c " a


Result with the bug:
| 1 |   |
| 2 |   |
| 3 |   |
#+TBLFM:  2='(orgtbl-ascii-draw $1 1 3 12)


Result with the fix:
| 1 |              |
| 2 | WWWWWW       |
| 3 | WWWWWWWWWWWW |
#+TBLFM: $2='(orgtbl-ascii-draw $1 1 3 12)



[-- Attachment #2: 0001-Fix-missing-dollar-bug-revealed-by-commit-16c7594.patch --]
[-- Type: text/x-diff, Size: 846 bytes --]

From 19f0712d8bdefb76f54218068599b5abce6b18b9 Mon Sep 17 00:00:00 2001
From: Thierry Banel <tbanelwebmin@free.fr>
Date: Mon, 14 Dec 2015 22:22:03 +0100
Subject: [PATCH] Fix missing dollar bug revealed by commit 16c7594

* org-table.el (orgtbl-ascii-plot):
add missing dollar in formula
---
 lisp/org-table.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/org-table.el b/lisp/org-table.el
index 474a44f..265e74b 100644
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -5295,7 +5295,7 @@ characters width of the plot.  ASK may also be the
     (org-table-store-formulas
      (cons
       (cons
-       (number-to-string (1+ col))
+       (concat "$" (number-to-string (1+ col)))
        (format "'(%s $%s %s %s %s)"
 	       "orgtbl-ascii-draw" col min max length))
       (org-table-get-stored-formulas)))
-- 
2.1.4

Re: fix bug in ascii plot

[Nicolas Goaziou]

Hello,

Thierry Banel <tbanelwebmin@free.fr> writes:

> The 16c7594 commit (found by bisecting) revealeda bug in ascii-plot.
> Attached is a fix.
>
> To test, consider this table:
> | 1 |
> | 2 |
> | 3 |
>
> The cursor being in the table, type C-c " a
>
>
> Result with the bug:
> | 1 |   |
> | 2 |   |
> | 3 |   |
> #+TBLFM:  2='(orgtbl-ascii-draw $1 1 3 12)
>
>
> Result with the fix:
> | 1 |              |
> | 2 | WWWWWW       |
> | 3 | WWWWWWWWWWWW |
> #+TBLFM: $2='(orgtbl-ascii-draw $1 1 3 12)

Applied. Thank you.

Regards,

-- 
Nicolas Goaziou