bevyengine / bevyengine/bevy

feature-gated `System::refresh_hotpatch` with no default impl is inconvenient for crate authors

未关闭 适合新手
#23,653 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
A-Dev-Tools A-ECS C-Bug D-Straightforward S-Ready-For-Implementation X-Contentious
主要语言
Rust
星标
48.2k
派生
4.8k
平均合并
3 天 15 小时
30 天内合并 PR
168

描述

When the user uses the `hotpatching` feature, the `System` trait has one additional method, [`refresh_hotpatch`](https://dev-docs.bevy.org/bevy/ecs/prelude/trait.System.html#tymethod.refresh_hotpatch).

However, this method has no default impl. This means that other 3rd party crates the user relies on need to offer that feature too if these manually implement `System`, even if hotpatching makes no sense for them. If they don't do this, the user's project wont compile.

A solution could be to add an empty body as the default impl.

贡献指南

打开贡献指南

调研方向

从 System trait 的定义开始,检查由 feature 控制的 refresh_hotpatch 方法。检查启用 hotpatching 时手动实现的 System trait 会受到怎样的影响;当不需要 hotpatching 的实现无需定义此方法即可编译,并且相关检查或编译路径通过时,即视为完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
rust
领域
game-dev
Issue 类型
功能
难度
2/5
预计耗时
1-3 小时
活跃度
冷清
描述清晰度
描述清楚
新手友好度
68/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。