Conversation
a-badin
left a comment
a-badin
left a comment
There was a problem hiding this comment.
Код не компилируется, не запускается, каких-то файлов просто нет
| file(GLOB_RECURSE SOURCE_FILES project/src/*.cpp) | ||
| file(GLOB_RECURSE TEST_FILES project/test/*.cpp) | ||
|
|
||
| add_library(sha256 SHARED project/src/SHA256.c project/include/SHA256.h) |
There was a problem hiding this comment.
Есть куча библиотек которые умеют это делать libcrypt, libopenssl
| target_link_libraries(backend INTERFACE sha256 | ||
| PUBLIC Boost::serialization | ||
| PUBLIC smallvec | ||
| PUBLIC cppcoro) |
There was a problem hiding this comment.
Если все равно пользуетесь стороней библиотекой - взяли бы boost::coro, asio с ними работает
| cppcoro::async_scope async; | ||
| for (auto &executor : executors) { | ||
| async.spawn( | ||
| executor->execute(std::forward<Command>(command)) | |
There was a problem hiding this comment.
forward не используется для rvalue, для rvalue используется move
| }; | ||
|
|
||
| TEST(executors, executing_command_through_local_command_executor) { | ||
| auto executor = LocalCommandExecutor(); |
There was a problem hiding this comment.
LocalCommandExecutor executor;
auto во многих стилях программирования ограничивают итераторами, результатом от make_shared/make_unque и теми случаеми, когда без него невозможно. В противном случае код быстро становится нечитаемым или происходят случаи, когда int или unique_ptr передают по константной ссылке
a-badin
left a comment
a-badin
left a comment
There was a problem hiding this comment.
Правьте замечания и сливайте в main
2 participants
No description provided.