You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@creadur.apache.org by kh...@apache.org on 2017/03/09 17:58:25 UTC
svn commit: r1786209 - in /creadur/rat/trunk/apache-rat-core/src:
main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java
test/java/org/apache/rat/document/impl/guesser/BinaryGuesserTest.java
Author: khmarbaise
Date: Thu Mar 9 17:58:25 2017
New Revision: 1786209
URL: http://svn.apache.org/viewvc?rev=1786209&view=rev
Log:
[RAT-229] Mp3 files are not treated as binary
o Added several audio format extensions.
Modified:
creadur/rat/trunk/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java
creadur/rat/trunk/apache-rat-core/src/test/java/org/apache/rat/document/impl/guesser/BinaryGuesserTest.java
Modified: creadur/rat/trunk/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java
URL: http://svn.apache.org/viewvc/creadur/rat/trunk/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java?rev=1786209&r1=1786208&r2=1786209&view=diff
==============================================================================
--- creadur/rat/trunk/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java (original)
+++ creadur/rat/trunk/apache-rat-core/src/main/java/org/apache/rat/document/impl/guesser/BinaryGuesser.java Thu Mar 9 17:58:25 2017
@@ -206,6 +206,10 @@ public class BinaryGuesser {
return BinaryGuesser.extensionMatches(name, KEYSTORE_EXTENSIONS);
}
+ public static final boolean isAudio(final String name) {
+ return BinaryGuesser.extensionMatches( name, AUDIO_EXTENSIONS );
+ }
+
/**
* @param name file name.
* @return Is a file by that name a known binary file?
@@ -217,7 +221,8 @@ public class BinaryGuesser {
String normalisedName = GuessUtils.normalise(name);
return BinaryGuesser.JAR_MANIFEST.equalsIgnoreCase(name) || BinaryGuesser.isImage(normalisedName)
|| BinaryGuesser.isKeystore(normalisedName) || BinaryGuesser.isBytecode(normalisedName)
- || BinaryGuesser.isBinaryData(normalisedName) || BinaryGuesser.isExecutable(normalisedName);
+ || BinaryGuesser.isBinaryData(normalisedName) || BinaryGuesser.isExecutable(normalisedName)
+ || BinaryGuesser.isAudio( normalisedName );
}
private static final String[] DATA_EXTENSIONS = {
@@ -255,6 +260,14 @@ public class BinaryGuesser {
"OBJ", "PYC",
};
+ private static final String[] AUDIO_EXTENSIONS = {
+ "AIF", "IFF",
+ "M3U", "M4A",
+ "MID", "MP3",
+ "MPA", "WAV",
+ "WMA"
+ };
+
/**
* Based on http://www.apache.org/dev/svn-eol-style.txt
*/
Modified: creadur/rat/trunk/apache-rat-core/src/test/java/org/apache/rat/document/impl/guesser/BinaryGuesserTest.java
URL: http://svn.apache.org/viewvc/creadur/rat/trunk/apache-rat-core/src/test/java/org/apache/rat/document/impl/guesser/BinaryGuesserTest.java?rev=1786209&r1=1786208&r2=1786209&view=diff
==============================================================================
--- creadur/rat/trunk/apache-rat-core/src/test/java/org/apache/rat/document/impl/guesser/BinaryGuesserTest.java (original)
+++ creadur/rat/trunk/apache-rat-core/src/test/java/org/apache/rat/document/impl/guesser/BinaryGuesserTest.java Thu Mar 9 17:58:25 2017
@@ -50,7 +50,14 @@ public class BinaryGuesserTest {
"my.truststore",//
//"foo.Java", //
//"manifest.Mf",//
- "deprecatedtechnology.swf"
+ "deprecatedtechnology.swf",
+ "xyz.aif",
+ "abc.iff",
+ // Audio Files
+ "test.m3u", "test.m4a",
+ "test-audio.mid", "test-audio.mp3",
+ "test-audio.mpa", "test-audio.wav",
+ "test-audio.wma"
);
@Test