You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by bu...@apache.org on 2005/01/31 03:46:32 UTC

svn commit: r149204 - in jakarta/commons/sandbox/feedparser/trunk: TODO src/java/org/apache/commons/feedparser/FeedParserImpl.java

Author: burton
Date: Sun Jan 30 18:46:30 2005
New Revision: 149204

URL: http://svn.apache.org/viewcvs?view=rev&rev=149204
Log:
notes WRT JDOM b10 vs b9

Modified:
    jakarta/commons/sandbox/feedparser/trunk/TODO
    jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedParserImpl.java

Modified: jakarta/commons/sandbox/feedparser/trunk/TODO
URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/feedparser/trunk/TODO?view=diff&r1=149203&r2=149204
==============================================================================
--- jakarta/commons/sandbox/feedparser/trunk/TODO (original)
+++ jakarta/commons/sandbox/feedparser/trunk/TODO Sun Jan 30 18:46:30 2005
@@ -37,6 +37,12 @@
     - The older betas are NOT present so I really don't have any way to figure
       this out... Do I?  When did I add it to NewsMonster?
 
+- What build of JDOM are we running?
+    1.0beta9 built January 24 2003
+
+- Would it be possible to dump JDOM and use DOM with Jaxen until we're ready to
+  use SAX?
+
 - Maven javadoc is broken.  Why is this?
 
 - Better Atom support:

Modified: jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedParserImpl.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedParserImpl.java?view=diff&r1=149203&r2=149204
==============================================================================
--- jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedParserImpl.java (original)
+++ jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedParserImpl.java Sun Jan 30 18:46:30 2005
@@ -59,13 +59,17 @@
             // 2.0 (or as soon as we use SAX) this code should be totally
             // removed to use the original stream.
 
-            is = getCorrectInputStream(is);
+            is = getCorrectInputStream( is );
 
             //OK.  Now we have the right InputStream so we should build our DOM
             //and exec.
             DOMBuilder builder = new DOMBuilder();
 
-            org.jdom.Document doc = builder.build(is);
+            //NOTE: in b10 of JDOM this won't accept an InputStream and requires
+            //a org.w3c.dom.Document so we'll have to build one here.  Will this
+            //slow things down any?
+
+            org.jdom.Document doc = builder.build( is );
 
             parse(listener, doc);
 



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org