(tika) branch branch_2x updated: TIKA-4531: remove leaky / dead / not needed code, fix javadoc

[tilman]

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

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


The following commit(s) were added to refs/heads/branch_2x by this push:
     new 3b273d607 TIKA-4531: remove leaky / dead / not needed code, fix javadoc
3b273d607 is described below

commit 3b273d607ba6c0a66387ad4337b3decb10214bbf
Author: Tilman Hausherr <til...@apache.org>
AuthorDate: Tue Oct 28 12:33:59 2025 +0100

    TIKA-4531: remove leaky / dead / not needed code, fix javadoc
---
 .../tika/parser/microsoft/ooxml/AbstractOOXMLExtractor.java      | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git 
a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/AbstractOOXMLExtractor.java
 
b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/AbstractOOXMLExtractor.java
index a99e26301..c7cfecfa4 100644
--- 
a/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/AbstractOOXMLExtractor.java
+++ 
b/tika-parsers/tika-parsers-standard/tika-parsers-standard-modules/tika-parser-microsoft-module/src/main/java/org/apache/tika/parser/microsoft/ooxml/AbstractOOXMLExtractor.java
@@ -72,7 +72,7 @@ import org.apache.tika.utils.XMLReaderUtils;
 
 /**
  * Base class for all Tika OOXML extractors.
- * <p/>
+ * <p>
  * Tika extractors decorate POI extractors so that the parsed content of
  * documents is returned as a sequence of XHTML SAX events. Subclasses must
  * implement the buildXHTML method {@link #buildXHTML(XHTMLContentHandler)} 
that
@@ -329,9 +329,8 @@ public abstract class AbstractOOXMLExtractor implements 
OOXMLExtractor {
             return;
         }
 
-        InputStream is = part.getInputStream();
         // Open the POIFS (OLE2) structure and process
-        POIFSFileSystem fs = null;
+        POIFSFileSystem fs;
         try {
             fs = new POIFSFileSystem(part.getInputStream());
         } catch (Exception e) {
@@ -398,9 +397,7 @@ public abstract class AbstractOOXMLExtractor implements 
OOXMLExtractor {
         } catch (IOException e) {
             EmbeddedDocumentUtil.recordEmbeddedStreamException(e, 
parentMetadata);
         } finally {
-            if (fs != null) {
-                fs.close();
-            }
+            fs.close();
             if (stream != null) {
                 stream.close();
             }