Don't expose JS_{Get,Set}PropertyInternal in the public API

This commit is contained in:
Saúl Ibarra Corretgé 2024-03-23 21:52:32 +01:00
parent 3b50de4848
commit b8341ecafa
2 changed files with 23 additions and 27 deletions

View file

@ -7124,9 +7124,9 @@ static int JS_AutoInitProperty(JSContext *ctx, JSObject *p, JSAtom prop,
return 0; return 0;
} }
JSValue JS_GetPropertyInternal2(JSContext *ctx, JSValue obj, static JSValue JS_GetPropertyInternal2(JSContext *ctx, JSValue obj,
JSAtom prop, JSValue this_obj, JSAtom prop, JSValue this_obj,
JSInlineCache* ic, BOOL throw_ref_error) JSInlineCache* ic, BOOL throw_ref_error)
{ {
JSObject *p; JSObject *p;
JSProperty *pr; JSProperty *pr;
@ -7276,13 +7276,17 @@ JSValue JS_GetPropertyInternal2(JSContext *ctx, JSValue obj,
} }
} }
JSValue JS_GetPropertyInternal(JSContext *ctx, JSValue obj, static JSValue JS_GetPropertyInternal(JSContext *ctx, JSValue obj,
JSAtom prop, JSValue this_obj, JSAtom prop, JSValue this_obj,
BOOL throw_ref_error) BOOL throw_ref_error)
{ {
return JS_GetPropertyInternal2(ctx, obj, prop, this_obj, NULL, throw_ref_error); return JS_GetPropertyInternal2(ctx, obj, prop, this_obj, NULL, throw_ref_error);
} }
JSValue JS_GetProperty(JSContext *ctx, JSValue this_obj, JSAtom prop)
{
return JS_GetPropertyInternal2(ctx, this_obj, prop, this_obj, NULL, FALSE);
}
static JSValue JS_GetPropertyInternalWithIC(JSContext *ctx, JSValue obj, static JSValue JS_GetPropertyInternalWithIC(JSContext *ctx, JSValue obj,
JSAtom prop, JSValue this_obj, JSAtom prop, JSValue this_obj,
@ -8412,9 +8416,9 @@ static void js_free_desc(JSContext *ctx, JSPropertyDescriptor *desc)
the new property is not added and an error is raised. the new property is not added and an error is raised.
'obj' must be an object when obj != this_obj. 'obj' must be an object when obj != this_obj.
*/ */
int JS_SetPropertyInternal2(JSContext *ctx, JSValue obj, static int JS_SetPropertyInternal2(JSContext *ctx, JSValue obj,
JSAtom prop, JSValue val, JSValue this_obj, JSAtom prop, JSValue val, JSValue this_obj,
int flags, JSInlineCache *ic) int flags, JSInlineCache *ic)
{ {
JSObject *p, *p1; JSObject *p, *p1;
JSShapeProperty *prs; JSShapeProperty *prs;
@ -8665,13 +8669,18 @@ fail:
return -1; return -1;
} }
int JS_SetPropertyInternal(JSContext *ctx, JSValue this_obj, static int JS_SetPropertyInternal(JSContext *ctx, JSValue this_obj,
JSAtom prop, JSValue val, int flags) JSAtom prop, JSValue val, int flags)
{ {
return JS_SetPropertyInternal2(ctx, this_obj, prop, val, this_obj, return JS_SetPropertyInternal2(ctx, this_obj, prop, val, this_obj,
flags, NULL); flags, NULL);
} }
int JS_SetProperty(JSContext *ctx, JSValue this_obj, JSAtom prop, JSValue val)
{
return JS_SetPropertyInternal2(ctx, this_obj, prop, val, this_obj, JS_PROP_THROW, NULL);
}
static int JS_SetPropertyInternalWithIC(JSContext *ctx, JSValue this_obj, static int JS_SetPropertyInternalWithIC(JSContext *ctx, JSValue this_obj,
JSAtom prop, JSValue val, int flags, JSAtom prop, JSValue val, int flags,
JSInlineCache *ic, int32_t offset) { JSInlineCache *ic, int32_t offset) {

View file

@ -656,27 +656,14 @@ JS_EXTERN int JS_IsArray(JSContext *ctx, JSValue val);
JS_EXTERN JSValue JS_NewDate(JSContext *ctx, double epoch_ms); JS_EXTERN JSValue JS_NewDate(JSContext *ctx, double epoch_ms);
JS_EXTERN JSValue JS_GetPropertyInternal(JSContext *ctx, JSValue obj, JS_EXTERN JSValue JS_GetProperty(JSContext *ctx, JSValue this_obj, JSAtom prop);
JSAtom prop, JSValue receiver,
JS_BOOL throw_ref_error);
static js_force_inline JSValue JS_GetProperty(JSContext *ctx, JSValue this_obj,
JSAtom prop)
{
return JS_GetPropertyInternal(ctx, this_obj, prop, this_obj, 0);
}
JS_EXTERN JSValue JS_GetPropertyStr(JSContext *ctx, JSValue this_obj, JS_EXTERN JSValue JS_GetPropertyStr(JSContext *ctx, JSValue this_obj,
const char *prop); const char *prop);
JS_EXTERN JSValue JS_GetPropertyUint32(JSContext *ctx, JSValue this_obj, JS_EXTERN JSValue JS_GetPropertyUint32(JSContext *ctx, JSValue this_obj,
uint32_t idx); uint32_t idx);
int JS_SetPropertyInternal(JSContext *ctx, JSValue this_obj, JS_EXTERN int JS_SetProperty(JSContext *ctx, JSValue this_obj,
JSAtom prop, JSValue val, JSAtom prop, JSValue val);
int flags);
static inline int JS_SetProperty(JSContext *ctx, JSValue this_obj,
JSAtom prop, JSValue val)
{
return JS_SetPropertyInternal(ctx, this_obj, prop, val, JS_PROP_THROW);
}
JS_EXTERN int JS_SetPropertyUint32(JSContext *ctx, JSValue this_obj, JS_EXTERN int JS_SetPropertyUint32(JSContext *ctx, JSValue this_obj,
uint32_t idx, JSValue val); uint32_t idx, JSValue val);
JS_EXTERN int JS_SetPropertyInt64(JSContext *ctx, JSValue this_obj, JS_EXTERN int JS_SetPropertyInt64(JSContext *ctx, JSValue this_obj,