  joywayerandClaude Opus 4.8
|
2a7658e0e3
|
fix(hot-path): 回退 query_circle 复用缓冲优化——存在重入覆写 bug
上个提交 35c1742 让 query_circle 复用成员 _query_result 缓冲，实测有重入缺陷：
GDScript 的 PackedArray 返回值是活引用别名（非写时复制副本），当调用方遍历返回
的 hits 期间触发嵌套 query_circle（命中→SpellEvaluator.execute_sub 的区域法术），
嵌套调用的 clear()+append 会就地覆写外层正在遍历的同一缓冲，导致碰撞遍历读到错误
实体 ID。编辑器内已复现：外层期望 [10,20,30]，重入后实得 [10,88]。
原优化仅省约 0.064ms/帧（帧预算 ~0.4%），不值得为其做跨 4 处调用点的 out 参数
+ 各自持久缓冲的重入安全改造，故回退为每次新建数组。
BulletManager 无冷数据命中快路径无重入问题，保留。
并加注释说明此处禁止复用缓冲的原因。
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
2026-07-23 16:35:31 +08:00 |
|
|
joywayerandClaude Opus 4.8
|
35c1742b4c
|
perf(hot-path): SpatialGrid.query_circle 复用结果缓冲 + BulletManager 无冷数据命中快路径
纯 GDScript 层零成本优化（不引入 C#/C++，遵循「先测量后优化」）：
- spatial_grid.gd: query_circle 改用持久成员 _query_result（clear()+append 就地填充后返回），
避免每次调用（每帧 1500+ 次）新建 PackedInt32Array。经引擎实测 ~11% 提速、逐点校验
2000 次查询结果与原实现完全一致；CoW 语义保证嵌套查询（命中→SpellEvaluator 区域查询）
不破坏调用方正在遍历的旧结果。
- bullet_manager.gd: _check_collision 命中处理为无冷数据的普通子弹加快路径，
避免 _bullet_contexts.get(bullet_idx, {}) 每次命中都分配空字典默认值 + 5 次字典查。
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
2026-07-23 16:13:33 +08:00 |
|