A couple notes about infill_only_where_needed

slic3r · Feb 22, 2015 · e88cf46 · e88cf46
1 parent 2655f3f
commit e88cf46
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/Slic3r/Print/Object.pm b/lib/Slic3r/Print/Object.pm
@@ -792,7 +792,11 @@ sub clip_fill_surfaces {
                     $new_internal,
                     1,
                 )};
-
+
+            # If there are voids it means that our internal infill is not adjacent to
+            # perimeters. In this case it would be nice to add a loop around infill to
+            # make it more robust and nicer. TODO.
+
             $layerm->fill_surfaces->clear;
             $layerm->fill_surfaces->append($_) for (@new, @other);
         }

diff --git a/xs/src/libslic3r/PrintConfig.cpp b/xs/src/libslic3r/PrintConfig.cpp
@@ -428,7 +428,7 @@ PrintConfigDef::build_def() {
     Options["infill_only_where_needed"].type = coBool;
     Options["infill_only_where_needed"].label = "Only infill where needed";
     Options["infill_only_where_needed"].category = "Infill";
-    Options["infill_only_where_needed"].tooltip = "This option will limit infill to the areas actually needed for supporting ceilings (it will act as internal support material).";
+    Options["infill_only_where_needed"].tooltip = "This option will limit infill to the areas actually needed for supporting ceilings (it will act as internal support material). If enabled, slows down the G-code generation due to the multiple checks involved.";
     Options["infill_only_where_needed"].cli = "infill-only-where-needed!";
 
     Options["infill_overlap"].type = coFloatOrPercent;