Event with same priority are fired in reverse order

[Anahkiasen]

Consider the following code:

$emitter = new Emitter();

$emitter->addListener('foobar', function () {
   echo 'a';
}, 50);

$emitter->addListener('foobar', function () {
    echo 'b';
}, 0);

$emitter->addListener('foobar', function () {
    echo 'c';
}, 0);

$emitter->addListener('foobar', function () {
   echo  'd';
}, -50);

$emitter->emit('foobar');

Now, as C was registered after B, I would expect it to fire after B too, but currently instead of abcd I get acbd. Is this expected behaviour? Is there any way to avoid this?

[dstockto]

I've opened a PR #45 which adds a new sort function that is stable. The emitter is using usort which is not stable so the order of elements that are deemed equivalent by the comparison function is not guaranteed to remain the same as in the original array. PR #45 should fix that and you'll get abcd.

[frankdejonge]

Closing, explanation is here: #45

[frankdejonge]

I've come back to this and ensured events are now emitted in order while respecting priority. Consider this fixed.

[Anahkiasen]

Nice, thanks!