Revision: 11518
          http://bibdesk.svn.sourceforge.net/bibdesk/?rev=11518&view=rev
Author:   amaxwell
Date:     2007-11-09 14:24:24 -0800 (Fri, 09 Nov 2007)

Log Message:
-----------
reduce number of #warnings, add comments

Modified Paths:
--------------
    trunk/bibdesk/BDSKWebGroupViewController.m

Modified: trunk/bibdesk/BDSKWebGroupViewController.m
===================================================================
--- trunk/bibdesk/BDSKWebGroupViewController.m  2007-11-09 21:32:22 UTC (rev 
11517)
+++ trunk/bibdesk/BDSKWebGroupViewController.m  2007-11-09 22:24:24 UTC (rev 
11518)
@@ -186,6 +186,7 @@
     NSArray *newPubs = [BDSKWebParser itemsFromDocument:domDocument 
fromURL:url error:&error];
     if (nil == newPubs) {
 #warning remove for release
+        // !!! logs are here to help diagnose problems that users are reporting
         NSLog(@"-[%@ [EMAIL PROTECTED] %@", [self class], 
NSStringFromSelector(_cmd), error);
         [NSApp presentError:error];
     }
@@ -200,13 +201,14 @@
 }
 
 - (void)webView:(WebView *)sender didFailProvisionalLoadWithError:(NSError 
*)error forFrame:(WebFrame *)frame{
-#warning is this really a failure?
+#warning FIXME: is this really a failure?
+    // arm: I'm not sure what the provisional load failure implies; from the 
docs it sounds like things will continue loading, so do we really want to kill 
things here?  If anyone knows or has a test case for this, please comment.
     if (frame == loadingWebFrame) {
         [self setRetrieving:NO];
         [group addPublications:nil];
         loadingWebFrame = nil;
     }
-#warning remove for release
+    // !!! logs are here to help diagnose problems that users are reporting
     NSLog(@"-[%@ [EMAIL PROTECTED] %@", [self class], 
NSStringFromSelector(_cmd), error);
 }
 
@@ -216,7 +218,7 @@
         [group addPublications:nil];
         loadingWebFrame = nil;
     }
-#warning remove for release
+    // !!! logs are here to help diagnose problems that users are reporting
     NSLog(@"-[%@ [EMAIL PROTECTED] %@", [self class], 
NSStringFromSelector(_cmd), error);
     [NSApp presentError:error];
 }


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Bibdesk-commit mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bibdesk-commit

Reply via email to