This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch 1.X in repository https://gitbox.apache.org/repos/asf/commons-beanutils.git
commit 2615e280aa393a6e38ab83c47bc28267576ea192 Author: Gary Gregory <[email protected]> AuthorDate: Sun Aug 18 08:46:32 2024 -0400 Use else if --- .../commons/beanutils/converters/NumberConverter.java | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/src/main/java/org/apache/commons/beanutils/converters/NumberConverter.java b/src/main/java/org/apache/commons/beanutils/converters/NumberConverter.java index 0ec16610..fb5931a9 100644 --- a/src/main/java/org/apache/commons/beanutils/converters/NumberConverter.java +++ b/src/main/java/org/apache/commons/beanutils/converters/NumberConverter.java @@ -222,18 +222,16 @@ public abstract class NumberConverter extends AbstractConverter { final DecimalFormatSymbols symbols = new DecimalFormatSymbols(locale); format = new DecimalFormat(pattern, symbols); } + } else if (locale == null) { + if (log().isDebugEnabled()) { + log().debug(" Using default Locale format"); + } + format = NumberFormat.getInstance(); } else { - if (locale == null) { - if (log().isDebugEnabled()) { - log().debug(" Using default Locale format"); - } - format = NumberFormat.getInstance(); - } else { - if (log().isDebugEnabled()) { - log().debug(" Using Locale[" + locale + "] format"); - } - format = NumberFormat.getInstance(locale); + if (log().isDebugEnabled()) { + log().debug(" Using Locale[" + locale + "] format"); } + format = NumberFormat.getInstance(locale); } if (!allowDecimals) { format.setParseIntegerOnly(true);
