[commons-beanutils] branch master updated: Use a switch instead of a cascading if-else.

[ggregory]

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-beanutils.git


The following commit(s) were added to refs/heads/master by this push:
     new d9ceea7  Use a switch instead of a cascading if-else.
d9ceea7 is described below

commit d9ceea77b8bdcbc0e3c9d2d4a04cc383f460ec7e
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Nov 20 15:28:06 2020 -0500

    Use a switch instead of a cascading if-else.
---
 .../apache/commons/beanutils2/expression/DefaultResolver.java | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/beanutils2/expression/DefaultResolver.java 
b/src/main/java/org/apache/commons/beanutils2/expression/DefaultResolver.java
index 32e30d8..6f23ccb 100644
--- 
a/src/main/java/org/apache/commons/beanutils2/expression/DefaultResolver.java
+++ 
b/src/main/java/org/apache/commons/beanutils2/expression/DefaultResolver.java
@@ -239,12 +239,17 @@ public class DefaultResolver implements Resolver {
                     return expression.substring(0, i + 1);
                 }
             } else {
-                if (c == NESTED) {
+                switch (c) {
+                case NESTED:
                     return expression.substring(0, i);
-                } else if (c == MAPPED_START) {
+                case MAPPED_START:
                     mapped = true;
-                } else if (c == INDEXED_START) {
+                    break;
+                case INDEXED_START:
                     indexed = true;
+                    break;
+                default:
+                    break;
                 }
             }
         }