This is an automated email from the ASF dual-hosted git repository. garydgregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-validator.git
commit 7fe8b61be1df7ce141f8c63d9c78338bfa57788b Author: Gary Gregory <[email protected]> AuthorDate: Sat Jul 11 07:21:17 2026 -0400 Merge if statements --- .../org/apache/commons/validator/routines/EmailValidator.java | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/main/java/org/apache/commons/validator/routines/EmailValidator.java b/src/main/java/org/apache/commons/validator/routines/EmailValidator.java index 2235f59b..406b43f7 100644 --- a/src/main/java/org/apache/commons/validator/routines/EmailValidator.java +++ b/src/main/java/org/apache/commons/validator/routines/EmailValidator.java @@ -178,13 +178,7 @@ public class EmailValidator implements Serializable { } // Check the whole email address structure final Matcher emailMatcher = EMAIL_PATTERN.matcher(email); - if (!emailMatcher.matches()) { - return false; - } - if (!isValidUser(emailMatcher.group(1))) { - return false; - } - if (!isValidDomain(emailMatcher.group(2))) { + if (!emailMatcher.matches() || !isValidUser(emailMatcher.group(1)) || !isValidDomain(emailMatcher.group(2))) { return false; } return true;
