From e9c0c93ff18f13a91d32c9705e17d4e0d44a32d8 Mon Sep 17 00:00:00 2001 From: Joseph Lenox Date: Thu, 28 Sep 2017 22:20:30 -0500 Subject: [PATCH] Address warnings about comparing signed/unsigned integers. --- xs/src/libslic3r/LayerRegion.cpp | 4 ++-- xs/src/libslic3r/PerimeterGenerator.cpp | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/xs/src/libslic3r/LayerRegion.cpp b/xs/src/libslic3r/LayerRegion.cpp index bf9b856ae..e2c8ce576 100644 --- a/xs/src/libslic3r/LayerRegion.cpp +++ b/xs/src/libslic3r/LayerRegion.cpp @@ -89,7 +89,7 @@ LayerRegion::process_external_surfaces() // (thus not visible from the outside), like a slab sustained by // pillars, include them in the bridge in order to have better and // more continuous bridging. - for (int i = 0; i < surfaces[j].expolygon.holes.size(); ++i) { + for (size_t i = 0; i < surfaces[j].expolygon.holes.size(); ++i) { // reverse the hole and consider it a polygon Polygon h = surfaces[j].expolygon.holes[i]; h.reverse(); @@ -97,7 +97,7 @@ LayerRegion::process_external_surfaces() // Is this hole fully contained in the layer slices? if (diff(h, this->layer()->slices).empty()) { // remove any other surface contained in this hole - for (int k = 0; k < surfaces.size(); ++k) { + for (size_t k = 0; k < surfaces.size(); ++k) { if (k == j) continue; if (h.contains(surfaces[k].expolygon.contour.first_point())) { surfaces.erase(surfaces.begin() + k); diff --git a/xs/src/libslic3r/PerimeterGenerator.cpp b/xs/src/libslic3r/PerimeterGenerator.cpp index 02c2488df..3d1b80cc1 100644 --- a/xs/src/libslic3r/PerimeterGenerator.cpp +++ b/xs/src/libslic3r/PerimeterGenerator.cpp @@ -215,7 +215,7 @@ PerimeterGenerator::process() // find the contour loop that contains it for (int t = d-1; t >= 0; --t) { - for (int j = 0; j < contours[t].size(); ++j) { + for (size_t j = 0; j < contours[t].size(); ++j) { PerimeterGeneratorLoop &candidate_parent = contours[t][j]; if (candidate_parent.polygon.contains(loop.polygon.first_point())) { candidate_parent.children.push_back(loop);