Add final modifier to local variables.
diff --git a/src/main/java/org/apache/commons/compress/compressors/snappy/FramedSnappyCompressorInputStream.java b/src/main/java/org/apache/commons/compress/compressors/snappy/FramedSnappyCompressorInputStream.java
index 96ac7f9..9c0f1b9 100644
--- a/src/main/java/org/apache/commons/compress/compressors/snappy/FramedSnappyCompressorInputStream.java
+++ b/src/main/java/org/apache/commons/compress/compressors/snappy/FramedSnappyCompressorInputStream.java
@@ -201,7 +201,7 @@
uncompressedBytesRemaining = readSize() - 4 /* CRC */;
expectedChecksum = unmask(readCrc());
} else if (type == COMPRESSED_CHUNK_TYPE) {
- boolean expectChecksum = dialect.usesChecksumWithCompressedChunks();
+ final boolean expectChecksum = dialect.usesChecksumWithCompressedChunks();
final long size = readSize() - (expectChecksum ? 4 : 0);
if (expectChecksum) {
expectedChecksum = unmask(readCrc());
diff --git a/src/main/java/org/apache/commons/compress/utils/ArchiveUtils.java b/src/main/java/org/apache/commons/compress/utils/ArchiveUtils.java
index 1a7f975..38a4cf8 100644
--- a/src/main/java/org/apache/commons/compress/utils/ArchiveUtils.java
+++ b/src/main/java/org/apache/commons/compress/utils/ArchiveUtils.java
@@ -283,7 +283,7 @@
for (int i = 0; i < len; i++) {
final char c = chars[i];
if (!Character.isISOControl(c)) {
- Character.UnicodeBlock block = Character.UnicodeBlock.of(c);
+ final Character.UnicodeBlock block = Character.UnicodeBlock.of(c);
if (block != null && block != Character.UnicodeBlock.SPECIALS) {
sb.append(c);
continue;
diff --git a/src/test/java/org/apache/commons/compress/ArchiveUtilsTest.java b/src/test/java/org/apache/commons/compress/ArchiveUtilsTest.java
index 875d800..5dc7925 100644
--- a/src/test/java/org/apache/commons/compress/ArchiveUtilsTest.java
+++ b/src/test/java/org/apache/commons/compress/ArchiveUtilsTest.java
@@ -69,12 +69,12 @@
@Test
public void sanitizeShortensString() {
- String input = "012345678901234567890123456789012345678901234567890123456789"
+ final String input = "012345678901234567890123456789012345678901234567890123456789"
+ "012345678901234567890123456789012345678901234567890123456789"
+ "012345678901234567890123456789012345678901234567890123456789"
+ "012345678901234567890123456789012345678901234567890123456789"
+ "012345678901234567890123456789012345678901234567890123456789";
- String expected = "012345678901234567890123456789012345678901234567890123456789"
+ final String expected = "012345678901234567890123456789012345678901234567890123456789"
+ "012345678901234567890123456789012345678901234567890123456789"
+ "012345678901234567890123456789012345678901234567890123456789"
+ "012345678901234567890123456789012345678901234567890123456789"
@@ -84,14 +84,14 @@
@Test
public void sanitizeLeavesShortStringsAlone() {
- String input = "012345678901234567890123456789012345678901234567890123456789";
+ final String input = "012345678901234567890123456789012345678901234567890123456789";
assertEquals(input, ArchiveUtils.sanitize(input));
}
@Test
public void sanitizeRemovesUnprintableCharacters() {
- String input = "\b12345678901234567890123456789012345678901234567890123456789";
- String expected = "?12345678901234567890123456789012345678901234567890123456789";
+ final String input = "\b12345678901234567890123456789012345678901234567890123456789";
+ final String expected = "?12345678901234567890123456789012345678901234567890123456789";
assertEquals(expected, ArchiveUtils.sanitize(input));
}
diff --git a/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZFileTest.java b/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZFileTest.java
index 78e3ffb..0b88725 100644
--- a/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZFileTest.java
+++ b/src/test/java/org/apache/commons/compress/archivers/sevenz/SevenZFileTest.java
@@ -271,7 +271,7 @@
SevenZArchiveEntry entry = sevenZFile.getNextEntry();
while (entry != null) {
entries++;
- int b = sevenZFile.read();
+ final int b = sevenZFile.read();
if ("2.txt".equals(entry.getName()) || "5.txt".equals(entry.getName())) {
assertEquals(-1, b);
} else {
diff --git a/src/test/java/org/apache/commons/compress/archivers/zip/ZipArchiveInputStreamTest.java b/src/test/java/org/apache/commons/compress/archivers/zip/ZipArchiveInputStreamTest.java
index 33d0efc..587daf0 100644
--- a/src/test/java/org/apache/commons/compress/archivers/zip/ZipArchiveInputStreamTest.java
+++ b/src/test/java/org/apache/commons/compress/archivers/zip/ZipArchiveInputStreamTest.java
@@ -187,8 +187,8 @@
ze = in.getNextZipEntry();
}
fail("expected EOFException");
- } catch (EOFException ex) {
- String m = ex.getMessage();
+ } catch (final EOFException ex) {
+ final String m = ex.getMessage();
assertTrue(m.startsWith("Truncated ZIP entry: ?2016")); // the first character is not printable
} finally {
in.close();
diff --git a/src/test/java/org/apache/commons/compress/compressors/snappy/FramedSnappyCompressorInputStreamTest.java b/src/test/java/org/apache/commons/compress/compressors/snappy/FramedSnappyCompressorInputStreamTest.java
index b12ddeb..a1992a0 100644
--- a/src/test/java/org/apache/commons/compress/compressors/snappy/FramedSnappyCompressorInputStreamTest.java
+++ b/src/test/java/org/apache/commons/compress/compressors/snappy/FramedSnappyCompressorInputStreamTest.java
@@ -175,7 +175,7 @@
public void readIWAFile() throws Exception {
final ZipFile zip = new ZipFile(getFile("testNumbersNew.numbers"));
try {
- InputStream is = zip.getInputStream(zip.getEntry("Index/Document.iwa"));
+ final InputStream is = zip.getInputStream(zip.getEntry("Index/Document.iwa"));
try {
final FramedSnappyCompressorInputStream in =
new FramedSnappyCompressorInputStream(is, FramedSnappyDialect.IWORK_ARCHIVE);