You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by ta...@apache.org on 2020/09/18 14:21:00 UTC

[tika] branch main updated: Refactor fillSet in OPCPackageDetector (#346)

This is an automated email from the ASF dual-hosted git repository.

tallison pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tika.git


The following commit(s) were added to refs/heads/main by this push:
     new 0c4c24d  Refactor fillSet in OPCPackageDetector (#346)
0c4c24d is described below

commit 0c4c24dafe28c5cebb03f3946a3cfaf3c2cd04f4
Author: Lee <55...@users.noreply.github.com>
AuthorDate: Fri Sep 18 22:20:51 2020 +0800

    Refactor fillSet in OPCPackageDetector (#346)
---
 .../org/apache/tika/detect/microsoft/ooxml/OPCPackageDetector.java | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/tika-parser-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/detect/microsoft/ooxml/OPCPackageDetector.java b/tika-parser-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/detect/microsoft/ooxml/OPCPackageDetector.java
index 212fdb3..83f5521 100644
--- a/tika-parser-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/detect/microsoft/ooxml/OPCPackageDetector.java
+++ b/tika-parser-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/detect/microsoft/ooxml/OPCPackageDetector.java
@@ -43,6 +43,7 @@ import org.xml.sax.helpers.DefaultHandler;
 
 import java.io.IOException;
 import java.io.InputStream;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashSet;
 import java.util.Locale;
@@ -107,11 +108,7 @@ public class OPCPackageDetector implements ZipContainerDetector {
     );
 
     private static Set<String> fillSet(String ... args) {
-        Set<String> tmp = new HashSet<>();
-        for (String arg : args) {
-            tmp.add(arg);
-        }
-        return Collections.unmodifiableSet(tmp);
+        return Collections.unmodifiableSet(new HashSet<>((Arrays.asList(args))));
     }
 
     static Map<String, MediaType> OOXML_CONTENT_TYPES = new ConcurrentHashMap<>();