Issue skulpt#33

Make open('filename') without the second parameter work. There is no unit test because I don't know how to set it up. The way I test this in a browser is without this change, the following code: h = open("mbox-short.txt") print len(h.read()) Throws "todo; haven't implemented non-read opens" With this fix the code works.
Duncanr-glitch · Jan 23, 2013 · 9f3630f · 9f3630f
1 parent 2fdf4d1
commit 9f3630f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/builtin.js b/src/builtin.js
@@ -124,7 +124,7 @@ Sk.builtin.repr = function repr(x)
 
 Sk.builtin.open = function open(filename, mode, bufsize)
 {
-    if (mode === undefined) mode = "r";
+    if (mode === undefined) mode = new Sk.builtin.str("r");
     if (mode.v !== "r" && mode.v !== "rb") throw "todo; haven't implemented non-read opens";
     return new Sk.builtin.file(filename, mode, bufsize);
 };
@@ -221,4 +221,4 @@ Sk.builtin.read = function read(x) {
     return Sk.builtinFiles["files"][x];
 };
 Sk.builtinFiles = undefined;
-*/
+*/