Throw appropriate errors. Add an @ to suppress debug output when...

Throw appropriate errors. Add an @ to suppress debug output when SimpleXMLElement constructor encounters problem. Output exception message (on FeedsSource level) instead.

libraries/opml_parser.inc

@@ -16,8 +16,7 @@
  */
 function opml_parser_parse($raw) {
   $feeds = $items = array();
-  $xml = new SimpleXMLElement($raw);
-
+  $xml = @ new SimpleXMLElement($raw);
   $feeds['title'] = (string)current($xml->xpath('//head/title'));
 
   // @todo Make xpath case insensitive.

plugins/FeedsFileFetcher.inc

@@ -16,8 +16,6 @@ class FeedsFileFetcher extends FeedsFetcher {
    */
   public function fetch(FeedsSource $source) {
     $source_config = $source->getConfigFor($this);
-    // Just return path to file, contents can be read easily with
-    // file_get_contents($file_path);
     return new FeedsImportBatch(NULL, $source_config['source']);
   }