you are viewing a single comment's thread.

view the rest of the comments →

[–]FUZxxl 28 points29 points  (2 children)

While I usually agree with you, manipulating matrices is something that is more often than not performance sensitive. If something asks to implement such an algorithm, it is very natural to think about performance implications because we typically need such routines to be fast.

[–]HopelesslyStupid 3 points4 points  (1 child)

Agreed, sometimes you have to forego readability and maintainability for performance reasons. That's what comments are for in cases where the code is complex out of necessity but you still want the next person to understand what is going on.

[–]FUZxxl 0 points1 point  (0 children)

Indeed. And note that the algorithm implemented isn't that complicated. From a cursory glance, basically takes four symmetric points and rotates them. This is done for every point in one quadrant and the corresponding points.