3DPrinting/PrusaSlicer-NonPlainar
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Updated Clipper to r467 but it doesn't seem to fix #2028 yet

Browse Source
This commit is contained in:
Alessandro Ranellucci 2014-05-25 22:48:58 +02:00
parent 88a2e5c791
commit a62457d6b5
4 changed files with 104 additions and 94 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
xs/src/ClipperUtils.cpp
View File

@ -1,6 +1,14 @@
#include "ClipperUtils.hpp" #include "ClipperUtils.hpp"
#include "Geometry.hpp" #include "Geometry.hpp"
#ifndef use_lines
#error "use_lines directive is not enabled in clipper.hpp"
#endif
#ifdef use_deprecated
#error "use_deprecated is not disabled in clipper.hpp"
#endif
namespace Slic3r { namespace Slic3r {
//----------------------------------------------------------- //-----------------------------------------------------------

112
xs/src/clipper.cpp
View File

@ -2,7 +2,7 @@
* * * *
* Author : Angus Johnson * * Author : Angus Johnson *
* Version : 6.1.5 * * Version : 6.1.5 *
* Date : 22 May 2014 * * Date : 24 May 2014 *
* Website : http://www.angusj.com * * Website : http://www.angusj.com *
* Copyright : Angus Johnson 2010-2014 * * Copyright : Angus Johnson 2010-2014 *
* * * *
@ -898,18 +898,60 @@ TEdge* FindNextLocMin(TEdge* E)
} }
//------------------------------------------------------------------------------ //------------------------------------------------------------------------------
TEdge* ClipperBase::ProcessBound(TEdge* E, bool IsClockwise) TEdge* ClipperBase::ProcessBound(TEdge* E, bool NextIsForward)
{ {
TEdge *EStart = E, *Result = E; TEdge *EStart = E, *Result = E;
TEdge *Horz = 0; TEdge *Horz = 0;
cInt StartX; cInt StartX;
if (E->OutIdx == Skip)
{
//if edges still remain in the current bound beyond the skip edge then
//create another LocMin and call ProcessBound once more
if (NextIsForward)
{
while (E->Top.Y == E->Next->Bot.Y) E = E->Next;
//don't include top horizontals when parsing a bound a second time,
//they will be contained in the opposite bound ...
while (E != Result && IsHorizontal(*E)) E = E->Prev;
}
else
{
while (E->Top.Y == E->Prev->Bot.Y) E = E->Prev;
while (E != Result && IsHorizontal(*E)) E = E->Next;
}
if (E == Result)
{
if (NextIsForward) Result = E->Next;
else Result = E->Prev;
}
else
{
//there are more edges in the bound beyond result starting with E
if (NextIsForward)
E = Result->Next;
else
E = Result->Prev;
LocalMinima* locMin = new LocalMinima;
locMin->Next = 0;
locMin->Y = E->Bot.Y;
locMin->LeftBound = 0;
locMin->RightBound = E;
locMin->RightBound->WindDelta = 0;
Result = ProcessBound(locMin->RightBound, NextIsForward);
InsertLocalMinima(locMin);
}
return Result;
}
if (IsHorizontal(*E)) if (IsHorizontal(*E))
{ {
//first we need to be careful here with open paths because this //we need to be careful with open paths because this may not be
//may not be a true local minima (ie may be following a skip edge). //a true local minima (ie may be following a skip edge).
//also, watch for adjacent horz edges to start heading left //Also, watch for adjacent horz edges that can head left
//before finishing right ... //before finishing right ...
if (IsClockwise) if (NextIsForward)
{ {
if (E->Prev->Bot.Y == E->Bot.Y) StartX = E->Prev->Bot.X; if (E->Prev->Bot.Y == E->Bot.Y) StartX = E->Prev->Bot.X;
else StartX = E->Prev->Top.X; else StartX = E->Prev->Top.X;
@ -922,9 +964,7 @@ TEdge* ClipperBase::ProcessBound(TEdge* E, bool IsClockwise)
if (E->Bot.X != StartX) ReverseHorizontal(*E); if (E->Bot.X != StartX) ReverseHorizontal(*E);
} }
if (Result->OutIdx != Skip) if (NextIsForward)
{
if (IsClockwise)
{ {
while (Result->Top.Y == Result->Next->Bot.Y && Result->Next->OutIdx != Skip) while (Result->Top.Y == Result->Next->Bot.Y && Result->Next->OutIdx != Skip)
Result = Result->Next; Result = Result->Next;
@ -937,7 +977,7 @@ TEdge* ClipperBase::ProcessBound(TEdge* E, bool IsClockwise)
while (IsHorizontal(*Horz->Prev)) Horz = Horz->Prev; while (IsHorizontal(*Horz->Prev)) Horz = Horz->Prev;
if (Horz->Prev->Top.X == Result->Next->Top.X) if (Horz->Prev->Top.X == Result->Next->Top.X)
{ {
if (!IsClockwise) Result = Horz->Prev; if (!NextIsForward) Result = Horz->Prev;
} }
else if (Horz->Prev->Top.X > Result->Next->Top.X) Result = Horz->Prev; else if (Horz->Prev->Top.X > Result->Next->Top.X) Result = Horz->Prev;
} }
@ -961,7 +1001,7 @@ TEdge* ClipperBase::ProcessBound(TEdge* E, bool IsClockwise)
while (IsHorizontal(*Horz->Next)) Horz = Horz->Next; while (IsHorizontal(*Horz->Next)) Horz = Horz->Next;
if (Horz->Next->Top.X == Result->Prev->Top.X) if (Horz->Next->Top.X == Result->Prev->Top.X)
{ {
if (!IsClockwise) Result = Horz->Next; if (!NextIsForward) Result = Horz->Next;
} }
else if (Horz->Next->Top.X > Result->Prev->Top.X) Result = Horz->Next; else if (Horz->Next->Top.X > Result->Prev->Top.X) Result = Horz->Next;
} }
@ -977,45 +1017,7 @@ TEdge* ClipperBase::ProcessBound(TEdge* E, bool IsClockwise)
ReverseHorizontal(*E); ReverseHorizontal(*E);
Result = Result->Prev; //move to the edge just beyond current bound Result = Result->Prev; //move to the edge just beyond current bound
} }
}
if (Result->OutIdx == Skip)
{
//if edges still remain in the current bound beyond the skip edge then
//create another LocMin and call ProcessBound once more
E = Result;
if (IsClockwise)
{
while (E->Top.Y == E->Next->Bot.Y) E = E->Next;
//don't include top horizontals when parsing a bound a second time,
//they will be contained in the opposite bound ...
while (E != Result && IsHorizontal(*E)) E = E->Prev;
} else
{
while (E->Top.Y == E->Prev->Bot.Y) E = E->Prev;
while (E != Result && IsHorizontal(*E)) E = E->Next;
}
if (E == Result)
{
if (IsClockwise) Result = E->Next;
else Result = E->Prev;
} else
{
//there are more edges in the bound beyond result starting with E
if (IsClockwise)
E = Result->Next;
else
E = Result->Prev;
LocalMinima* locMin = new LocalMinima;
locMin->Next = 0;
locMin->Y = E->Bot.Y;
locMin->LeftBound = 0;
locMin->RightBound = E;
locMin->RightBound->WindDelta = 0;
Result = ProcessBound(locMin->RightBound, IsClockwise);
InsertLocalMinima(locMin);
}
}
return Result; return Result;
} }
//------------------------------------------------------------------------------ //------------------------------------------------------------------------------
@ -1148,7 +1150,7 @@ bool ClipperBase::AddPath(const Path &pg, PolyType PolyTyp, bool Closed)
} }
m_edges.push_back(edges); m_edges.push_back(edges);
bool clockwise; bool nextIsForward;
TEdge* EMin = 0; TEdge* EMin = 0;
//workaround to avoid an endless loop in the while loop below when //workaround to avoid an endless loop in the while loop below when
@ -1170,12 +1172,12 @@ bool ClipperBase::AddPath(const Path &pg, PolyType PolyTyp, bool Closed)
{ {
locMin->LeftBound = E->Prev; locMin->LeftBound = E->Prev;
locMin->RightBound = E; locMin->RightBound = E;
clockwise = false; //Q.nextInLML = Q.prev nextIsForward = false; //Q.nextInLML = Q.prev
} else } else
{ {
locMin->LeftBound = E; locMin->LeftBound = E;
locMin->RightBound = E->Prev; locMin->RightBound = E->Prev;
clockwise = true; //Q.nextInLML = Q.next nextIsForward = true; //Q.nextInLML = Q.next
} }
locMin->LeftBound->Side = esLeft; locMin->LeftBound->Side = esLeft;
locMin->RightBound->Side = esRight; locMin->RightBound->Side = esRight;
@ -1186,15 +1188,15 @@ bool ClipperBase::AddPath(const Path &pg, PolyType PolyTyp, bool Closed)
else locMin->LeftBound->WindDelta = 1; else locMin->LeftBound->WindDelta = 1;
locMin->RightBound->WindDelta = -locMin->LeftBound->WindDelta; locMin->RightBound->WindDelta = -locMin->LeftBound->WindDelta;
E = ProcessBound(locMin->LeftBound, clockwise); E = ProcessBound(locMin->LeftBound, nextIsForward);
TEdge* E2 = ProcessBound(locMin->RightBound, !clockwise); TEdge* E2 = ProcessBound(locMin->RightBound, !nextIsForward);
if (locMin->LeftBound->OutIdx == Skip) if (locMin->LeftBound->OutIdx == Skip)
locMin->LeftBound = 0; locMin->LeftBound = 0;
else if (locMin->RightBound->OutIdx == Skip) else if (locMin->RightBound->OutIdx == Skip)
locMin->RightBound = 0; locMin->RightBound = 0;
InsertLocalMinima(locMin); InsertLocalMinima(locMin);
if (!clockwise) E = E2; if (!nextIsForward) E = E2;
} }
return true; return true;
} }

2
xs/src/clipper.hpp
View File

@ -2,7 +2,7 @@
* * * *
* Author : Angus Johnson * * Author : Angus Johnson *
* Version : 6.1.5 * * Version : 6.1.5 *
* Date : 22 May 2014 * * Date : 24 May 2014 *
* Website : http://www.angusj.com * * Website : http://www.angusj.com *
* Copyright : Angus Johnson 2010-2014 * * Copyright : Angus Johnson 2010-2014 *
* * * *

2
xs/t/11_clipper.t
View File

@ -155,7 +155,7 @@ if (0) { # Clipper does not preserve polyline orientation
} }
} }
# Disabled until Clipper bug #96 (our issue #2028) is fixed # Clipper bug #96 (our GH #2028)
if (0) { if (0) {
my $subject = Slic3r::Polyline->new( my $subject = Slic3r::Polyline->new(
[44735000,31936670],[55270000,31936670],[55270000,25270000],[74730000,25270000],[74730000,44730000],[68063296,44730000],[68063296,55270000],[74730000,55270000],[74730000,74730000],[55270000,74730000],[55270000,68063296],[44730000,68063296],[44730000,74730000],[25270000,74730000],[25270000,55270000],[31936670,55270000],[31936670,44730000],[25270000,44730000],[25270000,25270000],[44730000,25270000],[44730000,31936670] [44735000,31936670],[55270000,31936670],[55270000,25270000],[74730000,25270000],[74730000,44730000],[68063296,44730000],[68063296,55270000],[74730000,55270000],[74730000,74730000],[55270000,74730000],[55270000,68063296],[44730000,68063296],[44730000,74730000],[25270000,74730000],[25270000,55270000],[31936670,55270000],[31936670,44730000],[25270000,44730000],[25270000,25270000],[44730000,25270000],[44730000,31936670]