You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by ju...@apache.org on 2010/06/18 19:01:57 UTC
svn commit: r956050 - in /tika/trunk:
tika-core/src/main/java/org/apache/tika/mime/MediaTypeRegistry.java
tika-core/src/main/java/org/apache/tika/parser/CompositeParser.java
tika-parsers/src/test/java/org/apache/tika/mime/MimeTypesTest.java
Author: jukka
Date: Fri Jun 18 17:01:57 2010
New Revision: 956050
URL: http://svn.apache.org/viewvc?rev=956050&view=rev
Log:
TIKA-308: Improve supertype handling in type registry
It's "supertype" rather than "super type"
Modified:
tika/trunk/tika-core/src/main/java/org/apache/tika/mime/MediaTypeRegistry.java
tika/trunk/tika-core/src/main/java/org/apache/tika/parser/CompositeParser.java
tika/trunk/tika-parsers/src/test/java/org/apache/tika/mime/MimeTypesTest.java
Modified: tika/trunk/tika-core/src/main/java/org/apache/tika/mime/MediaTypeRegistry.java
URL: http://svn.apache.org/viewvc/tika/trunk/tika-core/src/main/java/org/apache/tika/mime/MediaTypeRegistry.java?rev=956050&r1=956049&r2=956050&view=diff
==============================================================================
--- tika/trunk/tika-core/src/main/java/org/apache/tika/mime/MediaTypeRegistry.java (original)
+++ tika/trunk/tika-core/src/main/java/org/apache/tika/mime/MediaTypeRegistry.java Fri Jun 18 17:01:57 2010
@@ -108,13 +108,14 @@ public class MediaTypeRegistry {
* Checks whether the given media type a is a specialization of a more
* generic type b.
*
+ * @since Apache Tika 0.8
* @param a media type
* @param b suspected supertype
* @return <code>true</code> if b is a supertype of a,
* <code>false</code> otherwise
*/
public boolean isSpecializationOf(MediaType a, MediaType b) {
- MediaType x = getSuperType(a);
+ MediaType x = getSupertype(a);
return x != null && (x.equals(b) || isSpecializationOf(x, b));
}
@@ -128,10 +129,11 @@ public class MediaTypeRegistry {
* supertype is known, and the return value for application/octet-stream
* is <code>null</code>.
*
+ * @since Apache Tika 0.8
* @param type media type
* @return supertype, or <code>null</code> for application/octet-stream
*/
- public MediaType getSuperType(MediaType type) {
+ public MediaType getSupertype(MediaType type) {
if (type.hasParameters()) {
return type.getBaseType();
} else if (inheritance.containsKey(type)) {
Modified: tika/trunk/tika-core/src/main/java/org/apache/tika/parser/CompositeParser.java
URL: http://svn.apache.org/viewvc/tika/trunk/tika-core/src/main/java/org/apache/tika/parser/CompositeParser.java?rev=956050&r1=956049&r2=956050&view=diff
==============================================================================
--- tika/trunk/tika-core/src/main/java/org/apache/tika/parser/CompositeParser.java (original)
+++ tika/trunk/tika-core/src/main/java/org/apache/tika/parser/CompositeParser.java Fri Jun 18 17:01:57 2010
@@ -135,7 +135,7 @@ public class CompositeParser implements
if (parser != null) {
return parser;
}
- type = registry.getSuperType(type);
+ type = registry.getSupertype(type);
}
return fallback;
}
Modified: tika/trunk/tika-parsers/src/test/java/org/apache/tika/mime/MimeTypesTest.java
URL: http://svn.apache.org/viewvc/tika/trunk/tika-parsers/src/test/java/org/apache/tika/mime/MimeTypesTest.java?rev=956050&r1=956049&r2=956050&view=diff
==============================================================================
--- tika/trunk/tika-parsers/src/test/java/org/apache/tika/mime/MimeTypesTest.java (original)
+++ tika/trunk/tika-parsers/src/test/java/org/apache/tika/mime/MimeTypesTest.java Fri Jun 18 17:01:57 2010
@@ -59,9 +59,9 @@ public class MimeTypesTest extends TestC
}
public void testSuperType() throws MimeTypeException {
- assertNull(registry.getSuperType(OCTET_STREAM));
- assertEquals(OCTET_STREAM, registry.getSuperType(TEXT_PLAIN));
- assertEquals(TEXT_PLAIN, registry.getSuperType(html.getType()));
+ assertNull(registry.getSupertype(OCTET_STREAM));
+ assertEquals(OCTET_STREAM, registry.getSupertype(TEXT_PLAIN));
+ assertEquals(TEXT_PLAIN, registry.getSupertype(html.getType()));
}
public void testIsDescendantOf() {