repl: simplify code

This simplifies some repl code and removes a code branch that is unreachable. PR-URL: #30907 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Apr 28, 2020 · 6188650 · 6188650
1 parent 9b893e1
commit 6188650
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/lib/repl.js b/lib/repl.js
@@ -965,7 +965,7 @@ REPLServer.prototype.createContext = function() {
   }
 
   const module = new CJSModule('<repl>');
-  module.paths = CJSModule._resolveLookupPaths('<repl>', parentModule) || [];
+  module.paths = CJSModule._resolveLookupPaths('<repl>', parentModule);
 
   ObjectDefineProperty(context, 'module', {
     configurable: true,
@@ -1321,21 +1321,17 @@ function complete(line, callback) {
             }
             // Works for non-objects
             try {
-              let sentinel = 5;
               let p;
               if (typeof obj === 'object' || typeof obj === 'function') {
                 p = ObjectGetPrototypeOf(obj);
               } else {
                 p = obj.constructor ? obj.constructor.prototype : null;
               }
-              while (p !== null) {
+              // Circular refs possible? Let's guard against that.
+              let sentinel = 5;
+              while (p !== null && sentinel-- !== 0) {
                 memberGroups.push(filteredOwnPropertyNames(p));
                 p = ObjectGetPrototypeOf(p);
-                // Circular refs possible? Let's guard against that.
-                sentinel--;
-                if (sentinel <= 0) {
-                  break;
-                }
               }
             } catch {}
           }