(commons-compress) 03/07: Make org.apache.commons.compress.archivers.sevenz.SubStreamsInfo ivars final

[ggregory]

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

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-compress.git

commit 01c3f687cda0990ab3142a0d42ccf3b6201eb81e
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Feb 2 13:19:44 2024 -0500

    Make org.apache.commons.compress.archivers.sevenz.SubStreamsInfo ivars
    final
---
 .../apache/commons/compress/archivers/sevenz/SevenZFile.java |  6 +-----
 .../commons/compress/archivers/sevenz/SubStreamsInfo.java    | 12 +++++++++---
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java 
b/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
index bc3ae48b1..c860fc1e6 100644
--- a/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
+++ b/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
@@ -1601,11 +1601,7 @@ public class SevenZFile implements Closeable {
         }
 
         final int totalUnpackStreams = (int) unpackStreamsCount;
-        final SubStreamsInfo subStreamsInfo = new SubStreamsInfo();
-        subStreamsInfo.unpackSizes = new long[totalUnpackStreams];
-        subStreamsInfo.hasCrc = new BitSet(totalUnpackStreams);
-        subStreamsInfo.crcs = new long[totalUnpackStreams];
-
+        final SubStreamsInfo subStreamsInfo = new 
SubStreamsInfo(totalUnpackStreams);
         int nextUnpackStream = 0;
         for (final Folder folder : archive.folders) {
             if (folder.numUnpackSubStreams == 0) {
diff --git 
a/src/main/java/org/apache/commons/compress/archivers/sevenz/SubStreamsInfo.java
 
b/src/main/java/org/apache/commons/compress/archivers/sevenz/SubStreamsInfo.java
index 646600b0e..875b7dbcb 100644
--- 
a/src/main/java/org/apache/commons/compress/archivers/sevenz/SubStreamsInfo.java
+++ 
b/src/main/java/org/apache/commons/compress/archivers/sevenz/SubStreamsInfo.java
@@ -26,15 +26,21 @@ final class SubStreamsInfo {
     /**
      * Unpacked size of each unpacked stream.
      */
-    long[] unpackSizes;
+    final long[] unpackSizes;
 
     /**
      * Whether CRC is present for each unpacked stream.
      */
-    BitSet hasCrc;
+    final BitSet hasCrc;
 
     /**
      * CRCs of unpacked streams, if present.
      */
-    long[] crcs;
+    final long[] crcs;
+
+    SubStreamsInfo(final int totalUnpackStreams) {
+        this.unpackSizes = new long[totalUnpackStreams];
+        this.hasCrc = new BitSet(totalUnpackStreams);
+        this.crcs = new long[totalUnpackStreams];
+    }
 }