Author: burton
Date: Fri Feb 4 11:54:23 2005
New Revision: 151398
URL: http://svn.apache.org/viewcvs?view=rev&rev=151398
Log:
dox for UTF-16 prolog bug... FIXME
Modified:
jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java
Modified:
jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java
URL:
http://svn.apache.org/viewcvs/jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java?view=diff&r1=151397&r2=151398
==============================================================================
---
jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java
(original)
+++
jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java
Fri Feb 4 11:54:23 2005
@@ -100,8 +100,13 @@
*/
private static String doRemoveLeadingProlog( String content, String
encoding ) {
- //if we're a UTF-16 or UTF-32 feed we need to LEAVE the prolog because
- //it triggers a UTF-16 parse.
+ // if we're a UTF-16 or UTF-32 feed we need to LEAVE the prolog because
+ // it triggers a UTF-16 parse due to the BOM.
+ //
+ // FIXME: this isn't actually true. We should leave the BOM and remove
+ // the prolog anyway due to the fact that this will still break the
+ // parser. Come up with some tests for UTF-16 to see if I can get it
to
+ // break and then update this method.
if ( "UTF-16".equals( encoding ) ||
"UTF-32".equals( encoding ) )
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
