Yarn: ResourceReloadListener#reload -> createReloadTask atau semacamnya

Dibuat pada 22 Okt 2019 · 3Komentar · Sumber: FabricMC/yarn

ResourceReloadListener#reload tidak seharusnya melakukan reload sendiri. Sebaliknya, tujuannya adalah untuk mengembalikan CompletableFuture yang, ketika dievaluasi, seharusnya mengirimkan tugas reload aktual ke berbagai Executor s.

Jika Anda mencoba menipu sistem dengan memasukkan kode langsung ke reload dan mengembalikan beberapa dummy CompletableFuture yang tidak menghasilkan apa-apa, permainan macet saat memuat ulang, karena masa depan yang dikembalikan oleh Synchronizer#whenPrepared tidak akan pernah didapat ditelepon.

Memberi nama seperti createReloadTask sepertinya merupakan cara untuk mencegah jalur ini dan memperjelas bahwa tidak ada pemuatan ulang yang harus dilakukan secara langsung dalam metode ini.

discussion enhancement refactor

Sumber

quat1024

👍1

Komentar yang paling membantu

imo, ResourceReloadListener harus diganti namanya menjadi ResourceReloader karena merekalah yang benar-benar melakukan reload (bandingkan dengan ResourceReloadHandler ). Nama reload baik-baik saja, atau kita dapat mempertimbangkan skema penamaan yang konsisten untuk semua nama terkait CompletableFuture .

liach pada 22 Okt 2019

👍2 ❤1

Semua 3 komentar

liach pada 22 Okt 2019

👍2 ❤1

Setelah melihat, kami sudah memiliki resource reloader yang memonitor reload.

Sekarang, saya agak bingung ini harus diberi nama apa, atau harus diganti namanya atau tidak.

liach pada 10 Mar 2021

Hmm, mungkin ResourceReloader -> SimpleResourceReload , ResourceReloadListener -> ResourceReloader . Selain itu, beri nama ResourceReloadMonitor menjadi ResourceReload dan beri nama implementasi yang sesuai. ResourceReload mewakili satu reload tunggal dan dibuang segera setelahnya, jadi nama ini sebenarnya terdengar bagus.