search

SuperTux / supertux

SuperTux source code
https://supertux.org
GNU General Public License v3.0
2.5k stars 476 forks source link

Removing unneeded dependencies like GLM #2689

Open MatusGuy opened 9 months ago

MatusGuy commented 9 months ago

GLM is just a struct with an x and y value and operator overloads. FMT is just snprintf.

More dependencies = harder to port. It's very disappointing having to deal with these libraries when they could've just been replaced anyway

mrkubax10 commented 9 months ago

GLM is just a struct with an x and y value and operator overloads.

Rather you mean glm::vec2 from GLM?

MatusGuy commented 9 months ago

Yes and that's the only thing we need from it

tobbi commented 9 months ago

https://github.com/SuperTux/supertux/pull/1735#issue-867610129

tobbi commented 9 months ago

https://github.com/SuperTux/supertux/pull/2004#issue-1087152849

MatusGuy commented 9 months ago

Thanks tobbi

MatusGuy commented 2 months ago

apparently fmt has certain performance benefits. I don't wanna touch it.