creadur-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rdon...@apache.org
Subject svn commit: r1548901 - /creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java
Date Sat, 07 Dec 2013 11:47:37 GMT
Author: rdonkin
Date: Sat Dec  7 11:47:37 2013
New Revision: 1548901

URL: http://svn.apache.org/r1548901
Log:
Delete TODO comment.

Modified:
    creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java

Modified: creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java
URL: http://svn.apache.org/viewvc/creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java?rev=1548901&r1=1548900&r2=1548901&view=diff
==============================================================================
--- creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java
(original)
+++ creadur/rat/branches/gsoc/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java
Sat Dec  7 11:47:37 2013
@@ -29,9 +29,7 @@ import java.nio.charset.CodingErrorActio
 
 import org.apache.rat.api.Document;
 
-/**
- * TODO: factor into MIME guesser and MIME->binary guesser
- */
+
 public class BinaryGuesser {
 
     private static final String[] DATA_EXTENSIONS = { "DAT", "DOC", "NCB",
@@ -56,9 +54,6 @@ public class BinaryGuesser {
     }
 
     public boolean matches(final Document document) {
-        // TODO: reimplement the binary test algorithm?
-        // TODO: more efficient to move into standard analysis
-        // TODO: then use binary as default
         return isBinary(document.getName()) ||
         // try a taste
                 isBinaryDocument(document);



Mime
View raw message