Fix args length == 1 case handling in ESLoggerUsageChecker (#102382)

* Fix args length == 1 case handling in ESLoggerUsageChecker

There was an operator precedence mistake in:
  (lengthWithoutMarker == 1 || lengthWithoutMarker == 2) && lengthWithoutMarker == 2 ? ...

Logical AND && has higher precedence than ternary operator ?:, So the above expression is equivalent to 
  lengthWithoutMarker == 2 ? ...

---------

Co-authored-by: Elastic Machine <[email protected]>

Author: sabi0

test/logger-usage/src/main/java/org/elasticsearch/test/loggerusage/ESLoggerUsageChecker.java

@@ -364,9 +364,8 @@ private void verifyLoggerUsage(
                 && argumentTypes[markerOffset + 1].equals(OBJECT_CLASS)) {
                     // MULTI-PARAM METHOD: debug(Marker?, String, Object p0, ...)
                     checkFixedArityArgs(methodNode, logMessageFrames[i], lineNumber, methodInsn, markerOffset + 0, lengthWithoutMarker - 1);
-                } else if ((lengthWithoutMarker == 1 || lengthWithoutMarker == 2) && lengthWithoutMarker == 2
-                    ? argumentTypes[markerOffset + 1].equals(THROWABLE_CLASS)
-                    : true) {
+                } else if (lengthWithoutMarker == 1
+                    || (lengthWithoutMarker == 2 && argumentTypes[markerOffset + 1].equals(THROWABLE_CLASS))) {
                         // all the rest: debug(Marker?, (Message|MessageSupplier|CharSequence|Object|String|Supplier), Throwable?)
                         checkFixedArityArgs(methodNode, logMessageFrames[i], lineNumber, methodInsn, markerOffset + 0, 0);
                     } else {