bevyengine / bevyengine/bevy

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

Ouverte Adaptée aux débutants
#23,653 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
A-Dev-Tools A-ECS C-Bug D-Straightforward S-Ready-For-Implementation X-Contentious
Langage dominant
Rust
Étoiles
48.2k
Forks
4.8k
Merge moyen
3 j 22 h
PR mergées (30 j)
161

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par la définition du trait System et examinez la méthode refresh_hotpatch conditionnée par une feature. Vérifiez comment les traits System implémentés manuellement sont affectés lorsque le hotpatching est activé ; le travail est terminé lorsque les implémentations qui n’ont pas besoin de hotpatching peuvent être compilées sans définir cette méthode, et que les vérifications ou chemins de compilation concernés passent.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
rust
Domaine
game-dev
Type d'issue
Fonctionnalité
Difficulté
2/5
Temps estimé
1-3 heures
Activité
Calme
Clarté
Clairement spécifiée
Accessibilité débutants
68/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.