Archived discussion

Code duplication of Convert_CompBezierCurvesToBSplineCurve and ShapeConstruct_CompBezierCurvesToBSplineCurve

RefactoringMon, 02/22/2016 - 17:271 reply

Browse this forum
QuestionAuthor

The code of Convert_CompBezierCurvesToBSplineCurve and ShapeConstruct_CompBezierCurvesToBSplineCurve is nearly the same. Looking at the history of both, the only difference seems to come from "0025256: Small optimization in Convert comp bezier to BSpline". Why are there these two classes? Shouldn't the ShapeConstruct class be better removed to avoid code duplication?

Regards

Discussion

1 archived reply

Posts are displayed in their archived order.

01Commenter-1

Hello Author,

You are right, these classes are duplicates (even if each contain some specific amendments made after separation). Thank you for spotting this! Classes from ShapeConstruct are not used inside OCCT; they shall be removed. I have registered issue #27234 for that.

Commenter-1