search

randomPoison / gunship-rs

An experimental game engine written in Rust.
MIT License
27 stars 3 forks source link

Remove Custom Time Functionality From Bootstrap #67

Closed randomPoison closed 7 years ago

randomPoison commented 8 years ago

The time measurement functionality in the standard library is gradually becoming stable and it almost exactly matches the functionality we've build in bootstrap. As much as I like the whole "build it from scratch" mentality there's not much to be gained in this case. We should remove the time module from bootstrap and replace all usage of its functionality with the appropriate parts of std::time.

randomPoison commented 8 years ago

Looking at the current state of the standard library time features it looks like it doesn't exactly align with what we want. Duration requires you to get seconds, milliseconds, and nanoseconds separately and doesn't have a way to do a combined f32 value which is what's used most commonly in game dev. This is something that we could extend within gunship but that's worth considering while making this change.

randomPoison commented 7 years ago

Looks like this was resolved in dcc2a01.