[JENKINS-75361] Use local variable instead of calling getter twice

dwnusbaum · dwnusbaum · commit 21c02bc6b942 · 2025-03-04T15:29:50.000-05:00
diff --git a/plugin/src/main/java/org/jenkinsci/plugins/workflow/cps/LoggingInvoker.java b/plugin/src/main/java/org/jenkinsci/plugins/workflow/cps/LoggingInvoker.java
@@ -175,7 +175,7 @@ private Object findReceiver(Object o) {
                     }
                     // TODO: Groovy will actually try the delegate first and then the owner if needed, but it is
                     // difficult for us to know what will happen in advance.
-                    yield findReceiver(c.getDelegate());
+                    yield findReceiver(delegate);
                 }
                 default ->
                     // TODO: Groovy will actually try the owner first and then the delegate if needed.

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ private Object findReceiver(Object o) {`
`175`	`175`	`}`
`176`	`176`	`// TODO: Groovy will actually try the delegate first and then the owner if needed, but it is`
`177`	`177`	`// difficult for us to know what will happen in advance.`
`178`		`- yield findReceiver(c.getDelegate());`
	`178`	`+ yield findReceiver(delegate);`
`179`	`179`	`}`
`180`	`180`	`default ->`
`181`	`181`	`// TODO: Groovy will actually try the owner first and then the delegate if needed.`