[tor-commits] [tor-launcher/master] Remove TODO comments.

brade at torproject.org brade at torproject.org
Tue Apr 30 12:54:12 UTC 2013


commit 566c8a931a7359eb55ca3cc7c9bbfc561d3c3971
Author: Kathy Brade <brade at torproject.org>
Date:   Tue Apr 30 08:53:46 2013 -0400

    Remove TODO comments.
---
 src/components/tl-process.js |    8 --------
 1 files changed, 0 insertions(+), 8 deletions(-)

diff --git a/src/components/tl-process.js b/src/components/tl-process.js
index 1057795..3c6ed23 100644
--- a/src/components/tl-process.js
+++ b/src/components/tl-process.js
@@ -307,13 +307,6 @@ TorProcessService.prototype =
 
       if (isInitialBootstrap)
       {
-        // TODO: Sometimes the code in network-settings.js tries to retrieve
-        //       Tor settings (via GETCONF) before the Tor process has time
-        //       to open its control port listener.  We either need to add
-        //       a short delay here, find a way to monitor Tor to know when
-        //       it is ready to accept control port connections, or we need
-        //       to delay before retry inside TorSendCommand()... which is
-        //       difficult b/c that is a synchronous API.
         if (this.mProtocolSvc)
           this._openNetworkSettings(true); // Blocks until dialog is closed.
       }
@@ -363,7 +356,6 @@ TorProcessService.prototype =
     if (!aStatusObj)
       return;
 
-    // TODO: keep viewable log of RECOMMENDATION == "ignore" messages.
     if (100 == aStatusObj.PROGRESS)
     {
       this.mIsBootstrapDone = true;



More information about the tor-commits mailing list