Page MenuHomePhabricator

PlanarEllipse has an 'unnecessary' fourth control-point and a second PolyLine
Closed, WontfixPublic

Description

The mitk::PlanarEllipse currently uses 4 Control-Points and two Polylines although the documentation of the class itself states, that an ellipse has 3 control-points by definition.

The first and the fourth control-point form a second polyline for which I cannot see any purpose. I have seen, that Peter obviously had something in mind when implementing that code, but it seems to be broken.

Does someone know what the purpose of this extension is or can it simply be removed??

Event Timeline

User engelm has pushed new remote branch:

bug-19691-fix-planarellipse

I once talked to Peter regarding this ugly extra line and he answered that it is somehow useful to see the maximum radius of the ellipse on an arbitrary axis while placing the ellipse figure. I agree to you, it's terrible and I vote for removing it. If people start to complain, we can still make it somehow optional.

This task was automatically closed because it wasn't updated at least since July 2016 (over 2 years). Please re-open this task if you think that it is still relevant. This most probably means that you will resolve it.