invokeCallbacks() instead of microtasks?

[vudaltsov]

Hi! I am studying the EventLoop and came across this:

event-loop/src/EventLoop/Internal/AbstractDriver.php

Lines 541 to 542 in 41f9c45

	// Invoke microtasks if we have some
	$this->invokeCallbacks();

Is it a mistake? Should $this->invokeMicrotasks(); be called instead?

[kelunik]

No, the callbacks need to be executed in a Fiber. That's what invoke callbacks does.

[vudaltsov]

Then perhaps the comment should be corrected?

[kelunik]

What do you suggest? Still seems correct to me. We could rather change the naming of invokeCallbacks / invokeMicrotasks, but I don't have a good suggestion there.