Status: New
Owner: ----

New issue 3529 by [email protected]: Array stuck in smi mode.
http://code.google.com/p/v8/issues/detail?id=3529

The issue first surfaced in this Chrome-specific bug report: https://github.com/mbostock/topojson/issues/192

The issue occurred in topojson.presimplify. This operates on "arcs"; each arc is an array of points. Each point is a two-element array. These points are transformed and delta-encoded as integers initially. topojson.presimplify decodes these points to "absolute" coordinates, and then turns each point into a three-element array; the third element being a special "effective area" (float).

Further investigation revealed that the third elements for these arrays appeared to be a smi, perhaps because V8 thinks the array is still entirely smi even though a floating point element has been added.

This simple workaround fixed the issue by creating a new array instance prior to adding the third floating point element: https://github.com/mbostock/topojson/pull/193

It seems like the initial delta encoding makes V8 correctly treat each point as being smi, but the subsequent transformations (both absolute transformation and addition of area element) mean the points should no longer be smi-only.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to