fix optional nullopt

Signed-off-by: dentiny <[email protected]>
ray-project · Oct 8, 2024 · e0fed71 · e0fed71
1 parent f9ca13e
commit e0fed71
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/ray/util/container_util.h b/src/ray/util/container_util.h
@@ -147,6 +147,9 @@ std::ostream &operator<<(std::ostream &os,
 
 template <typename T>
 std::ostream &operator<<(std::ostream &os, DebugStringWrapper<std::optional<T>> c) {
+  if (!c.obj_.has_value()) {
+    return os << "(nullopt)";
+  }
   return os << debug_string(c.obj_.value());
 }
 

diff --git a/src/ray/util/tests/container_util_test.cc b/src/ray/util/tests/container_util_test.cc
@@ -69,6 +69,7 @@ TEST(ContainerUtilTest, TestDebugString) {
 
   // Optional.
   ASSERT_EQ(debug_string_to_string(std::nullopt), "(nullopt)");
+  ASSERT_EQ(debug_string_to_string(std::optional<std::string>{}), "(nullopt)");
   ASSERT_EQ(debug_string_to_string(std::optional<std::string>{"hello"}), "hello");
 
   // Composable: tuples of pairs of maps and vectors.