Skip to content

Conversation

@themanyone
Copy link

@themanyone themanyone commented Jan 10, 2026

Fixes #13739

Optimized if statement so functional operation appears on right-hand side of the ||
That way, faster == checks are performed first and function doesn't operate on 0-length lines.

That prevented some assertions but CoolingBuffer was still trying to process some empty gcode, which is worked-around by skipping empty lines.

+        if (line.length() == 0)
+            continue; // Empty line.
...

Added a bounds check to DoubleSliderForLayers.cpp so operating the slider doesn't crash plater on bottom layer.
Finally, replaced two assertions in Polyline.cpp with if statements to prevent crash when trying to rotate the view.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CoolingBuffer.cpp:533 Assertion `sm.length > 0' failed in vase mode with fuzzy skin enabled.

1 participant