Skip to content

Commit 19478da

Browse files
committed
Use final on some lvars to clarify intent and avoid potential bugs. Should final be applied all over?
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/codec/trunk@1378966 13f79535-47bb-0310-9956-ffa450edef68
1 parent 1a3a1b3 commit 19478da

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -289,15 +289,15 @@ void decode(byte[] in, int inPos, int inAvail, Context context) { // package pro
289289
context.eof = true;
290290
}
291291
for (int i = 0; i < inAvail; i++) {
292-
byte b = in[inPos++];
292+
final byte b = in[inPos++];
293293
if (b == PAD) {
294294
// We're done.
295295
context.eof = true;
296296
break;
297297
} else {
298298
final byte[] buffer = ensureBufferSize(decodeSize, context);
299299
if (b >= 0 && b < this.decodeTable.length) {
300-
int result = this.decodeTable[b];
300+
final int result = this.decodeTable[b];
301301
if (result >= 0) {
302302
context.modulus = (context.modulus+1) % BYTES_PER_ENCODED_BLOCK;
303303
// collect decoded bytes
@@ -384,7 +384,7 @@ void encode(byte[] in, int inPos, int inAvail, Context context) { // package pro
384384
return; // no leftovers to process and not using chunking
385385
}
386386
final byte[] buffer = ensureBufferSize(encodeSize, context);
387-
int savedPos = context.pos;
387+
final int savedPos = context.pos;
388388
switch (context.modulus) { // % 5
389389
case 1 : // Only 1 octet; take top 5 bits then remainder
390390
buffer[context.pos++] = encodeTable[(int)(context.lbitWorkArea >> 3) & MASK_5BITS]; // 8-1*5 = 3

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -335,7 +335,7 @@ void encode(byte[] in, int inPos, int inAvail, Context context) {
335335
return; // no leftovers to process and not using chunking
336336
}
337337
final byte[] buffer = ensureBufferSize(encodeSize, context);
338-
int savedPos = context.pos;
338+
final int savedPos = context.pos;
339339
switch (context.modulus) { // 0-2
340340
case 1 : // 8 bits = 6 + 2
341341
// top 6 bits:

0 commit comments

Comments
 (0)