Minor fixes related to release build

Author: basiliscos

CMakeLists.txt

@@ -73,7 +73,8 @@ if(WIN32)
     add_definitions(-DBOOST_ALL_DYN_LINK -DBOOST_THREAD_USE_LIB -D_WIN32_WINNT=0x0501)
 else()
     add_definitions(-Wall -Wextra -pedantic -Werror)
-    set(LINK_DEPENDENCIES pthread ${Boost_LIBRARIES} catch_lib)
+    #add_definitions(-fsanitize=undefined -fsanitize=address -fno-omit-frame-pointer)
+    set(LINK_DEPENDENCIES pthread ${Boost_LIBRARIES} catch_lib) #  asan ubsan
 endif()
 
 add_subdirectory(examples)

examples/CMakeLists.txt

@@ -1,13 +1,15 @@
 find_package(Threads REQUIRED)
 
+set(LINK_DEPENDENCIES ${Boost_LIBRARIES} Threads::Threads)
+
 add_executable(stream-parse stream-parse.cpp)
-target_link_libraries(stream-parse ${Boost_LIBRARIES} Threads::Threads)
+target_link_libraries(stream-parse ${LINK_DEPENDENCIES})
 
 add_executable(synch-subscription synch-subscription.cpp)
-target_link_libraries(synch-subscription ${Boost_LIBRARIES} Threads::Threads)
+target_link_libraries(synch-subscription ${LINK_DEPENDENCIES})
 
 add_executable(speed_test_async_multi speed_test_async_multi.cpp)
-target_link_libraries(speed_test_async_multi ${Boost_LIBRARIES} Threads::Threads)
+target_link_libraries(speed_test_async_multi ${LINK_DEPENDENCIES})
 
 add_executable(multi-threads-1 multi-threads-1.cpp)
-target_link_libraries(multi-threads-1 ${Boost_LIBRARIES} Threads::Threads)
+target_link_libraries(multi-threads-1 ${LINK_DEPENDENCIES})

examples/speed_test_async_multi.cpp

@@ -100,6 +100,7 @@ int main(int argc, char **argv) {
         rx_buff,
         [&](const boost::system::error_code &ec, auto &&r) {
             assert(!ec);
+            (void)ec;
             auto &replies =
                 boost::get<r::markers::array_holder_t<Iterator>>(r.result);
             auto &last_reply = replies.elements.at(replies.elements.size() - 1);
@@ -115,6 +116,7 @@ int main(int argc, char **argv) {
 
     c.async_write(tx_buff, cmd_wpapper, [&](const boost::system::error_code &ec,
                                             auto bytes_transferred) {
+        (void)ec;
         assert(!ec);
         tx_buff.consume(bytes_transferred);
         std::cout << "done writing...\n";

include/bredis/impl/async_op.ipp

@@ -24,13 +24,14 @@ struct result_handler_t<Iterator, parsing_policy::drop_result> {
     using policy_t = parsing_policy::drop_result;
     using positive_result_t = parse_result_mapper_t<Iterator, policy_t>;
 
-    positive_result_t result;
     std::size_t replies_count;
     size_t cumulative_consumption;
     size_t count;
+    positive_result_t result;
 
     result_handler_t(std::size_t replies_count_)
-        : replies_count{replies_count_}, cumulative_consumption{0}, count{0} {}
+        : replies_count{replies_count_},
+          cumulative_consumption{0}, count{0}, result{0} {}
 
     void init() {
         // NO-OP;
@@ -52,11 +53,11 @@ struct result_handler_t<Iterator, parsing_policy::keep_result> {
     using policy_t = parsing_policy::keep_result;
     using positive_result_t = parse_result_mapper_t<Iterator, policy_t>;
 
-    positive_result_t result;
-    markers::array_holder_t<Iterator> tmp_results;
     std::size_t replies_count;
     size_t cumulative_consumption;
     size_t count;
+    positive_result_t result;
+    markers::array_holder_t<Iterator> tmp_results;
 
     result_handler_t(std::size_t replies_count_)
         : replies_count{replies_count_}, cumulative_consumption{0}, count{0} {}