Apache
/
poi
mirror of https://github.com/apache/poi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

findbugs: replace switch statement with fall-throughs with if statement, remove duplicate code 

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1751738 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Javen O'Neal 2016-07-07 00:48:52 +00:00
parent 981d9dabb4
commit 5c43bcf436
1 changed files with 27 additions and 50 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

57
src/java/org/apache/poi/util/HexRead.java
View File

@ -108,21 +108,28 @@ public class HexRead
int characterCount = 0;
byte b = (byte) 0;
List<Byte> bytes = new ArrayList<Byte>();
boolean done = false;
while ( !done )
final char a = 'a' - 10;
final char A = 'A' - 10;
while ( true )
{
int count = stream.read();
char baseChar = 'a';
if ( count == eofChar ) break;
switch ( count )
{
case '#':
int digitValue = -1;
if ( '0' <= count && count <= '9' ) {
digitValue = count - '0';
} else if ( 'A' <= count && count <= 'F' ) {
digitValue = count - A;
} else if ( 'a' <= count && count <= 'f' ) {
digitValue = count - a;
} else if ( '#' == count ) {
readToEOL( stream );
} else if ( -1 == count || eofChar == count ) {
break;
case '0': case '1': case '2': case '3': case '4': case '5':
case '6': case '7': case '8': case '9':
}
// else: ignore the character
if (digitValue != -1) {
b <<= 4;
b += (byte) ( count - '0' );
b += (byte) digitValue;
characterCount++;
if ( characterCount == 2 )
{
@ -130,36 +137,6 @@ public class HexRead
characterCount = 0;
b = (byte) 0;
}
break;
case 'A':
case 'B':
case 'C':
case 'D':
case 'E':
case 'F':
baseChar = 'A';
// fall through
case 'a':
case 'b':
case 'c':
case 'd':
case 'e':
case 'f':
b <<= 4;
b += (byte) ( count + 10 - baseChar );
characterCount++;
if ( characterCount == 2 )
{
bytes.add( Byte.valueOf( b ) );
characterCount = 0;
b = (byte) 0;
}
break;
case -1:
done = true;
break;
default :
break;
}
}
Byte[] polished = bytes.toArray( new Byte[0] );