Skip to content

Commit c8c94f9

Browse files
committed
Add final modifier to method parameters
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/codec/trunk@1811343 13f79535-47bb-0310-9956-ffa450edef68
1 parent 7f56168 commit c8c94f9

13 files changed

Lines changed: 32 additions & 32 deletions

File tree

src/main/java/org/apache/commons/codec/binary/BaseNCodec.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -483,7 +483,7 @@ public byte[] encode(final byte[] pArray, int offset, int length) {
483483
* <code>false</code>, otherwise
484484
*/
485485
public boolean isInAlphabet(final byte[] arrayOctet, final boolean allowWSPad) {
486-
for (byte octet : arrayOctet) {
486+
for (final byte octet : arrayOctet) {
487487
if (!isInAlphabet(octet) &&
488488
(!allowWSPad || (octet != pad) && !isWhiteSpace(octet))) {
489489
return false;

src/main/java/org/apache/commons/codec/cli/Digest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ private void run() throws IOException {
102102
}
103103

104104
private void run(String[] digestAlgorithms) throws IOException {
105-
for (String messageDigestAlgorithm : digestAlgorithms) {
105+
for (final String messageDigestAlgorithm : digestAlgorithms) {
106106
if (DigestUtils.isAvailable(messageDigestAlgorithm)) {
107107
run(messageDigestAlgorithm + " ", messageDigestAlgorithm);
108108
}
@@ -114,7 +114,7 @@ private void run(String prefix, final MessageDigest messageDigest) throws IOExce
114114
println(prefix, DigestUtils.digest(messageDigest, System.in));
115115
return;
116116
}
117-
for(String source : inputs) {
117+
for(final String source : inputs) {
118118
final File file = new File(source);
119119
if (file.isFile()) {
120120
println(prefix, DigestUtils.digest(messageDigest, file), source);
@@ -132,7 +132,7 @@ private void run(String prefix, final MessageDigest messageDigest) throws IOExce
132132
}
133133

134134
private void run(String prefix, MessageDigest messageDigest, File[] files) throws IOException {
135-
for (File file : files) {
135+
for (final File file : files) {
136136
if (file.isFile()) {
137137
println(prefix, DigestUtils.digest(messageDigest, file), file.getName());
138138
}

src/main/java/org/apache/commons/codec/digest/HmacUtils.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ public static boolean isAvailable(String name) {
6969
try {
7070
Mac.getInstance(name);
7171
return true;
72-
} catch (NoSuchAlgorithmException e) {
72+
} catch (final NoSuchAlgorithmException e) {
7373
return false;
7474
}
7575
}
@@ -85,7 +85,7 @@ public static boolean isAvailable(HmacAlgorithms name) {
8585
try {
8686
Mac.getInstance(name.getName());
8787
return true;
88-
} catch (NoSuchAlgorithmException e) {
88+
} catch (final NoSuchAlgorithmException e) {
8989
return false;
9090
}
9191
}

src/main/java/org/apache/commons/codec/digest/PureJavaCrc32C.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public PureJavaCrc32C() {
4747

4848
@Override
4949
public long getValue() {
50-
long ret = crc;
50+
final long ret = crc;
5151
return (~ret) & 0xffffffffL;
5252
}
5353

src/main/java/org/apache/commons/codec/language/Soundex.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ public Soundex(final char[] mapping) {
160160
}
161161

162162
private boolean hasMarker(char[] mapping) {
163-
for(char ch : mapping) {
163+
for(final char ch : mapping) {
164164
if (ch == SILENT_MARKER) {
165165
return true;
166166
}
@@ -318,11 +318,11 @@ public String soundex(String str) {
318318
out[count++] = first;
319319
char lastDigit = map(first); // previous digit
320320
for(int i = 1; i < str.length() && count < out.length ; i++) {
321-
char ch = str.charAt(i);
321+
final char ch = str.charAt(i);
322322
if ((this.specialCaseHW) && (ch == 'H' || ch == 'W')) { // these are ignored completely
323323
continue;
324324
}
325-
char digit = map(ch);
325+
final char digit = map(ch);
326326
if (digit == SILENT_MARKER) {
327327
continue;
328328
}

src/test/java/org/apache/commons/codec/binary/Base32Test.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public class Base32Test {
5252
// BASE32_BINARY_TEST_CASES[2][0] = new Hex().decode("739ce739ce");
5353

5454
static {
55-
Hex hex = new Hex();
55+
final Hex hex = new Hex();
5656
try {
5757
BASE32_BINARY_TEST_CASES = new Object[][] {
5858
new Object[] { hex.decode("623a01735836e9a126e12fbf95e013ee6892997c"),
@@ -62,7 +62,7 @@ public class Base32Test {
6262
new Object[] { hex.decode("739ce42108"),
6363
"OOOOIIII" }
6464
};
65-
} catch (DecoderException de) {
65+
} catch (final DecoderException de) {
6666
throw new Error(":(", de);
6767
}
6868
}

src/test/java/org/apache/commons/codec/binary/Base64Test.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ private void testBase64InBuffer(int startPasSize, int endPadSize) {
130130
final byte[] bytesUtf8 = StringUtils.getBytesUtf8(content);
131131
byte[] buffer = ArrayUtils.addAll(bytesUtf8, new byte[endPadSize]);
132132
buffer = ArrayUtils.addAll(new byte[startPasSize], buffer);
133-
byte[] encodedBytes = new Base64().encode(buffer, startPasSize, bytesUtf8.length);
133+
final byte[] encodedBytes = new Base64().encode(buffer, startPasSize, bytesUtf8.length);
134134
encodedContent = StringUtils.newStringUtf8(encodedBytes);
135135
assertEquals("encoding hello world", "SGVsbG8gV29ybGQ=", encodedContent);
136136
}

src/test/java/org/apache/commons/codec/digest/DigestUtilsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ File getTestFile() {
6464
public void setUp() throws Exception {
6565
new Random().nextBytes(testData);
6666
testFile = File.createTempFile(DigestUtilsTest.class.getName(), ".dat");
67-
FileOutputStream fos = new FileOutputStream(testFile);
67+
final FileOutputStream fos = new FileOutputStream(testFile);
6868
fos.write(testData);
6969
fos.close();
7070
}

src/test/java/org/apache/commons/codec/digest/MessageDigestAlgorithmsTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public static Object[] data() {
4949
}
5050

5151
private static boolean contains(String key) {
52-
for(String s : MessageDigestAlgorithms.values()) {
52+
for(final String s : MessageDigestAlgorithms.values()) {
5353
if (s.equals(key)) {
5454
return true;
5555
}
@@ -59,10 +59,10 @@ private static boolean contains(String key) {
5959

6060
@BeforeClass
6161
public static void checkValues() throws Exception {
62-
Field [] fields = MessageDigestAlgorithms.class.getDeclaredFields();
62+
final Field [] fields = MessageDigestAlgorithms.class.getDeclaredFields();
6363
boolean ok = true;
6464
int psf = 0;
65-
for(Field f : fields) {
65+
for(final Field f : fields) {
6666
// Only interested in public fields
6767
final int modifiers = f.getModifiers();
6868
if (Modifier.isPublic(modifiers) && Modifier.isStatic(modifiers) && Modifier.isFinal(modifiers)) {

src/test/java/org/apache/commons/codec/digest/PureJavaCrc32CTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ public void testDecreasing() {
6666
private void check(int expected) {
6767
crc.reset();
6868
crc.update(data, 0, data.length);
69-
int actual = (int) crc.getValue();
69+
final int actual = (int) crc.getValue();
7070
Assert.assertEquals(Integer.toHexString(expected), Integer.toHexString(actual));
7171
}
7272

0 commit comments

Comments
 (0)