Merge pull request #562 from mousetraps/i466

#466 Debugger does not start reliably running Node.js console app
microsoft · Oct 22, 2015 · fb5c884 · fb5c884
2 parents e2a97de + b57da3c
commit fb5c884
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Nodejs/Product/Nodejs/Debugger/Communication/DebuggerConnection.cs b/Nodejs/Product/Nodejs/Debugger/Communication/DebuggerConnection.cs
@@ -122,7 +122,7 @@ sealed class DebuggerConnection : IDebuggerConnection {
             lock (_networkClientLock) {
                 int connection_attempts = 0;
                 const int MAX_ATTEMPTS = 5;
-                while (!_isClosed) {
+                while (true) {
                     connection_attempts++;
                     try {
                         // TODO: This currently results in a call to the synchronous TcpClient
@@ -147,7 +147,7 @@ sealed class DebuggerConnection : IDebuggerConnection {
                             throw;
                         }
                         LiveLogger.WriteLine("Connection attempt {0} failed with: {1}", connection_attempts, ex);
-                        if (connection_attempts >= MAX_ATTEMPTS) {
+                        if (connection_attempts >= MAX_ATTEMPTS && !_isClosed) {
                             throw;
                         }
                         else {