corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Rewrite and cleanup GETELEM and SETELEM (455748, r=brendan).

Browse Source
This commit is contained in:
Andreas Gal
2008-09-25 11:31:40 -07:00
parent c5e52bc7d5
commit df4e66e7b4
5 changed files with 135 additions and 118 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
js/src/jsbuiltins.cpp
View File

@@ -513,33 +513,12 @@ js_Any_setprop(JSContext* cx, JSObject* obj, JSString* idstr, jsval v)
}
jsval FASTCALL
js_Any_getelem(JSContext* cx, JSObject* obj, jsdouble index)
js_Any_getelem(JSContext* cx, JSObject* obj, jsint index)
{
jsval v;
jsid id;
if (!js_ValueToStringId(cx, DOUBLE_TO_JSVAL(&index), &id))
if (index < 0)
return JSVAL_ERROR_COOKIE;
if (!OBJ_GET_PROPERTY(cx, obj, id, &v))
return JSVAL_ERROR_COOKIE;
return v;
}
JSBool FASTCALL
js_Any_setelem(JSContext* cx, JSObject* obj, jsdouble index, jsval v)
{
jsid id;
if (!js_ValueToStringId(cx, DOUBLE_TO_JSVAL(&index), &id))
return JS_FALSE;
return OBJ_SET_PROPERTY(cx, obj, id, &v);
}
jsval FASTCALL
js_Any_getelem_int(JSContext* cx, JSObject* obj, jsuint index)
{
jsval v;
jsid id;
if (!js_IndexToId(cx, index, &id))
return JSVAL_ERROR_COOKIE;
if (!OBJ_GET_PROPERTY(cx, obj, id, &v))
@@ -548,9 +527,11 @@ js_Any_getelem_int(JSContext* cx, JSObject* obj, jsuint index)
}
JSBool FASTCALL
js_Any_setelem_int(JSContext* cx, JSObject* obj, jsuint index, jsval v)
js_Any_setelem(JSContext* cx, JSObject* obj, jsint index, jsval v)
{
jsid id;
if (index < 0)
return JSVAL_ERROR_COOKIE;
if (!js_IndexToId(cx, index, &id))
return JS_FALSE;
return OBJ_SET_PROPERTY(cx, obj, id, &v);