Re: [O] [PATCH][babel] add a string input to ob-octave

2011-10-06 Thread Eric Schulte
Fantastic, I will apply this as soon as your FSF attribution goes through. Speaking of has there been any progress on that front? Thanks -- Eric Litvinov Sergey writes: >> In other words, replacing (or var "nil") with var would be enough > > Thanks. Fixed in the patch below. The patch also add

Re: [O] [PATCH][babel] add a string input to ob-octave

2011-10-01 Thread Litvinov Sergey
> In other words, replacing (or var "nil") with var would be enough Thanks. Fixed in the patch below. The patch also adds an ert test. >From b4b679abdc7bec9f3033b50f81d567a0bb48b147 Mon Sep 17 00:00:00 2001 From: Litvinov Sergey Date: Sat, 1 Oct 2011 13:37:56 +0200 Subject: [PATCH 2/2] Remove red

Re: [O] [PATCH][babel] add a string input to ob-octave

2011-10-01 Thread Nicolas Goaziou
Hello, Litvinov Sergey writes: > +(cond > + ((stringp var) > + (format "\'%s\'" (or var "nil"))) > + (t > + (format "%s" (or var "nil")) Just nitpicking: In the first case, var is already identified as a string, so it will always be non-nil, and your "or" is useless.

[O] [PATCH][babel] add a string input to ob-octave

2011-09-30 Thread Litvinov Sergey
Please consider a tiny patch to add a string input variable to ob-octave. I also add tests for ob-octave. >From 4848960cefc1b1486705f7aed022ba199189b6e1 Mon Sep 17 00:00:00 2001 From: Litvinov Sergey Date: Sat, 1 Oct 2011 02:04:49 +0200 Subject: [PATCH] Add string input variables to ob-octave.el.