search

TheCherno / Sparky

Cross-Platform High Performance 2D/3D game engine for people like me who like to write code.
Apache License 2.0
1.09k stars 222 forks source link

Redundant assertion in AllocateDebug() #69

Closed PistonMiner closed 8 years ago

PistonMiner commented 8 years ago

The assertion in L33 can be removed due to it already being asserted in Allocate().

TheCherno commented 8 years ago

I think my plan was to not return Allocate() from AllocateDebug, but rather use something else. I just didn't get around to it so I'm returning Allocate() from now. That assertion will need to be brought back when this function gets properly implemented, so no need to remove it for now.