Fix horrible leak in String::Data::operator=(String::Data&&)

And that, my friends, is why we recommend using standard containers
instead of rolling your own.

1 files changed, 3 insertions, 1 deletions

diff --git a/src/string.cc b/src/string.cc
index 5a229030..18ae5d80 100644
--- a/src/string.cc
+++ b/src/string.cc
@@ -34,7 +34,7 @@ String::Data::Data(Data&& other) noexcept
     if (other.is_long())
     {
         l = other.l;
-        other.s.size = 1;
+        other.set_empty();
     }
     else
         s = other.s;
@@ -52,6 +52,8 @@ String::Data& String::Data::operator=(const Data& other)
 
 String::Data& String::Data::operator=(Data&& other) noexcept
 {
+    release();
+
     if (other.is_long())
     {
         l = other.l;