It should be pretty easy to add an interactive mode to Narcissus, so it's more-or-less drop-in-replaceable for the JS shell. Dave

Comment on attachment 462205 [details] [diff] [review] adds interactive REPL to narcissus >+ if (typeof x == "object") { >+ if (x != null && "toSource" in x) { >+ try { >+ print(x.toSource()); >+ } catch (e) { >+ } Maybe something should be printed here. >+ } else { >+ print("null"); >+ } >+ } else if (typeof x == "string") { >+ print(uneval(x)); >+ } else if (typeof x != "undefined") { >+ print(String(x)); Do you mean the string() function below?

> Maybe something should be printed here. I should've commented-- the somewhat weird logic is (at least roughly) tracking the JS shell, just because I figured acting gratuitously differently might confuse someone trying to debug Narcissus. I'll add comments to the function. > >+ } else if (typeof x != "undefined") { > >+ print(String(x)); > > Do you mean the string() function below? Since it's not an object, it can't throw, so there's no need to call the string() function. I'll add a comment. Dave

Rebased and added comments to address pwalton's review. Dave

Comment on attachment 463346 [details] [diff] [review] adds interactive REPL to narcissus r=me