[JSC] Drop MapBase
Created attachment 310776 [details] Patch
Comment on attachment 310776 [details] Patch Very nice. Any idea why the indirection was there in the first place? Was this from a time when the HashMap wasn't a JSCell?
Comment on attachment 310776 [details] Patch Discussed with Saam at IRC (private message). We guess that this is accidentally introduced.
Comment on attachment 310776 [details] Patch Clearing flags on attachment: 310776 Committed r217192: <http://trac.webkit.org/changeset/217192>
All reviewed patches have been landed. Closing bug.
(In reply to Sam Weinig from comment #2) > Comment on attachment 310776 [details] > Patch > > Very nice. Any idea why the indirection was there in the first place? Was > this from a time when the HashMap wasn't a JSCell? There isn’t a goood reason. When I rewrote our map/set implementation last year I just left this in.