Conversion of circular arc to Bezier curve
This is a follow-up to LP332735. In the file \src\sp-ellipse.cpp there is a formula for calculating the length of a Bezier arm when converting a circular arc to a Bezier curve. The formula reads: len = 0.552*(e-s)/M_PI_2 where (e-s) is the angle of the arc. I think the formula should read: len = 4*tan((e-s)/4)/3
I've created a patch for this, at https://bugs.launchpad.net/inkscape/+bug/332735 , but I'm reluctant to commit it, because I don't actually compile the code myself, I work only from the Windows nightly builds, so it may be a few days before I can test it.
Would anyone be willing to confirm that this patch will not cause a crash, or is it okay if I just commit it? (As to the formula itself, it has already been tested in the file cubicsuperpath.py and it behaves as desired.) http://www.nabble.com/file/p22179587/sp-ellipse.diff sp-ellipse.diff
It seems that it doesn't crash or behave badly, so I committed it as rev 20771. Regards, Krzysztof Kosiński
thanks, I'll check it out on the next Windows nightly build,
Alvin
-----Original Message----- From: Alvin Penner [mailto:penner@...1856...] Sent: 2009 feb 25 0:32 To: inkscape-devel@lists.sourceforge.net Subject: Re: [Inkscape-devel] Conversion of circular arc to Bezier curve
thanks, I'll check it out on the next Windows nightly build,
... which are currently not available :-(. I've already mailed Bob about this, so you'll just have to be patient or compile your own build.
Diederik
participants (3)
-
Alvin Penner
-
Diederik van Lierop
-
Krzysztof Kosiński