Yarn: ResourceReloadListener#reload -> createReloadTask o algo así

Creado en 22 oct. 2019 · 3Comentarios · Fuente: FabricMC/yarn

No se supone que ResourceReloadListener#reload realice ninguna recarga por sí solo. En cambio, su objetivo es devolver un CompletableFuture que, cuando se evalúa, se supone que debe enviar las tareas de recarga reales a varios Executor s.

Si intenta engañar al sistema poniendo el código directamente en reload y devolviendo un ficticio CompletableFuture que no hace nada, el juego se cuelga al recargar, porque el futuro devuelto por Synchronizer#whenPrepared nunca llega llamado.

Nombrarlo algo así como createReloadTask parece una forma de desalentar esta ruta y aclarar que no se debe recargar directamente en este método.

discussion enhancement refactor

Fuente

quat1024

👍1

Comentario más útil

En mi opinión, ResourceReloadListener debe cambiarse de nombre a ResourceReloader , ya que son ellos los que realmente realizan la recarga (en comparación con ResourceReloadHandler ). El nombre reload está bien en mi opinión, o podemos considerar un esquema de nombres coherente para todos los nombres relacionados con CompletableFuture .

liach en 22 oct. 2019

👍2 ❤1

Todos 3 comentarios

liach en 22 oct. 2019

👍2 ❤1

Después de un vistazo, ya tenemos un recargador de recursos que monitorea la recarga.

Ahora, estoy un poco confundido sobre cómo debería llamarse esto, o si debería renombrarse o no.

liach en 10 mar. 2021

Hmm, tal vez ResourceReloader -> SimpleResourceReload , ResourceReloadListener -> ResourceReloader . Además, asigne un nombre ResourceReloadMonitor a ResourceReload y asigne el nombre correspondiente a las implementaciones. ResourceReload representa una sola recarga y se descarta inmediatamente después, por lo que este nombre realmente suena bien.