Author: fanningpj
Date: Sat Sep 10 18:56:15 2022
New Revision: 1903985

URL: http://svn.apache.org/viewvc?rev=1903985&view=rev
Log:
[bug-66257] javadoc

Modified:
    
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java

Modified: 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java?rev=1903985&r1=1903984&r2=1903985&view=diff
==============================================================================
--- 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java
 (original)
+++ 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java
 Sat Sep 10 18:56:15 2022
@@ -61,19 +61,21 @@ public class XSSFBReader extends XSSFRea
     private static final Logger LOGGER = 
LogManager.getLogger(XSSFBReader.class);
     private static final Set<String> WORKSHEET_RELS =
             Collections.unmodifiableSet(new HashSet<>(
-                    Arrays.asList(new String[]{
+                    Arrays.asList(
                             XSSFRelation.WORKSHEET.getRelation(),
                             XSSFRelation.CHARTSHEET.getRelation(),
                             XSSFRelation.MACRO_SHEET_BIN.getRelation(),
                             XSSFRelation.INTL_MACRO_SHEET_BIN.getRelation(),
                             XSSFRelation.DIALOG_SHEET_BIN.getRelation()
-                    })
+                    )
             ));
 
     /**
      * Creates a new XSSFReader, for the given package
      *
      * @param pkg opc package
+     * @throws OpenXML4JException if the package data format is invalid
+     * @throws IOException if there is an I/O issue reading the data
      */
     public XSSFBReader(OPCPackage pkg) throws IOException, OpenXML4JException {
         super(pkg);
@@ -101,6 +103,10 @@ public class XSSFBReader extends XSSFRea
      * Each sheet's InputStream is only opened when fetched
      *  from the Iterator. It's up to you to close the
      *  InputStreams when done with each one.
+     *
+     * @return iterator of {@link InputStream}s
+     * @throws InvalidFormatException if the sheet data format is invalid
+     * @throws IOException if there is an I/O issue reading the data
      */
     @Override
     public Iterator<InputStream> getSheetsData() throws IOException, 
InvalidFormatException {
@@ -123,8 +129,10 @@ public class XSSFBReader extends XSSFRea
          * Construct a new SheetIterator
          *
          * @param wb package part holding workbook.xml
+         * @throws InvalidFormatException if the sheet data format is invalid
+         * @throws IOException if there is an I/O issue reading the data
          */
-        private SheetIterator(PackagePart wb) throws IOException {
+        private SheetIterator(PackagePart wb) throws IOException, 
InvalidFormatException {
             super(wb);
         }
 
@@ -180,7 +188,7 @@ public class XSSFBReader extends XSSFRea
 
 
     private static class PathExtractor extends XSSFBParser {
-        private static SparseBitSet RECORDS = new SparseBitSet();
+        private static final SparseBitSet RECORDS = new SparseBitSet();
         static {
             RECORDS.set(XSSFBRecordType.BrtAbsPath15.getId());
         }
@@ -277,10 +285,7 @@ public class XSSFBReader extends XSSFRea
             if (StringUtil.isNotBlank(relId)) {
                 sheets.add(new XSSFSheetRef(relId, name));
             }
-            if (offset == data.length) {
-                return true;
-            }
-            return false;
+            return offset == data.length;
         }
 
         List<XSSFSheetRef> getSheets() {



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to