This repository has been archived by the owner. It is now read-only.
Permalink
Browse files

repl completion: comment/TODO cleanup, no functional change

  • Loading branch information...
trentm authored and ry committed Aug 12, 2010
1 parent c5eb1b5 commit 2134982ea4d6262e96bcf2430dd79031d96ebf33
Showing with 2 additions and 3 deletions.
  1. +2 −3 lib/repl.js
View
@@ -150,7 +150,8 @@ REPLServer.prototype.readline = function (cmd) {
* -> [['sys.print', 'sys.debug', 'sys.log', 'sys.inspect', 'sys.pump'],
* 'sys.' ]
*
- * TODO: add warning about exec'ing code... property getters could be run
+ * Warning: This eval's code like "foo.bar.baz", so it will run property
+ * getter code.
*/
REPLServer.prototype.complete = function (line) {
@@ -231,8 +232,6 @@ REPLServer.prototype.complete = function (line) {
//console.log("completion eval error, expr='"+expr+"': "+e);
}
if (obj != null) {
- //TODO: The following, for example, misses "Object.isSealed". Is there
- // a way to introspec those? Need to hardcode?
if (typeof obj === "object" || typeof obj === "function") {
memberGroups.push(Object.getOwnPropertyNames(obj));
}

0 comments on commit 2134982

Please sign in to comment.