From 484acd42468aaedff3311706fb14035de0d5af85 Mon Sep 17 00:00:00 2001 From: jean-pierre charras Date: Tue, 9 May 2023 09:18:20 +0200 Subject: [PATCH] BOARD_ADAPTER::addText() do not call Fracture twice: it is useless. --- 3d-viewer/3d_canvas/create_3Dgraphic_brd_items.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/3d-viewer/3d_canvas/create_3Dgraphic_brd_items.cpp b/3d-viewer/3d_canvas/create_3Dgraphic_brd_items.cpp index d939b26571..3b85c33032 100644 --- a/3d-viewer/3d_canvas/create_3Dgraphic_brd_items.cpp +++ b/3d-viewer/3d_canvas/create_3Dgraphic_brd_items.cpp @@ -97,8 +97,9 @@ void BOARD_ADAPTER::addText( const EDA_TEXT* aText, CONTAINER_2D_BASE* aContaine aText->TransformBoundingBoxToPolygon( &finalPoly, margin ); finalPoly.BooleanSubtract( knockouts, SHAPE_POLY_SET::PM_FAST ); - finalPoly.Fracture( SHAPE_POLY_SET::PM_FAST ); + // Do not call finalPoly.Fracture() here: ConvertPolygonToTriangles() call it + // if needed, and Fracture() called twice can create bad results and is useless ConvertPolygonToTriangles( finalPoly, *aContainer, m_biuTo3Dunits, *aOwner ); } else