Make wpi::ct_string implicitly convert to std::string

wpilibsuite · Dec 4, 2023 · 8a12a97 · 8a12a97
1 parent 507a082
commit 8a12a97
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/wpilibc/src/test/native/cpp/util/Color8BitTest.cpp b/wpilibc/src/test/native/cpp/util/Color8BitTest.cpp
@@ -2,6 +2,8 @@
 // Open Source Software; you can modify and/or share it under the terms of
 // the WPILib BSD license file in the root directory of this project.
 
+#include <string>
+
 #include <gtest/gtest.h>
 
 #include "frc/util/Color8Bit.h"
@@ -59,6 +61,9 @@ TEST(Color8BitTest, ToHexString) {
   constexpr frc::Color8Bit color1{255, 128, 64};
   EXPECT_EQ("#FF8040", color1.HexString());
 
+  // Ensure conversion to std::string works
+  [[maybe_unused]] std::string str = color1.HexString();
+
   frc::Color8Bit color2{255, 128, 64};
   EXPECT_EQ("#FF8040", color2.HexString());
 }
diff --git a/wpilibc/src/test/native/cpp/util/ColorTest.cpp b/wpilibc/src/test/native/cpp/util/ColorTest.cpp
@@ -2,6 +2,8 @@
 // Open Source Software; you can modify and/or share it under the terms of
 // the WPILib BSD license file in the root directory of this project.
 
+#include <string>
+
 #include <gtest/gtest.h>
 
 #include "frc/util/Color.h"
@@ -59,6 +61,9 @@ TEST(ColorTest, ToHexString) {
   constexpr frc::Color color1{255, 128, 64};
   EXPECT_EQ("#FF8040", color1.HexString());
 
+  // Ensure conversion to std::string works
+  [[maybe_unused]] std::string str = color1.HexString();
+
   frc::Color color2{255, 128, 64};
   EXPECT_EQ("#FF8040", color2.HexString());
 }
diff --git a/wpiutil/src/main/native/include/wpi/ct_string.h b/wpiutil/src/main/native/include/wpi/ct_string.h
@@ -99,6 +99,11 @@ struct ct_string {
   constexpr auto data() const noexcept { return chars.data(); }
   constexpr auto c_str() const noexcept { return chars.data(); }
 
+  constexpr operator std::basic_string<Char, Traits>()  // NOLINT
+      const noexcept {
+    return std::basic_string<Char, Traits>{chars.data(), N};
+  }
+
   constexpr operator std::basic_string_view<Char, Traits>()  // NOLINT
       const noexcept {
     return std::basic_string_view<Char, Traits>{chars.data(), N};