JS_LookupElement Broken in 1.8RC1?

[dave]

I recently started compiling my project against the new release of
spidermonkey (1.8 release candidate 1) and noticed that one of my
functions stopped behaving properly. I looked into it and it seems as
though JS_LookupElement, when run on an array, returns every element
of the array as being a boolean value.

Here's a short snippet of my code:

if (JS_FALSE == JS_GetArrayLength(cx, obj, &a_len)) {
EDJS_ERR(cx, EDJSERR_ARRAY_LEN);
goto error;
}
printf("array: %s\n", JS_GetStringBytes(JS_ValueToString(cx,
OBJECT_TO_JSVAL(obj))));
//prints: "array: a,b,c"

for (i = 0; i < a_len; i++) {
if (JS_FALSE == JS_LookupElement(cx, obj, i, &a_val)) {
goto error;
}

if (JS_TRUE == JSVAL_IS_BOOLEAN(a_val))
printf("value %d is boolean\n", i);
//prints boolean 3 times
//...
}

Am I crazy?

[Igor Bukanov]

This is a regression in JS1.8RC1 which I filed as
https://bugzilla.mozilla.org/show_bug.cgi?id=486106 . Thanks for
reporting this!

Igor

2009/3/31 dave <man...@gmail.com>:

> _______________________________________________
> dev-tech-js-engine mailing list
> dev-tech-...@lists.mozilla.org
> https://lists.mozilla.org/listinfo/dev-tech-js-engine
>

[dave]

On Mar 31, 9:35 am, Igor Bukanov <i...@mir2.org> wrote:
> This is a regression in JS1.8RC1 which I filed ashttps://bugzilla.mozilla.org/show_bug.cgi?id=486106. Thanks for
> reporting this!
>
> Igor
>

No, thank you!

[Igor Bukanov]

Another note is that in your case you should use JS_GetElement API to
get the elements of the array. JS_LookupElement will give you a wrong
result when the corresponding element in the array has a getter as the
API does not run the getter. Its purpose is mostly intended to query
if the corresponding element exists rather then querying its value.

Igor

2009/3/31 dave <man...@gmail.com>:

[dave]

On Mar 31, 12:40 pm, Igor Bukanov <i...@mir2.org> wrote:
> Another note is that in your case you should use JS_GetElement API to
> get the elements of the array. JS_LookupElement will give you a wrong
> result when the corresponding element in the array has a getter as the
> API does not run the getter. Its purpose is mostly intended to query
> if the corresponding element exists rather then querying its value.
>
> Igor

Thanks for the tip. I will implement the change.