Dividable-by-three instead of next-dividable-by-three

Project-OSRM · Feb 13, 2016 · 27fe85a · 27fe85a
1 parent eb6f01e
commit 27fe85a
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/include/engine/object_encoder.hpp b/include/engine/object_encoder.hpp
@@ -47,13 +47,15 @@ template <typename T> std::string encodeBase64(const T &x)
                                      reinterpret_cast<const char *>(&x) + sizeof(T)};
     BOOST_ASSERT(!bytes.empty());
 
-    const auto next_divisible_by_three = ((bytes.size() / 3u) + 1u) * 3u;
-    BOOST_ASSERT(next_divisible_by_three >= bytes.size());
+    std::size_t bytes_to_pad{0};
 
-    const auto bytes_to_pad = next_divisible_by_three - bytes.size();
-    BOOST_ASSERT(bytes_to_pad == 0 || bytes_to_pad == 1 || bytes_to_pad == 2);
+    while (bytes.size() % 3 != 0)
+    {
+        bytes_to_pad += 1;
+        bytes.push_back(0);
+    }
 
-    bytes.insert(end(bytes), bytes_to_pad, 0x00);
+    BOOST_ASSERT(bytes_to_pad == 0 || bytes_to_pad == 1 || bytes_to_pad == 2);
     BOOST_ASSERT_MSG(0 == bytes.size() % 3, "base64 input data size is not a multiple of 3");
 
     std::string encoded{detail::Base64FromBinary{bytes.data()},