Use std::optional::emplace() when loading to construct and load the contained value directly in place

2021-09-17 11:09:39 -07:00
parent 0bbbb142f9
commit f8338dbb73
1 changed files with 2 additions and 3 deletions
@@ -56,9 +56,8 @@ namespace cereal {
    if (nullopt) {
      optional = std::nullopt;
    } else {
-      T value;
-      ar(CEREAL_NVP_("data", value));
-      optional = std::move(value);
+      optional.emplace();
+      ar(CEREAL_NVP_("data", *optional));
    }
  }
 } // namespace cereal