Fix painless return type cast for list shortcut (#126724)

This fixes an issue where if a Painless getter method return type
 didn't match a Java getter method return type we add a cast. 
Currentlythis is adding an extraneous cast.

Closes: #70682

Author: jdconrad

docs/changelog/126724.yaml

@@ -0,0 +1,5 @@
+pr: 126724
+summary: Fix painless return type cast for list shortcut
+area: Infra/Scripting
+type: bug
+issues: []

modules/lang-painless/src/main/java/org/elasticsearch/painless/phase/DefaultIRTreeToASMBytesPhase.java

@@ -1465,7 +1465,7 @@ public void visitLoadDotShortcut(LoadDotShortcutNode irDotSubShortcutNode, Write
         PainlessMethod getterPainlessMethod = irDotSubShortcutNode.getDecorationValue(IRDMethod.class);
         methodWriter.invokeMethodCall(getterPainlessMethod);
 
-        if (getterPainlessMethod.returnType().equals(getterPainlessMethod.javaMethod().getReturnType()) == false) {
+        if (getterPainlessMethod.returnType() != getterPainlessMethod.javaMethod().getReturnType()) {
             methodWriter.checkCast(MethodWriter.getType(getterPainlessMethod.returnType()));
         }
     }
@@ -1478,7 +1478,7 @@ public void visitLoadListShortcut(LoadListShortcutNode irLoadListShortcutNode, W
         PainlessMethod getterPainlessMethod = irLoadListShortcutNode.getDecorationValue(IRDMethod.class);
         methodWriter.invokeMethodCall(getterPainlessMethod);
 
-        if (getterPainlessMethod.returnType() == getterPainlessMethod.javaMethod().getReturnType()) {
+        if (getterPainlessMethod.returnType() != getterPainlessMethod.javaMethod().getReturnType()) {
             methodWriter.checkCast(MethodWriter.getType(getterPainlessMethod.returnType()));
         }
     }