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;