Yarn: VoxelShape -> Forma

Hot Take Of The Week ™ ️

Não há nada inerentemente "voxel" sobre a implementação, apenas representa uma coleção de pontos que se formam em uma espécie de forma. O único ponto em que os voxels são introduzidos como um conceito são os métodos de terceiros fornecidos na classe Block, que dividem os pontos dados por 16,0 (o conceito de voxel por cubo padronizado de 16x16x16) antes de construir uma forma. Mas fora disso, nas classes e tipos primários, não há aplicação desse conceito e eles são puramente pontos que representam uma forma que pode ser de qualquer dimensão ou resolução. Eu reconheço que Mojang se refere a eles como VoxelShape dentro de String e que Yarn pode ter uma tendência a respeitar as decisões de nomenclatura de Mojang, mas sinto que este é um dos vários casos em que o nome pode ser facilmente simplificado, já que o prefixo pode ser visto como redundante ou conceitualmente incorreto. Outro pensamento sobre isso é que um caso em Advancement onde o toString de Mojang realmente o nomeia SimpleAdvancement foi reconhecido como incorreto e que Mojang realmente chamou a classe de Advancement , e isso poderia muito bem seja o caso de VoxelShape também. Mojang, na verdade, apenas se refere a ele como VoxelShape 2 vezes na base de código: o toString de VoxelShape e uma mensagem IllegalArgumentException em ArrayVoxelShape. Enquanto isso, também em VoxelShape há uma mensagem UnsupportedOperationException meramente descrevendo-a como uma forma ( "No bounds for empty shape." ). Não tenho muito mais para colocar aqui, então vou apenas concluir lembrando a todos que esta é uma "tomada quente" e não estou pressionando por essa mudança, é apenas uma sugestão curinga.