You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@abdera.apache.org by ro...@apache.org on 2006/08/19 17:25:48 UTC

svn commit: r432838 - /incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMEntry.java

Author: rooneg
Date: Sat Aug 19 08:25:48 2006
New Revision: 432838

URL: http://svn.apache.org/viewvc?rev=432838&view=rev
Log:
Let FindBugs have its way with FOMEntry.

* parser/src/main/java/org/apache/abdera/parser/stax/FOMEntry.java
  (setContent): Remove useless conditional, we were doing the same thing
   in both cases anyway.

Modified:
    incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMEntry.java

Modified: incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMEntry.java
URL: http://svn.apache.org/viewvc/incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMEntry.java?rev=432838&r1=432837&r2=432838&view=diff
==============================================================================
--- incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMEntry.java (original)
+++ incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMEntry.java Sat Aug 19 08:25:48 2006
@@ -208,10 +208,7 @@
    */
   public Content setContent(Element value) {
     FOMFactory factory = (FOMFactory) this.factory;
-    Content content = factory.newContent(
-      (value instanceof Div) ? 
-        Content.Type.XHTML : 
-        Content.Type.XHTML);
+    Content content = factory.newContent(Content.Type.XHTML);
     content.setValueElement(value);
     setContentElement(content);
     return content;