From 4f54f29a2daa62839d22804d22544e4197cfda75 Mon Sep 17 00:00:00 2001 From: PJ Fanning Date: Sat, 16 Jul 2022 10:13:18 +0000 Subject: [github-345] Use switch instead of if. Thanks to XenoAmess. This closes #345 git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1902772 13f79535-47bb-0310-9956-ffa450edef68 --- .../main/java/org/apache/poi/hwpf/dev/RecordUtil.java | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) (limited to 'poi-scratchpad/src') diff --git a/poi-scratchpad/src/main/java/org/apache/poi/hwpf/dev/RecordUtil.java b/poi-scratchpad/src/main/java/org/apache/poi/hwpf/dev/RecordUtil.java index cd11e5d879..6c401cc512 100644 --- a/poi-scratchpad/src/main/java/org/apache/poi/hwpf/dev/RecordUtil.java +++ b/poi-scratchpad/src/main/java/org/apache/poi/hwpf/dev/RecordUtil.java @@ -107,12 +107,17 @@ public class RecordUtil byte numBits = 0; int mask = (int) Long.parseLong( bitMask.substring( 2 ), 16 ); - if ( parentType.equals( "byte" ) ) - parentSize = 8; - else if ( parentType.equals( "short" ) ) - parentSize = 16; - else if ( parentType.equals( "int" ) ) - parentSize = 32; + switch (parentType) { + case "byte": + parentSize = 8; + break; + case "short": + parentSize = 16; + break; + case "int": + parentSize = 32; + break; + } for ( int x = 0; x < parentSize; x++ ) { -- cgit v1.2.3