Could be useful in places where we have to write bindings.

NPOTB, not currently in use.

Comment on attachment 509339 [details] [diff] [review] patch Land it.

Comment on attachment 509339 [details] [diff] [review] patch Does it not seem confusing to have the old JS_ValueToInt32, which rounds to nearest and fails if the double is out of range, the newer but still ancient JS_ValueToECMA{I,Ui}nt32 pair, and now these JS_DoubleTo{I,Ui}nt32 which match ECMA-262 and do not match JS_ValueToInt32? We could rotate names to give the old, pre-ECMA JS_ValueToInt32 a long and better name, giving the ECMA names the short and "ECMA"-free names. We could mangle the names of the JS_DoubleTo{I,Ui}nt32 APIs to contain "ECMA". But just adding more mixed naming does not seem good. Right? /be

Hyperbole for sure, but rotating names feels somewhat like rearranging deck chairs to me. The names certainly aren't pretty, but name stability seems to trump aesthetics to me, as far as external APIs go. (Although fixing bugs/enabling better features itself trumps stability, cf. operation callback breakages and so on.) If we want to regularize names, it seems most sensible to do that as part of JSAPI2.

Agree with comment 5, which leaves adding "ECMA" to the parallel DoubleTo names. Agreed? /be

This is blocking some of my work. Any chance this could be landed soonish?

What about comment 4 through comment 6?

Pre-landing I'm willing to care about names (or not care, as my previous comment seems to say). Post-landing, whatever got landed is good enough, since naming consistency is basically impossible until JSAPI2.

Ah ****. I overlooked the post review comments. I am happy with any naming. Waldo, guide me. What should I name how? I will do a little follow-up before I merge to m-c.

Whoops, sorry, this wasn't merged. Reopened.

Yes it did? http://hg.mozilla.org/mozilla-central/file/1a89509e25e4/js/src/jsapi.cpp#l523

Documented: https://developer.mozilla.org/en/SpiderMonkey/JSAPI_Reference/JS_DoubleToInt32 Also mentioned on Firefox 5 for developers.