Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

exception handler added to main function #49

Merged
merged 2 commits into from
Apr 26, 2023
Merged

Conversation

dimas1185
Copy link
Contributor

resolves #39

tools/main.cpp Outdated
@@ -74,7 +74,11 @@ int main(int argc, char** argv) {

try {
return runner.exec();
} catch(...) {
} catch(std::exception& ex) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
} catch(std::exception& ex) {
} catch(const std::exception& ex) {

Copy link
Contributor

@ScottBailey ScottBailey left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

@dimas1185 dimas1185 merged commit d1a884f into main Apr 26, 2023
@dimas1185 dimas1185 deleted the add-exception-handler branch April 26, 2023 17:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

If a directory doesn't exist, nothing happens when remove command is used
3 participants