From 2f9d0a7de8e2f1247dd4a34811220106f7359aa5 Mon Sep 17 00:00:00 2001 From: igor-suhorukov Date: Fri, 9 Mar 2018 00:03:09 +0300 Subject: [PATCH] simplify boolean NOT operation --- .../jdbc/core/namedparam/NamedParameterUtils.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/core/namedparam/NamedParameterUtils.java b/spring-jdbc/src/main/java/org/springframework/jdbc/core/namedparam/NamedParameterUtils.java index 1dd22bcaa7..b3b91cb3f0 100644 --- a/spring-jdbc/src/main/java/org/springframework/jdbc/core/namedparam/NamedParameterUtils.java +++ b/spring-jdbc/src/main/java/org/springframework/jdbc/core/namedparam/NamedParameterUtils.java @@ -119,7 +119,7 @@ public abstract class NamedParameterUtils { String parameter = null; if (j < statement.length && c == ':' && statement[j] == '{') { // :{x} style parameter - while (j < statement.length && !('}' == statement[j])) { + while (j < statement.length && '}' != statement[j]) { j++; if (':' == statement[j] || '{' == statement[j]) { throw new InvalidDataAccessApiUsageException("Parameter name contains invalid character '" + @@ -210,7 +210,7 @@ public abstract class NamedParameterUtils { if (statement[position] == START_SKIP[i].charAt(0)) { boolean match = true; for (int j = 1; j < START_SKIP[i].length(); j++) { - if (!(statement[position + j] == START_SKIP[i].charAt(j))) { + if (statement[position + j] != START_SKIP[i].charAt(j)) { match = false; break; } @@ -226,7 +226,7 @@ public abstract class NamedParameterUtils { // last comment not closed properly return statement.length; } - if (!(statement[m + n] == STOP_SKIP[i].charAt(n))) { + if (statement[m + n] != STOP_SKIP[i].charAt(n)) { endMatch = false; break; }