pmmp / Math

PHP library containing math related code used in PocketMine-MP
GNU Lesser General Public License v3.0
41 stars 20 forks source link

Add toVector2 in Vector3 #62

Closed AkmalFairuz closed 2 years ago

dktapps commented 2 years ago

This doesn't really make any sense. There are 3 axes to choose from. It makes more sense to let the programmer call the constructor directly with the axes they want to retain.

dktapps commented 2 years ago

Vector2 is pretty much unused. I don't see the value of adding a bunch of extra methods that no one will use anyway. It's simpler to just call new Vector2 directly.