search

stefanmielke / libdragon-extensions

List of structs and functions to help on Libdragon development.
MIT License
6 stars 1 forks source link

Add Size_int, perhaps consolidate position_int and position headers? #13

Open adamcate opened 1 year ago

adamcate commented 1 year ago

I've been looking over position and position_int and I think it would be less confusing for users if position_int was in the same header as the floating point version. I'm probably going to be adding a size_int type, and I think it should follow the same pattern.

Let me know what you think

adamcate commented 1 year ago

12 You can see I have a very basic outline for some SizeInt and RectInt functions in the latest commits. I also decided to replace some of the constructor code with C designated initializers since it's a bit cleaner.

stefanmielke commented 1 year ago

I think it's fine to have both on the same header. As for the C initializers, definitely cleaner :)