Skip to content

Conversation

Noisyfox
Copy link
Collaborator

Fix #6807

All places that use current_extruder_id will first do a if(current_extruder_id) check, which will only be false if this optional is not set. But when it's initialized with -1 all the checks will become useless which causes this crash.

Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix!
Thank you

@SoftFever SoftFever merged commit ad6ea3f into SoftFever:main Sep 18, 2024
15 checks passed
@Noisyfox Noisyfox deleted the bugfox/empty-1st-crash branch September 18, 2024 16:34
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.

Repeatable CRASH on slice
2 participants