NovelRT icon indicating copy to clipboard operation
NovelRT copied to clipboard

Maths namespace has incomplete doxygen documentation in its headers.

Open RubyNova opened this issue 3 years ago • 5 comments

This one is pretty much what it says on the tin - the Maths namespace is lacking complete doxygen-based documentation in it's headers, especially when compared to a namespace such as ECS. Adding doxygen comments of similar caliber to the Maths headers will resolve this issue.

RubyNova avatar Oct 01 '21 00:10 RubyNova

Taking on this one, thank you for setting up issue!

LinuxLegend avatar Oct 01 '21 00:10 LinuxLegend

Assigned. :)

RubyNova avatar Oct 01 '21 00:10 RubyNova

For future issue to work on: Split up intersectsWith into intersectsWithAABB (for evaluating non-rotated bounding boxes) and intersectsWIth (for evaluating rotated bounding boxes)

LinuxLegend avatar Oct 01 '21 00:10 LinuxLegend

Assuming ticket has been abandoned, reopening.

RubyNova avatar May 09 '22 15:05 RubyNova

I think i am up for the task

Pheubel avatar Jul 14 '22 22:07 Pheubel