Feature top cover yjx - #503
Merged
iesteem merged 3 commits intoJun 17, 2026
Merged
Conversation
extruders_count config option has type coInt, so the underlying boost::any stores an int. Casting it as size_t directly throws boost::bad_any_cast, which wxWidgets reports as "Unhandled unknown exception; terminating the application." Use the safe pattern already established at line 4688: cast to int first, then convert to size_t. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Replace size_t(...) with static_cast<size_t>(...) to comply with coding standard that prohibits C-style casts. Also fix the same issue in the existing code at line 4688. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…ation file Two root causes for "Unhandled unknown exception" at startup: 1. Plater.cpp referenced three config options that were never defined in PrintConfig.cpp: enable_infill_filament_override, infill_filament_use_base_first_layers, infill_filament_use_base_last_layers. This caused "Unknown option exception" during on_init_inner() when the config system looked up these keys. 2. zh_CN .po file had a duplicate block of 5 msgid/msgstr entries inserted during the main merge, plus a malformed line with two msgstr strings concatenated. msgfmt reported 8 fatal errors. Also fixes boost::any_cast<size_t> → static_cast<size_t>(boost::any_cast<int>) in Tab.cpp to match the safe pattern at line 4688 (coInt options store int). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description