Deactivated debug statements

Author: TheNinth7

source/main/data/communications/connection/ConnectionManager.mc

@@ -169,7 +169,7 @@ import Toybox.Time;
             // we perform the check only once, or if requested by a sync delegate after
             // a failed sync.
 
-            Logger.debug( "ConnectionManager.tryWifiConnectionAndTriggerNextRequest: checking for Wi-Fi connection" );
+            // Logger.debug( "ConnectionManager.tryWifiConnectionAndTriggerNextRequest: checking for Wi-Fi connection" );
 
             // WIFI_CHECK_PENDING is only used for the transition from
             // BLUETOOTH_CONNECTED to other states

source/main/data/communications/sync-delegates/base/BaseSyncDelegate.mc

@@ -166,7 +166,7 @@ class BaseSyncDelegate extends SyncDelegate {
     // Exits sync mode. Subclasses must call this method when their sync
     // operation has completed successfully.
     public function finishSync() as Void {
-        Logger.debug( "BaseSyncDelegate.finishSync" );
+        // Logger.debug( "BaseSyncDelegate.finishSync" );
         onSyncFinished();
         Communications.notifySyncComplete( null );
         // Logger.debug( "BaseSyncDelegate.finishSync end" );
@@ -191,14 +191,14 @@ class BaseSyncDelegate extends SyncDelegate {
     // To prevent this, we set _isSyncNeeded to true when a sync
     // starts and reset it to false once the sync has finished.
     public function isSyncNeeded() as Boolean {
-        Logger.debug( "BaseSyncDelegate.isSyncNeeded=" + _isSyncNeeded );
+        // Logger.debug( "BaseSyncDelegate.isSyncNeeded=" + _isSyncNeeded );
         return _isSyncNeeded;
     }
 
     // Subclasses must call this method when their sync operation fails.
     // It exits sync mode and displays an error message to the user.
     public function onSyncException( ex as Exception ) as Void {
-        Logger.debug( "BaseSyncDelegate.onSyncException" );
+        // Logger.debug( "BaseSyncDelegate.onSyncException" );
 
         onSyncFinished();
 
@@ -221,7 +221,7 @@ class BaseSyncDelegate extends SyncDelegate {
     // is handled here, so performSync implementation may throw 
     // exceptions and expect them to be handled.
     public function onStartSync() as Void {
-        Logger.debug( "BaseSyncDelegate.onStartSync" );
+        // Logger.debug( "BaseSyncDelegate.onStartSync" );
         try {
             // Logger.debugConnectionInfo();
 
@@ -243,7 +243,7 @@ class BaseSyncDelegate extends SyncDelegate {
     // with an error message.
     // Cancels all pending requests and exits sync mode.
     public function onStopSync() as Void {
-        Logger.debug( "BaseSyncDelegate.onStopSync" );
+        // Logger.debug( "BaseSyncDelegate.onStopSync" );
 
         // See consumeWifiCheckRequest() for details
         _wifiCheckRequest = true;
@@ -268,9 +268,9 @@ class BaseSyncDelegate extends SyncDelegate {
     // connection, which is likely to be closed before the request
     // completes.
     public function onSyncFinished() as Void {
-        Logger.debug( "BaseSyncDelegate.onSyncFinished" );
+        // Logger.debug( "BaseSyncDelegate.onSyncFinished" );
         if( ! _hasSyncFinished ) {
-            Logger.debug( "BaseSyncDelegate.onSyncFinished: cleaning up!" );
+            // Logger.debug( "BaseSyncDelegate.onSyncFinished: cleaning up!" );
 
             // See isSyncNeeded() for details.
             _isSyncNeeded = false;
@@ -285,7 +285,7 @@ class BaseSyncDelegate extends SyncDelegate {
             // See the class-level comment on onStopSync() for details.
             _hasSyncFinished = true;
         } else {
-            Logger.debug( "BaseSyncDelegate.onSyncFinished: was called already, doing nothing!" );
+            // Logger.debug( "BaseSyncDelegate.onSyncFinished: was called already, doing nothing!" );
         }
     }
 
@@ -300,7 +300,7 @@ class BaseSyncDelegate extends SyncDelegate {
     // Marks this instance as needing a sync, stops the sitemap request timer,
     // and requests sync mode from the CIQ API.
     public function startSync( msg as String ) as Void {
-        Logger.debug( "BaseSyncDelegate.startSync" );
+        // Logger.debug( "BaseSyncDelegate.startSync" );
         try {
             // Logger.debugConnectionInfo();
 
@@ -319,7 +319,7 @@ class BaseSyncDelegate extends SyncDelegate {
             _isSyncNeeded = true;
             _hasSyncFinished = false;
         } catch( ex ) {
-            Logger.debug( "BaseSyncDelegate.startSync: exception, restarting sitemap request" );
+            // Logger.debug( "BaseSyncDelegate.startSync: exception, restarting sitemap request" );
             SitemapRequest.get().start();
             throw ex;
         }

source/main/data/communications/sync-delegates/command/CommandSyncDelegate.mc

@@ -34,7 +34,7 @@ class CommandSyncDelegate extends BaseSyncDelegate {
 
     // Constructor
     private function initialize() {
-        Logger.debug( "CommandSyncDelegate.initialize" );
+        // Logger.debug( "CommandSyncDelegate.initialize" );
         BaseSyncDelegate.initialize();
     }
 
@@ -54,15 +54,15 @@ class CommandSyncDelegate extends BaseSyncDelegate {
         if( _cmd != null ) {
             return _cmd;
         } else {
-            Logger.debug( "CommandSyncDelegate.ensureCommand: command is missing" );
+            // Logger.debug( "CommandSyncDelegate.ensureCommand: command is missing" );
             throw new GeneralException( "CommandSyncDelegate: command is missing" );
         }
     }
 
     // Provides the item name to the command request
     // Part of the CommandRequestDelegate interface
     public function getItemName() as String {
-        Logger.debug( "CommandSyncDelegate.getItemName" );
+        // Logger.debug( "CommandSyncDelegate.getItemName" );
         return ensureCommand()[0];
     }
 
@@ -93,7 +93,7 @@ class CommandSyncDelegate extends BaseSyncDelegate {
     // a Monkey C limitation that prevents overriding protected or
     // private methods.
     public function performSync() as Void {
-        Logger.debug( "CommandSyncDelegate.performSync" );
+        // Logger.debug( "CommandSyncDelegate.performSync" );
 
         // Create the command request with self as dummy-item and 
         // true indicating that it should be in sync mode

source/main/data/communications/sync-delegates/sitemap/SitemapSyncTasks.mc

@@ -39,7 +39,7 @@ import Toybox.Lang;
     }
 
     public function invoke() as Void {
-        Logger.debug( "PostSyncTask: restarting sitemap request" );
+        // Logger.debug( "PostSyncTask: restarting sitemap request" );
         SitemapRequest.get().start();
     }
 }

source/main/infrastructure/appbase/OHApp.mc

@@ -111,15 +111,15 @@ class OHApp extends Application.AppBase {
     // to request a sitemap or send a command via WiFi.
     (:typecheck(disableGlanceCheck))
     function getSyncDelegate() as $.Toybox.Communications.SyncDelegate or Null {
-        Logger.debug( "OHApp.getSyncDelegate" );
+        // Logger.debug( "OHApp.getSyncDelegate" );
 
         var syncDelegate = BaseSyncDelegate.getCurrentSyncDelegate();
 
         if( syncDelegate != null ) {
-            Logger.debug( "OHApp.getSyncDelegate: returning current sync delegate" );
+            // Logger.debug( "OHApp.getSyncDelegate: returning current sync delegate" );
             return syncDelegate;
         } else {
-            Logger.debug( "OHApp.getSyncDelegate: no current sync delegate, defaulting to sitemap sync delegate" );
+            // Logger.debug( "OHApp.getSyncDelegate: no current sync delegate, defaulting to sitemap sync delegate" );
             return SitemapSyncDelegate.get();
         }
     }