SuperMartijn642 / Fusion

20 stars 3 forks source link

Avoid pointless ternary in ConnectingBakedModel #64

Open embeddedt opened 2 months ago

embeddedt commented 2 months ago

ModelData#get() already returns null if the property doesn't exist, so there isn't a need to call has explicitly. Since calling both has and get requires looking up the property from the underlying map twice, this should slightly improve performance of getQuads (but I did not profile, and the improvement may be insignificant).

This is just a drive-by PR because I happened to be reading the file and it seemed like an obvious cleanup.