[PATCH] Babel: add results value support to Scala

[PATCH] Babel: add results value support to Scala

[Asrail]

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

* lisp/ob-scala.el (org-babel-scala-wrapper-method): Use an scala
block enclosing the submitted code

    The string representing an well formed block was not an Scala
code. I put the string from the user into an block, surrounded by an
call to replace the default output stream.

[-- Attachment #2: 0001-Babel-add-results-value-support-to-Scala.patch --]
[-- Type: application/octet-stream, Size: 1058 bytes --]

From 0d7b1b3e7e5d63a77a1fe47cee80fd7f7bcb4cce Mon Sep 17 00:00:00 2001
From: Caio Tiago Oliveira <asrail@gmail.com>
Date: Mon, 15 Oct 2012 15:52:50 -0300
Subject: [PATCH] Babel: add results value support to Scala

* lisp/ob-scala.el (org-babel-scala-wrapper-method): Use an scala block enclosing the submitted code

The string representing an well formed block was not an Scala code. I put the string from the user into an block, surrounded by an call to replace the default output stream.

TINYCHANGE
---
 lisp/ob-scala.el |   12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/lisp/ob-scala.el b/lisp/ob-scala.el
index bd855a8..ea3c3f2 100644
--- a/lisp/ob-scala.el
+++ b/lisp/ob-scala.el
@@ -72,9 +72,17 @@ Emacs-lisp table, otherwise return the results as a string."
 
 
 (defvar org-babel-scala-wrapper-method
-  "(
+
+"var str_result :String = null;
+
+Console.withOut(new java.io.OutputStream() {def write(b: Int){
+}}) {
+  str_result = {
 %s
-) asString print
+  }.toString
+}
+
+print(str_result)
 ")
 
 
-- 
1.7.10.4

Re: [PATCH] Babel: add results value support to Scala

[Asrail]

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

Hi,

Could someone please review this patch?
On Oct 15, 2012 4:39 PM, "Asrail" <asrail@gmail.com> wrote:

> * lisp/ob-scala.el (org-babel-scala-wrapper-method): Use an scala
> block enclosing the submitted code
>
>     The string representing an well formed block was not an Scala
> code. I put the string from the user into an block, surrounded by an
> call to replace the default output stream.
>

[-- Attachment #2: Type: text/html, Size: 658 bytes --]

Re: [PATCH] Babel: add results value support to Scala

[Eric Schulte]

Asrail <asrail@gmail.com> writes:

> * lisp/ob-scala.el (org-babel-scala-wrapper-method): Use an scala
> block enclosing the submitted code
>
>     The string representing an well formed block was not an Scala
> code. I put the string from the user into an block, surrounded by an
> call to replace the default output stream.
>

Hi Asrail,

Perhaps you can contact the author of ob-scala (Andrzej Lichnerowicz)
directly if he is not reading the mailing list every day.

Best,

-- 
Eric Schulte
http://cs.unm.edu/~eschulte

Re: [PATCH] Babel: add results value support to Scala

[Bastien]

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

Hi Andrzej,

Asrail posted this patch against ob-scala.el a while ago.

Can you confirm it is okay to apply it?  I'm reattaching 
the patch in case you are not reading the Orgmode mailing
list anymore.

Thanks in advance!

Asrail <asrail@gmail.com> writes:

> * lisp/ob-scala.el (org-babel-scala-wrapper-method): Use an scala
> block enclosing the submitted code


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Babel-add-results-value-support-to-Scala.patch --]
[-- Type: text/x-patch, Size: 1058 bytes --]

From 0d7b1b3e7e5d63a77a1fe47cee80fd7f7bcb4cce Mon Sep 17 00:00:00 2001
From: Caio Tiago Oliveira <asrail@gmail.com>
Date: Mon, 15 Oct 2012 15:52:50 -0300
Subject: [PATCH] Babel: add results value support to Scala

* lisp/ob-scala.el (org-babel-scala-wrapper-method): Use an scala block enclosing the submitted code

The string representing an well formed block was not an Scala code. I put the string from the user into an block, surrounded by an call to replace the default output stream.

TINYCHANGE
---
 lisp/ob-scala.el |   12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/lisp/ob-scala.el b/lisp/ob-scala.el
index bd855a8..ea3c3f2 100644
--- a/lisp/ob-scala.el
+++ b/lisp/ob-scala.el
@@ -72,9 +72,17 @@ Emacs-lisp table, otherwise return the results as a string."
 
 
 (defvar org-babel-scala-wrapper-method
-  "(
+
+"var str_result :String = null;
+
+Console.withOut(new java.io.OutputStream() {def write(b: Int){
+}}) {
+  str_result = {
 %s
-) asString print
+  }.toString
+}
+
+print(str_result)
 ")
 
 
-- 
1.7.10.4


[-- Attachment #3: Type: text/plain, Size: 14 bytes --]


-- 
 Bastien

Re: [PATCH] Babel: add results value support to Scala

[Andrzej Lichnerowicz]

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

Hi Bastien,

I'm here, but frankly the list is a bit hard to follow regularly :)

As for the patch, definitely! It's a lot more clean way, and fixes value
results. Thanks Asrail!


--a.

2012/10/24 Bastien <bzg@altern.org>

> Hi Andrzej,
>
> Asrail posted this patch against ob-scala.el a while ago.
>
> Can you confirm it is okay to apply it?  I'm reattaching
> the patch in case you are not reading the Orgmode mailing
> list anymore.
>
> Thanks in advance!
>
> Asrail <asrail@gmail.com> writes:
>
> > * lisp/ob-scala.el (org-babel-scala-wrapper-method): Use an scala
> > block enclosing the submitted code
>
>
>
> --
>  Bastien
>
>

[-- Attachment #2: Type: text/html, Size: 1045 bytes --]

Re: [PATCH] Babel: add results value support to Scala

[Bastien]

Hi Andrzej,

Andrzej Lichnerowicz <unjello@gmail.com> writes:

> I'm here, but frankly the list is a bit hard to follow regularly :)

You bet...

> As for the patch, definitely! It's a lot more clean way, and fixes
> value results. Thanks Asrail!

Thanks to both of you, I've now applied the patch!

Best,

-- 
 Bastien