Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-24 Thread Rainer M Krug
"Charles C. Berry" writes: > On Mon, 23 Jun 2014, Rainer M Krug wrote: > >> "Charles C. Berry" writes: >> >> V> On Fri, 20 Jun 2014, Rainer M Krug wrote: >>> Attached please find =the reworked patch. 1) uses local() and closes connection 2) does not leave a variable cal;led f

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-24 Thread Rainer M Krug
"Charles C. Berry" writes: > On Mon, 23 Jun 2014, Rainer M Krug wrote: > >> One more question concerning the variable transfer of tables: >> >> , >> |(let ((file (orgtbl-to-tsv value '(:fmt org-babel-R-quote-tsv-field))) >> | (header (if (or (eq (nth 1 value) 'hline) colnames-p)

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-23 Thread Charles C. Berry
On Mon, 23 Jun 2014, Rainer M Krug wrote: "Charles C. Berry" writes: V> On Fri, 20 Jun 2014, Rainer M Krug wrote: Attached please find =the reworked patch. 1) uses local() and closes connection 2) does not leave a variable cal;led file behind [deleted] OK - attached please find two

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-23 Thread Charles C. Berry
On Mon, 23 Jun 2014, Rainer M Krug wrote: One more question concerning the variable transfer of tables: , | (let ((file (orgtbl-to-tsv value '(:fmt org-babel-R-quote-tsv-field))) | (header (if (or (eq (nth 1 value) 'hline) colnames-p) | "TRUE" "FALS

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-23 Thread Rainer M Krug
Aaron Ecay writes: > Hi Rainer, > > 2014ko ekainak 21an, "Charles C. Berry"-ek idatzi zuen: >> >> On Fri, 20 Jun 2014, Rainer M Krug wrote: >> >>> Attached please find =the reworked patch. >>> >>> 1) uses local() and closes connection >>> 2) does not leave a variable cal;led file behind >>> >

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-23 Thread Rainer M Krug
"Charles C. Berry" writes: V> On Fri, 20 Jun 2014, Rainer M Krug wrote: > >> Attached please find =the reworked patch. >> >> 1) uses local() and closes connection >> 2) does not leave a variable cal;led file behind >> > > Seems to work on a few test cases. > > I'd move the strings to defconsts. >

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-23 Thread Rainer M Krug
One more question concerning the variable transfer of tables: , | (let ((file (orgtbl-to-tsv value '(:fmt org-babel-R-quote-tsv-field))) | (header (if (or (eq (nth 1 value) 'hline) colnames-p) | "TRUE" "FALSE")) | (row-names (if rownames-p

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-23 Thread Rainer M Krug
"Charles C. Berry" writes: > On Fri, 20 Jun 2014, Rainer M Krug wrote: > >> Attached please find =the reworked patch. >> >> 1) uses local() and closes connection >> 2) does not leave a variable cal;led file behind >> > > Seems to work on a few test cases. > > I'd move the strings to defconsts. >

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-21 Thread Aaron Ecay
Hi Rainer, 2014ko ekainak 21an, "Charles C. Berry"-ek idatzi zuen: > > On Fri, 20 Jun 2014, Rainer M Krug wrote: > >> Attached please find =the reworked patch. >> >> 1) uses local() and closes connection >> 2) does not leave a variable cal;led file behind >> > > Seems to work on a few test ca

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-21 Thread Charles C. Berry
On Fri, 20 Jun 2014, Rainer M Krug wrote: Attached please find =the reworked patch. 1) uses local() and closes connection 2) does not leave a variable cal;led file behind Seems to work on a few test cases. I'd move the strings to defconsts. Putting a newline before each `%S' will outdent t

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-20 Thread Rainer M Krug
Attached please find =the reworked patch. 1) uses local() and closes connection 2) does not leave a variable cal;led file behind Cheers, Rainer From 2989c5890736099521a077d3b15d2ccc37c88b67 Mon Sep 17 00:00:00 2001 From: "Rainer M. Krug" Date: Fri, 20 Jun 2014 22:19:59 +0200 Subject: [PATCH] l

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-20 Thread Rainer M Krug
Charles Berry writes: > Aaron Ecay gmail.com> writes: > >> >> Hi Rainer, >> >> I have not tested the patch (I rarely use :var), but here are some >> comments from reading the patch. >> > [snip] >> >> > (header (if (or (eq (nth 1 value) 'hline) colnames-p) >> > "

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-20 Thread Rainer M Krug
Aaron Ecay writes: > Hi Rainer, > > I have not tested the patch (I rarely use :var), but here are some > comments from reading the patch. > > 2014ko ekainak 19an, Rainer M Krug-ek idatzi zuen: >> >> Hi >> >> This patch should fix the problem of tangling with variable transfer of >> tables. It u

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-19 Thread Charles Berry
Aaron Ecay gmail.com> writes: > > Hi Rainer, > > I have not tested the patch (I rarely use :var), but here are some > comments from reading the patch. > [snip] > > > (header (if (or (eq (nth 1 value) 'hline) colnames-p) > > "TRUE" "FALSE")) > > (row-n

Re: [O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-19 Thread Aaron Ecay
Hi Rainer, I have not tested the patch (I rarely use :var), but here are some comments from reading the patch. 2014ko ekainak 19an, Rainer M Krug-ek idatzi zuen: > > Hi > > This patch should fix the problem of tangling with variable transfer of > tables. It uses a textConection() instead of a f

[O] [PATCH] -for review- Tangling with variable transfer of variables

2014-06-19 Thread Rainer M Krug
Hi This patch should fix the problem of tangling with variable transfer of tables. It uses a textConection() instead of a file to transfer the table. This results in variable transfer of variables in R, without having to include additional files. Hope this does not work only for me. From f77e982