bevyengine / bevyengine/bevy

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

オープン 初心者向け
#23,653 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
A-Dev-Tools A-ECS C-Bug D-Straightforward S-Ready-For-Implementation X-Contentious
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。