From d65ee04b74cc166e98fdb409568bfbc9f40da40f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Ho=C5=99e=C5=88ovsk=C3=BD?= Date: Sun, 3 Oct 2021 20:19:32 +0200 Subject: [PATCH] Remove double serialization when stringifying std::optional --- src/catch2/catch_tostring.hpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/catch2/catch_tostring.hpp b/src/catch2/catch_tostring.hpp index e82552bf..b718a18b 100644 --- a/src/catch2/catch_tostring.hpp +++ b/src/catch2/catch_tostring.hpp @@ -388,13 +388,11 @@ namespace Catch { template struct StringMaker > { static std::string convert(const std::optional& optional) { - ReusableStringStream rss; if (optional.has_value()) { - rss << ::Catch::Detail::stringify(*optional); + return ::Catch::Detail::stringify(*optional); } else { - rss << "{ }"; + return "{ }"; } - return rss.str(); } }; }