-
Notifications
You must be signed in to change notification settings - Fork 369
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EntitySize -> EntityDimensions #715
Comments
It was decided after discussion that EntitySize better represents this data class. I believe constant is fine too, representing an immutable size. Imo fixed doesn't really imply that well. |
I am personally not fond of having class names differ from the string representation, especially when the name is not particularly bad. It's just confusing to have blatant conflicting information. |
this mojang is indeed a bad obe. we have dimensions for worlds already! but the rename from constant to fixed i support. |
on a side note, i initially mapped this class before mojang overrode the toString method. we should add to the guideline about whether we are mapping to mojang names (which are sometimes incorrect or off) or correct names. |
EntitySize is wrong. The word "size" means only one value, and using it to refer to the dimensions of an object is bad English. This class represents the dimensions of an entity (width, height), not the size of the entity (big, small). In fact, I think we already have another |
This would be a good issue to get a PR going for |
|
Yes, |
EntityDimensions is literally what it says in the issue name and string... |
If everyone is on the side of doing a PR I can get one going later today. Abd yes, I mean to say |
Yup, go for it |
PR: #745 |
EntityDimensions and all the fields there-in are indirectly confirmed by Mojang through string constants in their code:
EntitySize -> EntityDimensions
EntitySize.constant -> EntityDimensions.constant
In addition to that, I would also change these two methods to be more in line with what they seem to be doing:
EntitySize.resizeable(...) -> EntityDimensions.changing(...)
EntitySize.constant(...) -> EntityDimensions.fixed(...)
The text was updated successfully, but these errors were encountered: