bevyengine / bevyengine/bevy

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

Abierto Apto para principiantes
#23,653 1 comentario 0 reacciones 0 asignados Ver en GitHub
A-Dev-Tools A-ECS C-Bug D-Straightforward S-Ready-For-Implementation X-Contentious
Lenguaje dominante
Rust
Estrellas
48.2k
Forks
4.8k
Merge medio
3 d 16 h
PR fusionados (30 d)
171

Descripción

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.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza en la definición del trait System e inspecciona el método refresh_hotpatch condicionado por una feature. Comprueba cómo se ven afectados los traits System implementados manualmente cuando el hotpatching está habilitado; se considera terminado cuando las implementaciones que no necesitan hotpatching pueden compilar sin definir este método y las comprobaciones o rutas de compilación relevantes pasan correctamente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
rust
Área
game-dev
Tipo de issue
Nueva funcionalidad
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Tranquilo
Claridad
Bien especificado
Aptitud para principiantes
68/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.