feature-gated `System::refresh_hotpatch` with no default impl is inconvenient for crate authors
- 主要言語
- Rust
- スター
- 48.2k
- フォーク
- 4.8k
- 平均マージ
- 3日 22時間
- マージ済み PR(30日)
- 161
説明
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