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

[Nicolas Goaziou]

Hello,

Litvinov Sergey <address@hidden> 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.

In the second case, (or var "nil") is redundant, as (format "%s" nil)
already returns "nil".

In other words, replacing (or var "nil") with var would be enough in
both cases.

Regards,

-- 
Nicolas Goaziou