search

bitshifter / glam-rs

A simple and fast linear algebra library for games and graphics
Apache License 2.0
1.46k stars 145 forks source link

IVec2 and similar types should be Ord. #508

Closed mintlu8 closed 1 month ago

mintlu8 commented 3 months ago

So they can be put in a priority queue and work without a wrapper type.

bitshifter commented 1 month ago

A long time ago glam used to implement PartialOrd on float vectors (this was before glam had IVec*) but it confused users and people asked for them to be removed (#138 , #168) , which they were.

I think the same reasons that people didn't like PartialOrd on Vec* would apply to Ord on IVec* as well, so I'm inclined to not add them back.