diff --git a/rome-modules/src/main/java/com/rometools/modules/cc/types/License.java b/rome-modules/src/main/java/com/rometools/modules/cc/types/License.java index f904e5c..307bc5c 100644 --- a/rome-modules/src/main/java/com/rometools/modules/cc/types/License.java +++ b/rome-modules/src/main/java/com/rometools/modules/cc/types/License.java @@ -66,7 +66,6 @@ public class License { License.lookupLicense.put(uri, this); if (this.uri.endsWith("/")) { - // LOG.debug(uri.substring(0,this.uri.lastIndexOf("/"))); License.lookupLicense.put(uri.substring(0, this.uri.lastIndexOf("/")), this); } } diff --git a/rome-modules/src/main/java/com/rometools/modules/content/ContentItem.java b/rome-modules/src/main/java/com/rometools/modules/content/ContentItem.java index 8ca9600..ce21a17 100644 --- a/rome-modules/src/main/java/com/rometools/modules/content/ContentItem.java +++ b/rome-modules/src/main/java/com/rometools/modules/content/ContentItem.java @@ -114,39 +114,31 @@ public class ContentItem implements Cloneable { } final ContentItem other = (ContentItem) obj; if (contentFormat == null ? other.contentFormat != null : !contentFormat.equals(other.contentFormat)) { - // LOG.debug("format"); return false; } if (contentEncoding == null ? other.contentEncoding != null : !contentEncoding.equals(other.contentEncoding)) { - // LOG.debug("enc"); return false; } final String thisCV = contentValue.replaceAll(" xmlns=\"http://www.w3.org/1999/xhtml\"", "").trim(); - ; final String thatCV = other.contentValue.replaceAll(" xmlns=\"http://www.w3.org/1999/xhtml\"", "").trim(); if (contentValue == null ? other.contentValue != null : !thisCV.equals(thatCV)) { return false; } if (contentValueDOM != other.contentValueDOM && (contentValueDOM == null || !contentValueDOM.equals(other.contentValueDOM))) { - // LOG.debug("vd"); return false; } if (contentAbout == null ? other.contentAbout != null : !contentAbout.equals(other.contentAbout)) { - // LOG.debug("abt"); return false; } if (contentValueParseType == null ? other.contentValueParseType != null : !contentValueParseType.equals(other.contentValueParseType)) { - // LOG.debug("pt"); return false; } if (contentValueNamespace != other.contentValueNamespace && (contentValueNamespace == null || !contentValueNamespace.equals(other.contentValueNamespace))) { - // LOG.debug("ns"); return false; } if (contentResource == null ? other.contentResource != null : !contentResource.equals(other.contentResource)) { - // LOG.debug("res"); return false; } return true; diff --git a/rome-modules/src/main/java/com/rometools/modules/content/io/ContentModuleGenerator.java b/rome-modules/src/main/java/com/rometools/modules/content/io/ContentModuleGenerator.java index 9094385..77372d3 100644 --- a/rome-modules/src/main/java/com/rometools/modules/content/io/ContentModuleGenerator.java +++ b/rome-modules/src/main/java/com/rometools/modules/content/io/ContentModuleGenerator.java @@ -95,7 +95,6 @@ public class ContentModuleGenerator implements com.rometools.rome.io.ModuleGener } if (contentItem.getContentFormat() != null) { - // LOG.debug( "Format"); final Element format = new Element("format", CONTENT_NS); final Attribute formatResource = new Attribute("resource", contentItem.getContentFormat(), RDF_NS); format.setAttribute(formatResource); @@ -104,7 +103,6 @@ public class ContentModuleGenerator implements com.rometools.rome.io.ModuleGener } if (contentItem.getContentEncoding() != null) { - // LOG.debug( "Encoding"); final Element encoding = new Element("encoding", CONTENT_NS); final Attribute encodingResource = new Attribute("resource", contentItem.getContentEncoding(), RDF_NS); encoding.setAttribute(encodingResource); diff --git a/rome-modules/src/main/java/com/rometools/modules/sle/io/ItemParser.java b/rome-modules/src/main/java/com/rometools/modules/sle/io/ItemParser.java index 607c974..5e4ff12 100644 --- a/rome-modules/src/main/java/com/rometools/modules/sle/io/ItemParser.java +++ b/rome-modules/src/main/java/com/rometools/modules/sle/io/ItemParser.java @@ -83,10 +83,9 @@ public class ItemParser implements com.rometools.rome.io.ModuleParser { values = values.size() == 0 ? values : new ArrayList(); final List sorts = new ArrayList(element.getChildren("sort", ModuleParser.TEMP)); - // LOG.debug("]]] sorts on element"+sorts.size()); + for (final Element sort : sorts) { final String dataType = sort.getAttributeValue("data-type"); - // LOG.debug("Doing datatype "+dataType +" :: "+sorts.size()); if (dataType == null || dataType.equals(Sort.TEXT_TYPE)) { final StringValue value = new StringValue(); value.setElement(sort.getAttributeValue("element")); @@ -145,7 +144,7 @@ public class ItemParser implements com.rometools.rome.io.ModuleParser { throw new RuntimeException("Unknown datatype"); } } - // LOG.debug("Values created "+values.size()+" from sorts" +sorts.size()); + sle.setSortValues(values.toArray(new EntryValue[values.size()])); return sle; diff --git a/rome-modules/src/main/java/com/rometools/modules/sle/io/ModuleParser.java b/rome-modules/src/main/java/com/rometools/modules/sle/io/ModuleParser.java index 1cde3f6..3eca7d1 100644 --- a/rome-modules/src/main/java/com/rometools/modules/sle/io/ModuleParser.java +++ b/rome-modules/src/main/java/com/rometools/modules/sle/io/ModuleParser.java @@ -148,7 +148,7 @@ public class ModuleParser implements com.rometools.rome.io.ModuleParser { continue; } - // LOG.debug(e.getName()); + final Element value = e.getChild(sort2.getElement(), sort2.getNamespace()); if (value == null) { LOG.debug("No value for {} : {}", sort2.getElement(), sort2.getNamespace()); diff --git a/rome-modules/src/test/java/com/rometools/modules/sle/io/ModuleParserTest.java b/rome-modules/src/test/java/com/rometools/modules/sle/io/ModuleParserTest.java index 3e8caae..3038512 100644 --- a/rome-modules/src/test/java/com/rometools/modules/sle/io/ModuleParserTest.java +++ b/rome-modules/src/test/java/com/rometools/modules/sle/io/ModuleParserTest.java @@ -59,7 +59,7 @@ public class ModuleParserTest extends AbstractTestCase { final SyndFeedInput input = new SyndFeedInput(); final SyndFeed feed = input.build(new File(super.getTestFile("data/bookexample.xml"))); final SimpleListExtension sle = (SimpleListExtension) feed.getModule(SimpleListExtension.URI); - // LOG.debug( sle ); + assertEquals("list", sle.getTreatAs()); final Group[] groups = sle.getGroupFields(); assertEquals("genre", groups[0].getElement()); diff --git a/rome-modules/src/test/java/com/rometools/modules/sse/SSEParserTest.java b/rome-modules/src/test/java/com/rometools/modules/sse/SSEParserTest.java index f0f5846..6f4f8fb 100644 --- a/rome-modules/src/test/java/com/rometools/modules/sse/SSEParserTest.java +++ b/rome-modules/src/test/java/com/rometools/modules/sse/SSEParserTest.java @@ -226,13 +226,6 @@ public class SSEParserTest extends AbstractTestCase { } } - // if (!foundEqual) { - // // show accumulated error messages - // for (Iterator mesgIter = messages.iterator(); mesgIter.hasNext();) { - // LOG.debug((String) mesgIter.next()); - // } - // } - // look for the content in the other tree assertTrue("could not find matching element for: " + one.getName(), foundEqual); } diff --git a/rome/src/main/java/com/rometools/rome/io/impl/DateParser.java b/rome/src/main/java/com/rometools/rome/io/impl/DateParser.java index 68b5702..64adb6d 100644 --- a/rome/src/main/java/com/rometools/rome/io/impl/DateParser.java +++ b/rome/src/main/java/com/rometools/rome/io/impl/DateParser.java @@ -113,9 +113,7 @@ public class DateParser { if (pp.getIndex() != sDate.length()) { d = null; } - // LOG.debug("pp["+pp.getIndex()+"] s["+sDate+" m["+masks[i]+"] d["+d+"]"); } catch (final Exception ex1) { - // LOG.debug("s: "+sDate+" m: "+masks[i]+" d: "+null); } } return d;