From c07c48e8bd29821375159f46d84b4236aaf2bb24 Mon Sep 17 00:00:00 2001
From: David Vazquez <davazp@gmail.com>
Date: Fri, 25 Jan 2013 16:45:57 +0000
Subject: [PATCH] Some comments

---
 ecmalisp.js   |    9 +++++++--
 ecmalisp.lisp |    9 ++++++++-
 2 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/ecmalisp.js b/ecmalisp.js
index fdef127..a19ea18 100644
--- a/ecmalisp.js
+++ b/ecmalisp.js
@@ -11,7 +11,7 @@
     return eval.apply(window, [string]);
 })();
 (function(){
-    var string = "var values = mv;";
+    var string = "var values = pv;";
     if (typeof string != 'string')
         throw 'The value ' + string + ' is not a type string.';
     return eval.apply(window, [string]);
@@ -10690,7 +10690,12 @@ var l283 = {name: "JS-EVAL"};
                                 var value = symbol.value;
                                 if (value === undefined) throw "Variable `" + symbol.name + "' is unbound.";
                                 return value;
-                            })(), l133.fvalue(pv, "var v = eval.apply(window, [string]));", "if (typeof v !== 'object' || !('multiple-value' in v)){", (function(){
+                            })(), l133.fvalue(pv, "var v = eval.apply(window, [string]);", (function(){
+                                var symbol = l123;
+                                var value = symbol.value;
+                                if (value === undefined) throw "Variable `" + symbol.name + "' is unbound.";
+                                return value;
+                            })(), "if (typeof v !== 'object' || !('multiple-value' in v)){", (function(){
                                 var symbol = l123;
                                 var value = symbol.value;
                                 if (value === undefined) throw "Variable `" + symbol.name + "' is unbound.";
diff --git a/ecmalisp.lisp b/ecmalisp.lisp
index feb8048..20a17b1 100644
--- a/ecmalisp.lisp
+++ b/ecmalisp.lisp
@@ -28,6 +28,10 @@
 #+ecmalisp
 (js-eval "function mv(){ var r = []; r['multiple-value'] = true; for (var i=0; i<arguments.length; i++) r.push(arguments[i]); return r; }")
 
+;;; NOTE: Define VALUES to be MV for toplevel forms. It is because
+;;; `eval' compiles the forms and execute the Javascript code at
+;;; toplevel with `js-eval', so it is necessary to return multiple
+;;; values from the eval function.
 #+ecmalisp
 (js-eval "var values = mv;")
 
@@ -988,6 +992,9 @@
 ;;; too. The respective real functions are defined in the target (see
 ;;; the beginning of this file) as well as some primitive functions.
 
+;;; If the special variable `*multiple-value-p*' is NON-NIL, then the
+;;; compilation of the current form is allowed to return multiple
+;;; values, using the VALUES variable.
 (defvar *multiple-value-p* nil)
 
 (defvar *compilation-unit-checks* '())
@@ -1881,7 +1888,7 @@
   (type-check (("string" "string" string))
     (if *multiple-value-p*
         (js!selfcall
-          "var v = eval.apply(window, [string]));"
+          "var v = eval.apply(window, [string]);" *newline*
           "if (typeof v !== 'object' || !('multiple-value' in v)){" *newline*
           (indent "v = [v];" *newline*
                   "v['multiple-value'] = true;" *newline*)
-- 
1.7.10.4