Skip to content

Commit

Permalink
KQueueIoHandler should pass IoOps to evSet (#14054)
Browse files Browse the repository at this point in the history
Motivation:

7ed972d introduced a change but did
break our native kqueue implementation

Modifications:

Correctly use values of KQueueIoOps when calling evSet

Result:

KQueue tests pass again
  • Loading branch information
normanmaurer committed May 16, 2024
1 parent 59a7096 commit 1102847
Showing 1 changed file with 5 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -385,10 +385,13 @@ public long submit(IoOps ops) {
if (!isValid()) {
return -1;
}
short filter = kQueueIoOps.filter();
short flags = kQueueIoOps.flags();
int fflags = kQueueIoOps.fflags();
if (eventLoop.inEventLoop()) {
evSet(event.filter(), event.flags(), event.fflags());
evSet(filter, flags, fflags);
} else {
eventLoop.execute(() -> evSet(event.filter(), event.flags(), event.fflags()));
eventLoop.execute(() -> evSet(filter, flags, fflags));
}
return 0;
}
Expand Down

0 comments on commit 1102847

Please sign in to comment.