diff --git a/frameworks/ets/ani/include/sts_common.h b/frameworks/ets/ani/include/sts_common.h index 8fc00d4e520015cd791cee58b18bd28400fa8e37..1129ade5a425aa2cd4ed153a493ad8c7415628c6 100644 --- a/frameworks/ets/ani/include/sts_common.h +++ b/frameworks/ets/ani/include/sts_common.h @@ -46,7 +46,7 @@ ani_object ConvertArrayDoubleToAniObj(ani_env *env, const std::vector &values); // property -bool SetPropertyOptionalByBoolean(ani_env *env, ani_object object, const char *name, bool value); -bool SetPropertyOptionalByDouble(ani_env *env, ani_object object, const char *name, double value); -bool SetPropertyOptionalByString(ani_env *env, ani_object object, const char *name, const std::string value); -bool SetPropertyOptionalByInt(ani_env *env, ani_object object, const char *name, int32_t value); -bool SetPropertyByRef(ani_env *env, ani_object object, const char *name, ani_ref value); +bool SetPropertyOptionalByBoolean(ani_env *env, ani_object &object, const char *name, bool value); +bool SetPropertyOptionalByDouble(ani_env *env, ani_object &object, const char *name, double value); +bool SetPropertyOptionalByString(ani_env *env, ani_object &object, const char *name, const std::string value); +bool SetPropertyOptionalByInt(ani_env *env, ani_object &object, const char *name, int32_t value); +bool SetPropertyByRef(ani_env *env, ani_object &object, const char *name, ani_ref value); bool CreateClassObjByClassName(ani_env *env, const char *className, ani_class &cls, ani_object &outAniObj); @@ -87,7 +87,7 @@ inline ani_boolean BoolToAniBoolean(bool value) } template -static bool CallSetter(ani_env* env, ani_class cls, ani_object object, const char* propertyName, valueType value) +static bool CallSetter(ani_env* env, ani_class cls, ani_object &object, const char* propertyName, valueType value) { if (env == nullptr || cls == nullptr || object == nullptr) { return false; @@ -115,7 +115,7 @@ static bool CallSetter(ani_env* env, ani_class cls, ani_object object, const cha return true; } -[[maybe_unused]]static bool CallSetterNull(ani_env* env, ani_class cls, ani_object object, const char* propertyName) +[[maybe_unused]]static bool CallSetterNull(ani_env* env, ani_class cls, ani_object &object, const char* propertyName) { ani_ref nullRef = nullptr; ani_status status = env->GetNull(&nullRef); diff --git a/frameworks/ets/ani/src/sts_common.cpp b/frameworks/ets/ani/src/sts_common.cpp index e58fc37ac75dca986d30d61455f0ae04c6c3b262..f0002ca403414ea5350dd91519757c827d11c59a 100644 --- a/frameworks/ets/ani/src/sts_common.cpp +++ b/frameworks/ets/ani/src/sts_common.cpp @@ -138,7 +138,7 @@ ani_status GetPropertyString(ani_env *env, ani_object obj, const char *name, } ani_status GetPropertyBool(ani_env *env, ani_object obj, const char *name, - ani_boolean isUndefined, bool outvalue) + ani_boolean &isUndefined, bool &outvalue) { ANS_LOGD("GetPropertyBool start"); if (env == nullptr || obj == nullptr || name == nullptr) { @@ -559,7 +559,7 @@ ani_object ConvertArrayDoubleToAniObj(ani_env *env, const std::vector &values) { if (env == nullptr || cls == nullptr || object == nullptr || fieldName.empty()) { @@ -658,7 +658,7 @@ ani_object CreateInt(ani_env *env, int32_t value) return outObj; } -bool SetPropertyOptionalByBoolean(ani_env *env, ani_object object, const char *name, bool value) +bool SetPropertyOptionalByBoolean(ani_env *env, ani_object &object, const char *name, bool value) { ANS_LOGD("enter SetPropertyOptionalByBoolean"); if (env == nullptr || object == nullptr || name == nullptr) { @@ -673,7 +673,7 @@ bool SetPropertyOptionalByBoolean(ani_env *env, ani_object object, const char *n return SetPropertyByRef(env, object, name, boolObj); } -bool SetPropertyOptionalByDouble(ani_env *env, ani_object object, const char *name, double value) +bool SetPropertyOptionalByDouble(ani_env *env, ani_object &object, const char *name, double value) { ANS_LOGD("enter SetPropertyOptionalByDouble"); if (env == nullptr || object == nullptr || name == nullptr) { @@ -688,7 +688,7 @@ bool SetPropertyOptionalByDouble(ani_env *env, ani_object object, const char *na return SetPropertyByRef(env, object, name, doubleObj); } -bool SetPropertyOptionalByString(ani_env *env, ani_object object, const char *name, const std::string value) +bool SetPropertyOptionalByString(ani_env *env, ani_object &object, const char *name, const std::string value) { ANS_LOGD("enter SetPropertyOptionalByString"); if (env == nullptr || object == nullptr || name == nullptr) { @@ -708,7 +708,7 @@ bool SetPropertyOptionalByString(ani_env *env, ani_object object, const char *na return SetPropertyByRef(env, object, name, static_cast(stringObj)); } -bool SetPropertyOptionalByInt(ani_env *env, ani_object object, const char *name, int32_t value) +bool SetPropertyOptionalByInt(ani_env *env, ani_object &object, const char *name, int32_t value) { ANS_LOGD("enter SetPropertyOptionalByInt"); if (env == nullptr || object == nullptr || name == nullptr) { @@ -723,7 +723,7 @@ bool SetPropertyOptionalByInt(ani_env *env, ani_object object, const char *name, return SetPropertyByRef(env, object, name, IntObj); } -bool SetPropertyByRef(ani_env *env, ani_object object, const char *name, ani_ref value) +bool SetPropertyByRef(ani_env *env, ani_object &object, const char *name, ani_ref value) { ANS_LOGD("enter SetPropertyByRef"); ani_status status = ANI_OK;