Merge pull request #11 from basiliscos/issue-4

Issue #4 : remove unneeded commit on tx_buff

Author: basiliscos

README.md

@@ -17,6 +17,9 @@ Boost::ASIO low-level redis client (connector)
 
 ## Changelog
 
+### 0.04
+ - [bugfix] removed unneeded `tx_buff.commit()` on `async_write` which corrupted buffer
+
 ### 0.03
 - improved protocol parser (no memory allocations during input stream validity check)
 - more detailed information in `protocol_error_t`

include/bredis/impl/connection.ipp

@@ -33,7 +33,6 @@ Connection<NextLayer>::async_write(DynamicBuffer &tx_buff,
     std::ostream os(&tx_buff);
     auto string = boost::apply_visitor(command_serializer_visitor(), command);
     os.write(string.c_str(), string.size());
-    tx_buff.commit(string.size());
 
     real_handler_t handler(std::forward<WriteCallback>(write_callback));
     return async_write(stream_, tx_buff, handler);