Skip to content

Commit 7079680

Browse files
authored
Merge pull request #1130 from PowerGridModel/Feature/sonar-cloud-issue-4
Sonar cloud issues: destructors must be noexcept
2 parents ca58e8c + 2aa714b commit 7079680

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

power_grid_model_c/power_grid_model/include/power_grid_model/common/multi_threaded_logging.hpp

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,13 @@ class MultiThreadedLoggerImpl : public MultiThreadedLogger {
2424
ThreadLogger& operator=(ThreadLogger const&) = default;
2525
ThreadLogger(ThreadLogger&&) noexcept = default;
2626
ThreadLogger& operator=(ThreadLogger&&) noexcept = default;
27-
~ThreadLogger() noexcept override { sync(); }
27+
~ThreadLogger() noexcept override {
28+
try {
29+
sync();
30+
} catch (...) { // NOLINT(bugprone-empty-catch) // NOSONAR
31+
// we can't sync so we need to ignore the error
32+
}
33+
}
2834
void sync() const { parent_->sync(*this); }
2935

3036
private:

0 commit comments

Comments
 (0)