This is an automated email from the ASF dual-hosted git repository.

jochen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-compress.git


The following commit(s) were added to refs/heads/master by this push:
     new acb980b  Use Boolean constants instead of Boolean.valueOf
     new 757c229  Merge pull request #225 from stefanbirkner/boolean
acb980b is described below

commit acb980bcb4b39607abc22860d7eb6fc2ecbeef60
Author: Stefan Birkner <m...@stefan-birkner.de>
AuthorDate: Tue Oct 12 19:07:19 2021 +0200

    Use Boolean constants instead of Boolean.valueOf
    
    Boolean.TRUE and Boolean.FALSE are shorter and easier to read. It also
    fixes a PMD violation.
---
 .../java/org/apache/commons/compress/harmony/unpack200/BcBands.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/compress/harmony/unpack200/BcBands.java 
b/src/main/java/org/apache/commons/compress/harmony/unpack200/BcBands.java
index d091a23..fbab072 100644
--- a/src/main/java/org/apache/commons/compress/harmony/unpack200/BcBands.java
+++ b/src/main/java/org/apache/commons/compress/harmony/unpack200/BcBands.java
@@ -189,12 +189,12 @@ public class BcBands extends BandSet {
                             bcLabelCount++;
                             break;
                         case 170: // tableswitch
-                            switchIsTableSwitch.add(Boolean.valueOf(true));
+                            switchIsTableSwitch.add(Boolean.TRUE);
                             bcCaseCountCount++;
                             bcLabelCount++;
                             break;
                         case 171: // lookupswitch
-                            switchIsTableSwitch.add(Boolean.valueOf(false));
+                            switchIsTableSwitch.add(Boolean.FALSE);
                             bcCaseCountCount++;
                             bcLabelCount++;
                             break;

Reply via email to