Skip to content

refs #10736 - cli/main.cpp: removed conditional (NDEBUG) exception handling#8122

Merged
firewave merged 1 commit intodanmar:mainfrom
firewave:ndebug
Jan 21, 2026
Merged

refs #10736 - cli/main.cpp: removed conditional (NDEBUG) exception handling#8122
firewave merged 1 commit intodanmar:mainfrom
firewave:ndebug

Conversation

@firewave
Copy link
Collaborator

No description provided.

@firewave
Copy link
Collaborator Author

firewave commented Jan 14, 2026

It was only enabled in Windows build. And did not make much sense since the called function does not throw. This also makes it easier to detect uncaught exceptions in static analysis.

@firewave firewave changed the title refs #10736 - removed conditional (NDEBUG) exception handling refs #10736 - cli/main.cpp - removed conditional (NDEBUG) exception handling Jan 14, 2026
@firewave firewave changed the title refs #10736 - cli/main.cpp - removed conditional (NDEBUG) exception handling refs #10736 - cli/main.cpp: removed conditional (NDEBUG) exception handling Jan 14, 2026
@sonarqubecloud
Copy link

@firewave firewave merged commit fdd7b1d into danmar:main Jan 21, 2026
63 checks passed
@firewave firewave deleted the ndebug branch January 21, 2026 18:51
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.

2 participants