Merge pull request #3609 from lordofhyphens/3597-fix-path-references

Fixed small issue with perimeter path splitting could cause negative …
This commit is contained in:
Alessandro Ranellucci 2016-12-09 10:58:53 +01:00 committed by GitHub
commit 540d632bbb

View File

@ -164,8 +164,10 @@ ExtrusionLoop::split_at(const Point &point)
}
// now split path_idx in two parts
ExtrusionPath p1(this->paths[path_idx].role), p2(this->paths[path_idx].role);
this->paths[path_idx].polyline.split_at(p, &p1.polyline, &p2.polyline);
const ExtrusionPath &path = this->paths[path_idx];
ExtrusionPath p1(path.role, path.mm3_per_mm, path.width, path.height);
ExtrusionPath p2(path.role, path.mm3_per_mm, path.width, path.height);
path.polyline.split_at(p, &p1.polyline, &p2.polyline);
if (this->paths.size() == 1) {
if (! p1.polyline.is_valid())