-
Notifications
You must be signed in to change notification settings - Fork 23
homeworks by Krivoruchko Dmitry #15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
homeworks by Krivoruchko Dmitry #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
3c9e526
to
4ad0080
Compare
add6e0f
to
246c0b0
Compare
return Graph(edges); | ||
|
||
} else if constexpr (std::is_same_v<vert_t, std::string>) | ||
throw std::logic_error( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
кажется лучше тут не кидать в рантайме, а по раньше как-то кидать, например при компиляции?
но в принципе не обязательно, пока все и так на автомат выглядит (но многое ещё не смотрел)
No description provided.