Re: [PATCH 0/5] MR5048: jscript: ArrayBuffer and DataView implementations.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/jscript/arraybuf.c:
> - desc.mask  = PROPF_CONFIGURABLE | PROPF_ENUMERABLE;
> - desc.explicit_getter = desc.explicit_setter = TRUE;
> - desc.explicit_value = FALSE;
> - desc.setter = NULL;
> -
> - for(i = 0; i < ARRAY_SIZE(DataView_getters); i++) {
> -
>    hres = create_builtin_function(ctx, DataView_getters[i].get, NULL, NULL, PROPF_METHOD, NULL, &desc.getter);
> -
>    if(SUCCEEDED(hres)) {
> -
>        hres = jsdisp_define_property(&view->dispex, DataView_getters[i].name, &desc);
> -
>        jsdisp_release(desc.getter);
> -
>    }
> -
>    if(FAILED(hres)) {
> -
>        jsdisp_release(&view->dispex);
> -
>        return hres;
> -
>    }
> - }

Ideally it would use DataView_info for those properties. I guess you don't do that because builtins are currently not exposed as accessor properties. We could potentially change that. How unique ArrayBuffer properties are in this regard?