Chen Liang
Chen Liang
Imo ctx should only exist as lambda params, especially when in conflict with outer local var
Position isn't really more accurate than Result imo.
@sfPlayer1 I recall forge had such issues before and they create extra mappings to fix that. Should we fix this through remapper adding custom bridges instead?
the base link seems to be https://www.slf4j.org/apidocs/
How about just calling it ChunkStorage instead?
How do they show up in vanilla registry class?
Hmm, we have `ItemStack` vs `Item`, `InfoEnchantment` vs `Enchantment`, and `StatusEffectInstance` vs `StatusEffect`. They are all type + count combinations with some extra informations. Don't think simply renaming `StatusEffectInstance` to...
`InfoEnchantment` is the enchantment type + level pair. maybe we can have `InfoEnchantment` -> `Enchantment` and `Enchantment` -> `EnchantmentType`
Sure, I agree with juuz's rename proposal now.
Note that `EntityCategory` is no more (now renamed to `SpawnGroup`), may need a better name for `StatusEffectCategory`