已合并
reflectOwnkeys异常未按照期望抛出 #13522
reflectOwnkeys异常未按照期望抛出 #13522
已合并
yang-yunfei32创建于 2025年11月6日
yang-yunfei32
yang-yunfei32
2025年11月6日

关联的Issue

reflectOwnkeys异常未按照期望抛出

修改原因(目的、解决的问题等,例如:修复xx场景崩溃问题)

reflectOwnkeys返回数组不包含不可配置属性的异常未按照期望抛出
原因是在检测前判断了不可配置属性leng是否为0,但是使用错了变量名导致程序提前结束,未走到检测抛异常的分支

修改描述(做了什么,变更了什么,例如:xx函数入口增加判空)

修改错误变量

自测试项(测试结果截图,直接贴到每一个测试项底下)

jit-workload 本地执行(compiler/jit/isa/stub/ic/对象布局等修改必须执行)

interpreter-js_perf 性能测试(性能优化/性能敏感场景:IR化/ic/hclass/对象布局/isa/builtins等修改必须执行)

aot-js_perf 性能测试(性能优化/性能敏感场景:IR化/ic/hclass/对象布局/isa/builtins等修改必须执行)

interpreter-workload性能测试(性能优化/性能敏感场景:IR化/ic/hclass/对象布局/isa/builtins等修改必须执行)

aot-workload性能测试(性能优化/性能敏感场景:IR化/compiler/aot/isa/stub/ic/对象布局等修改必须执行)

非StableArray不能采用ElementKind读取元素(修改解释器相关IR代码需要自检)

在IR时添加fastpath时,需要检查是否有分支会改变非字典序模式或者修改数组长度,此时应该走slowpath(修改解释器相关IR代码需要自检)

IR化后的逻辑需要与c++实现一致,不一致的地方需要有分支用例看护(修改解释器相关IR代码需要自检)

c++代码中检测或抛出异常的地方,IR需要同样实现(修改解释器相关IR代码需要自检)

IR中使用默认入参是HeapObject对象的接口时,需要在上下文中保证了参数的正确性(修改解释器相关IR代码需要自检)

修改c解释器,需要同时检查汇编解释器与AOT是否需要修改

是否已执行L0用例

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 yang-yunfei32 的贡献)
yang-yunfei32yang-yunfei32
2025年11月6日 关联了issue:reflectOwnkeys异常未按照期望抛出
openharmony_ciopenharmony_ci成员
2025年11月6日 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
2025年11月6日 评论:

感谢提交 Pull Requests!如果您提交的PR已经开发完毕,请评论 "start build" 触发门禁,更多交互操作,请访问OpenHarmony社区支持命令清单。如果需要调整订阅PR、Issue的变更状态,请访问订阅链接


Thanks for submitting the pull request. If your Pull Request has already been developed, you can leave a "start build" comment to trigger the gated system. For more commands, please visit OpenHarmony Command List. If you need to change the subscription of a Pull Request or Issue, please visit the link.

likedislike
openharmony_ciopenharmony_ci成员
2025年11月6日 添加了label:dco检查成功
xwcai98
xwcai982025年11月6日进行代码检视1
ecmascript/js_proxy.cpp
@@ -822,3 +822,3 @@
822822 // 19.If extensibleTarget is true and targetNonconfigurableKeys is empty, then
823823 // a.Return trapResult.
824- if (extensibleTarget && (cfigLength == 0)) {
824+ if (extensibleTarget && (noCfigLength == 0)) {
xwcai98
xwcai982025年11月6日评论:

已检视无问题

likedislike
此处折叠了93条消息 查看更多
ggwatcher成员
2025年11月13日 评论:

Build for commit 65c188ff40b65ec19cb342299ba68806c72cb1cb: pass

likedislike
klooerklooer成员
2025年11月13日 通过审查
yang-yunfei32
yang-yunfei32
2025年11月13日 评论:

submit

likedislike
openharmony_ciopenharmony_ci成员
2025年11月13日 合入了pull request,合并节点 SHA:3b5bc9763e11cddd3d5f48e976fced060a76def5
openharmony_ciopenharmony_ci成员
2025年11月13日 删除了label:waiting_for_review