Skip to content

events: remove internal spliceOne()

Rodrigo Muino Tomonari requested to merge github/fork/Trott/rm-spliceOne into master

spliceOne() was added as a faster alternative to Array#splice in the particular use case in events.js. However, current master shows no difference in performance with or without spliceOne():

                                  improvement confidence   p.value
 events/ee-add-remove.js n=250000      0.14 %            0.7573913
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

events

Merge request reports