You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/05/01 12:45:30 UTC

[commons-bcel] 01/49: Clean up: else { if { ... } } -> else if { ... }

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

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

commit b0a9a0a50c15b36a1580a6c875ecea6b29b70c4e
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun May 1 08:14:14 2022 -0400

    Clean up: else { if { ... } } -> else if { ... }
---
 src/main/java/org/apache/bcel/classfile/AccessFlags.java | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/apache/bcel/classfile/AccessFlags.java b/src/main/java/org/apache/bcel/classfile/AccessFlags.java
index 2d03a763..28ae68f8 100644
--- a/src/main/java/org/apache/bcel/classfile/AccessFlags.java
+++ b/src/main/java/org/apache/bcel/classfile/AccessFlags.java
@@ -81,10 +81,8 @@ public abstract class AccessFlags {
             if (!set) {
                 access_flags ^= flag;
             }
-        } else { // Flag not set
-            if (set) {
-                access_flags |= flag;
-            }
+        } else if (set) {
+            access_flags |= flag;
         }
     }