From 3cd73543a1122d3c81409e3e9a26c3e94c3d324f Mon Sep 17 00:00:00 2001 From: Jonathan Wakely Date: Mon, 4 Dec 2023 12:03:28 +0000 Subject: [PATCH] libstdc++: Disable std::formatter::set_debug_format [PR112832] All set_debug_format member functions should be guarded by the __cpp_lib_formatting_ranges macro (which is not defined yet). libstdc++-v3/ChangeLog: PR libstdc++/112832 * include/std/format (formatter::set_debug_format): Ensure this member is defined conditionally for all specializations. * testsuite/std/format/formatter/112832.cc: New test. --- libstdc++-v3/include/std/format | 8 +++++ .../testsuite/std/format/formatter/112832.cc | 29 +++++++++++++++++++ 2 files changed, 37 insertions(+) create mode 100644 libstdc++-v3/testsuite/std/format/formatter/112832.cc diff --git a/libstdc++-v3/include/std/format b/libstdc++-v3/include/std/format index 58cd310db4d..01f0a58392a 100644 --- a/libstdc++-v3/include/std/format +++ b/libstdc++-v3/include/std/format @@ -1815,9 +1815,11 @@ namespace __format return _M_f.format(__u, __fc); } +#if __cpp_lib_format_ranges constexpr void set_debug_format() noexcept { _M_f._M_spec._M_type = __format::_Pres_esc; } +#endif private: __format::__formatter_int _M_f; @@ -1843,7 +1845,9 @@ namespace __format format(_CharT* __u, basic_format_context<_Out, _CharT>& __fc) const { return _M_f.format(__u, __fc); } +#if __cpp_lib_format_ranges constexpr void set_debug_format() noexcept { _M_f.set_debug_format(); } +#endif private: __format::__formatter_str<_CharT> _M_f; @@ -1866,7 +1870,9 @@ namespace __format basic_format_context<_Out, _CharT>& __fc) const { return _M_f.format(__u, __fc); } +#if __cpp_lib_format_ranges constexpr void set_debug_format() noexcept { _M_f.set_debug_format(); } +#endif private: __format::__formatter_str<_CharT> _M_f; @@ -1888,7 +1894,9 @@ namespace __format basic_format_context<_Out, _CharT>& __fc) const { return _M_f.format({__u, _Nm}, __fc); } +#if __cpp_lib_format_ranges constexpr void set_debug_format() noexcept { _M_f.set_debug_format(); } +#endif private: __format::__formatter_str<_CharT> _M_f; diff --git a/libstdc++-v3/testsuite/std/format/formatter/112832.cc b/libstdc++-v3/testsuite/std/format/formatter/112832.cc new file mode 100644 index 00000000000..9aa2095a73d --- /dev/null +++ b/libstdc++-v3/testsuite/std/format/formatter/112832.cc @@ -0,0 +1,29 @@ +// { dg-do compile { target c++20 } } + +#include + +template()[0])>> +constexpr bool +test_pr112832() +{ + std::formatter f; + if constexpr (requires{ f.set_debug_format(); }) + f.set_debug_format(); + return true; +} + +int main() +{ + static_assert(test_pr112832()); + static_assert(test_pr112832()); + static_assert(test_pr112832()); + static_assert(test_pr112832()); +#ifdef _GLIBCXX_USE_WCHAR_T + static_assert(test_pr112832()); + static_assert(test_pr112832()); + static_assert(test_pr112832()); + static_assert(test_pr112832()); + static_assert(test_pr112832()); +#endif +}