Skip to content

Commit 48d82a8

Browse files
committed
Permute arguments
1 parent 264f10e commit 48d82a8

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

server/src/main/java/org/elasticsearch/action/ActionListener.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -340,7 +340,7 @@ static <Response> ActionListener<Response> runBefore(ActionListener<Response> de
340340
* Wraps a given listener and returns a new listener which releases the provided {@code releaseBefore}
341341
* resource before the listener is notified via either {@code #onResponse} or {@code #onFailure}.
342342
*/
343-
static <Response> ActionListener<Response> releaseBefore(ActionListener<Response> delegate, Releasable releaseBefore) {
343+
static <Response> ActionListener<Response> releaseBefore(Releasable releaseBefore, ActionListener<Response> delegate) {
344344
return assertOnce(
345345
new ActionListenerImplementations.RunBeforeActionListener<>(delegate, checkedRunnableFromReleasable(releaseBefore))
346346
);

server/src/test/java/org/elasticsearch/action/ActionListenerTests.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -612,9 +612,9 @@ public String toString() {
612612
}
613613

614614
public void testReleaseBefore() {
615-
runReleaseListenerTest(true, false, ActionListener::releaseBefore);
616-
runReleaseListenerTest(true, true, ActionListener::releaseBefore);
617-
runReleaseListenerTest(false, false, ActionListener::releaseBefore);
615+
runReleaseListenerTest(true, false, (delegate, releasable) -> ActionListener.releaseBefore(releasable, delegate));
616+
runReleaseListenerTest(true, true, (delegate, releasable) -> ActionListener.releaseBefore(releasable, delegate));
617+
runReleaseListenerTest(false, false, (delegate, releasable) -> ActionListener.releaseBefore(releasable, delegate));
618618
}
619619

620620
public void testReleaseAfter() {

0 commit comments

Comments
 (0)