{"action":"created","issue":{"url":"https://api.github.com/repos/HaxeFoundation/haxe/issues/4704","labels_url":"https://api.github.com/repos/HaxeFoundation/haxe/issues/4704/labels{/name}","comments_url":"https://api.github.com/repos/HaxeFoundation/haxe/issues/4704/comments","events_url":"https://api.github.com/repos/HaxeFoundation/haxe/issues/4704/events","html_url":"https://github.com/HaxeFoundation/haxe/pull/4704","id":120803886,"number":4704,"title":"use ListNode class instead of Array<Dynamic> as container objects for List elements","user":{"login":"nadako","id":49749,"avatar_url":"https://avatars.githubusercontent.com/u/49749?v=3","gravatar_id":"","url":"https://api.github.com/users/nadako","html_url":"https://github.com/nadako","followers_url":"https://api.github.com/users/nadako/followers","following_url":"https://api.github.com/users/nadako/following{/other_user}","gists_url":"https://api.github.com/users/nadako/gists{/gist_id}","starred_url":"https://api.github.com/users/nadako/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nadako/subscriptions","organizations_url":"https://api.github.com/users/nadako/orgs","repos_url":"https://api.github.com/users/nadako/repos","events_url":"https://api.github.com/users/nadako/events{/privacy}","received_events_url":"https://api.github.com/users/nadako/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-07T15:55:35Z","updated_at":"2015-12-07T16:11:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/HaxeFoundation/haxe/pulls/4704","html_url":"https://github.com/HaxeFoundation/haxe/pull/4704","diff_url":"https://github.com/HaxeFoundation/haxe/pull/4704.diff","patch_url":"https://github.com/HaxeFoundation/haxe/pull/4704.patch"},"body":"Subj. I'm not sure why it was implemented using `Array<Dynamic>` in the first place. I haven't really done any benchmarks, but I'm pretty sure that at least on static targets, it's much better to use a simple node class instead of Array (considering its internal bound-checking and buffer allocation). Actually, I wouldn't be surprised if it performs better on modern JS engines like V8 as well (due to hidden class optimizations)."},"comment":{"url":"https://api.github.com/repos/HaxeFoundation/haxe/issues/comments/162572099","html_url":"https://github.com/HaxeFoundation/haxe/pull/4704#issuecomment-162572099","issue_url":"https://api.github.com/repos/HaxeFoundation/haxe/issues/4704","id":162572099,"user":{"login":"ncannasse","id":1022912,"avatar_url":"https://avatars.githubusercontent.com/u/1022912?v=3","gravatar_id":"","url":"https://api.github.com/users/ncannasse","html_url":"https://github.com/ncannasse","followers_url":"https://api.github.com/users/ncannasse/followers","following_url":"https://api.github.com/users/ncannasse/following{/other_user}","gists_url":"https://api.github.com/users/ncannasse/gists{/gist_id}","starred_url":"https://api.github.com/users/ncannasse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ncannasse/subscriptions","organizations_url":"https://api.github.com/users/ncannasse/orgs","repos_url":"https://api.github.com/users/ncannasse/repos","events_url":"https://api.github.com/users/ncannasse/events{/privacy}","received_events_url":"https://api.github.com/users/ncannasse/received_events","type":"User","site_admin":false},"created_at":"2015-12-07T16:11:55Z","updated_at":"2015-12-07T16:11:55Z","body":"I think it was originally developed for Neko: could you keep the neko.NativeArray implementation for neko?"}}