Refine byte writer resizing

Author: dshil

components/ocs_core/byte_writer.cpp

@@ -47,11 +47,8 @@ bool ByteWriter::write_word(uint16_t data) {
     return write(data);
 }
 
-size_t ByteWriter::reserve(size_t size) {
-    const size_t ret = std::min(size, left_());
-    offset_ += ret;
-
-    return ret;
+void ByteWriter::resize(size_t size) {
+    offset_ = std::min(size_, size);
 }
 
 ssize_t ByteWriter::find(uint8_t data) {

components/ocs_core/byte_writer.h

@@ -40,11 +40,8 @@ class ByteWriter : private core::NonCopyable<> {
     //! Write single word.
     bool write_word(uint16_t data);
 
-    //! Reserve space for @p size bytes.
-    //!
-    //! @return
-    //!  Number of reserved bytes.
-    size_t reserve(size_t size);
+    //! Change number of written bytes.
+    void resize(size_t size);
 
     //! Find the position of the byte.
     ssize_t find(uint8_t data);

components/ocs_core/test/test_byte_writer.cpp

@@ -67,27 +67,27 @@ TEST_CASE("Byte writer: write array: by parts", "[ocs_core], [byte_writer]") {
     TEST_ASSERT_TRUE(memcmp(want_buf, writer.get_data(), sizeof(want_buf)) == 0);
 }
 
-TEST_CASE("Byte writer: reserve space: in range", "[ocs_core], [byte_writer]") {
+TEST_CASE("Byte writer: resize", "[ocs_core], [byte_writer]") {
     uint8_t write_buf[2];
     memset(write_buf, 7, sizeof(write_buf));
 
     ByteWriter writer(write_buf, sizeof(write_buf));
 
-    TEST_ASSERT_EQUAL(1, writer.reserve(1));
+    writer.resize(1);
     TEST_ASSERT_TRUE(writer.write(static_cast<uint8_t>(2)));
     TEST_ASSERT_FALSE(writer.write(static_cast<uint8_t>(2)));
     TEST_ASSERT_EQUAL(7, write_buf[0]);
     TEST_ASSERT_EQUAL(2, write_buf[1]);
 }
 
-TEST_CASE("Byte writer: reserve space: no space left", "[ocs_core], [byte_writer]") {
+TEST_CASE("Byte writer: resize: no space left", "[ocs_core], [byte_writer]") {
     uint8_t write_buf[2];
     memset(write_buf, 7, sizeof(write_buf));
 
     ByteWriter writer(write_buf, sizeof(write_buf));
 
-    TEST_ASSERT_EQUAL(writer.get_cap(), writer.reserve(writer.get_cap() + 1));
-    TEST_ASSERT_EQUAL(0, writer.reserve(1));
+    writer.resize(writer.get_cap() + 1);
+    TEST_ASSERT_FALSE(writer.write_byte(10));
 }
 
 TEST_CASE("Byte writer: find byte: in range", "[ocs_core], [byte_writer]") {