Merge pull request #1 from barijaona/issue623-624

Follow up on pull request #623

src/AppController.m

@@ -2496,6 +2496,10 @@ -(void)viewArticlePages:(id)sender inPreferredBrowser:(BOOL)usePreferredBrowser
             }
 		}
 		[self openURLs:urls inPreferredBrowser:usePreferredBrowser];
+
+        if (([Preferences standardPreferences].markReadInterval > 0.0f) && !db.readOnly) {
+            [articleController markReadByArray:articlesWithLinks readFlag:YES];
+        }
 	}
 }
 

src/RichXMLParser.m

@@ -59,13 +59,16 @@ -(BOOL)parseRichXML:(NSData *)xmlData
                                                         error:&error];
         if (xmlDocument == nil && error != nil) {
 			if ([error.domain isEqualToString:NSXMLParserErrorDomain]) {
-				if (error.code == NSXMLParserGTRequiredError)
-					return NO;
-				if (error.code == NSXMLParserPrematureDocumentEndError)
+			    // handle here cases identified to cause
+			    // application crashes caused by
+			    // NSXMLDocument's -initWithData:options:error
+			    // when option NSXMLDocumentTidyXML is enabled
+				switch (error.code) {
+				case NSXMLParserGTRequiredError:
 					return NO;
+				}
 			}
-			// Do we even want to try this?
-			// We get assertion failures with the above 2 cases, so we'll probably get them with other cases too.
+			// recover some cases like text encoding errors, non standard tags...
             xmlDocument = [[NSXMLDocument alloc] initWithData:xmlData
                                                   options: NSXMLDocumentTidyXML
                                                   error:&error];