{"action":"created","issue":{"url":"https://api.github.com/repos/nodejs/io.js/issues/1732","labels_url":"https://api.github.com/repos/nodejs/io.js/issues/1732/labels{/name}","comments_url":"https://api.github.com/repos/nodejs/io.js/issues/1732/comments","events_url":"https://api.github.com/repos/nodejs/io.js/issues/1732/events","html_url":"https://github.com/nodejs/io.js/pull/1732","id":77954804,"number":1732,"title":"lib: improve spliceOne() for large arrays","user":{"login":"mscdex","id":54666,"avatar_url":"https://avatars.githubusercontent.com/u/54666?v=3","gravatar_id":"","url":"https://api.github.com/users/mscdex","html_url":"https://github.com/mscdex","followers_url":"https://api.github.com/users/mscdex/followers","following_url":"https://api.github.com/users/mscdex/following{/other_user}","gists_url":"https://api.github.com/users/mscdex/gists{/gist_id}","starred_url":"https://api.github.com/users/mscdex/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mscdex/subscriptions","organizations_url":"https://api.github.com/users/mscdex/orgs","repos_url":"https://api.github.com/users/mscdex/repos","events_url":"https://api.github.com/users/mscdex/events{/privacy}","received_events_url":"https://api.github.com/users/mscdex/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/nodejs/io.js/labels/events","name":"events","color":"bfdadc"},{"url":"https://api.github.com/repos/nodejs/io.js/labels/url","name":"url","color":"bfe5bf"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-05-19T06:24:34Z","updated_at":"2015-05-19T21:39:59Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nodejs/io.js/pulls/1732","html_url":"https://github.com/nodejs/io.js/pull/1732","diff_url":"https://github.com/nodejs/io.js/pull/1732.diff","patch_url":"https://github.com/nodejs/io.js/pull/1732.patch"},"body":"`array.splice()` works faster for arrays of size ~125 or larger, independent of the index (except for `array.length - 1` evidently).\r\n\r\nUsually event listener arrays aren't that large, but I figured why not, especially since the changes do not negatively impact the (typical) < 125 element case at all."},"comment":{"url":"https://api.github.com/repos/nodejs/io.js/issues/comments/103675099","html_url":"https://github.com/nodejs/io.js/pull/1732#issuecomment-103675099","issue_url":"https://api.github.com/repos/nodejs/io.js/issues/1732","id":103675099,"user":{"login":"mscdex","id":54666,"avatar_url":"https://avatars.githubusercontent.com/u/54666?v=3","gravatar_id":"","url":"https://api.github.com/users/mscdex","html_url":"https://github.com/mscdex","followers_url":"https://api.github.com/users/mscdex/followers","following_url":"https://api.github.com/users/mscdex/following{/other_user}","gists_url":"https://api.github.com/users/mscdex/gists{/gist_id}","starred_url":"https://api.github.com/users/mscdex/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mscdex/subscriptions","organizations_url":"https://api.github.com/users/mscdex/orgs","repos_url":"https://api.github.com/users/mscdex/repos","events_url":"https://api.github.com/users/mscdex/events{/privacy}","received_events_url":"https://api.github.com/users/mscdex/received_events","type":"User","site_admin":false},"created_at":"2015-05-19T21:39:59Z","updated_at":"2015-05-19T21:39:59Z","body":"@trevnorris Did you also try with a much smaller array size? Like 10 or 50 or something less than 125-ish?"}}