[libc++][Modules] Restore the <string> include to <__format/format_functions.h>

[iana]

Summary

<__format/format_functions.h> was using <string>, we need to bring the include back that was removed in D154122.

[Mordante]

I think the change is the wrong direction. std::format returns a std::string. In my experience arguments and return values often need to be explicitly exported. So I think the proper fix would be to include string in this header. The main issue is that accidentally forgotten headers worked in the past.

[iana]

In D155116#4501194, @Mordante wrote:

I think the change is the wrong direction. std::format returns a std::string. In my experience arguments and return values often need to be explicitly exported. So I think the proper fix would be to include string in this header. The main issue is that accidentally forgotten headers worked in the past.

But you just barely removed that include in D154122

[iana]

In D155116#4501682, @iana wrote:

In D155116#4501194, @Mordante wrote:

I think the change is the wrong direction. std::format returns a std::string. In my experience arguments and return values often need to be explicitly exported. So I think the proper fix would be to include string in this header. The main issue is that accidentally forgotten headers worked in the past.

But you just barely removed that include in D154122

Then again, the CI says you're right and the include is needed.

[iana]

Restore the <string> include to <__format/format_functions.h> instead

[Mordante]

In D155116#4501682, @iana wrote:

In D155116#4501194, @Mordante wrote:

I think the change is the wrong direction. std::format returns a std::string. In my experience arguments and return values often need to be explicitly exported. So I think the proper fix would be to include string in this header. The main issue is that accidentally forgotten headers worked in the past.

But you just barely removed that include in D154122

That seems like a mistake on my side.

Thanks for the patch, LGTM!

libcxx/include/__format/format_functions.h

 #include <__format/formatter_string.h>
 #include <__format/parser_std_format_spec.h>
 #include <__iterator/back_insert_iterator.h>
 #include <__iterator/concepts.h>
 #include <__iterator/incrementable_traits.h>
 #include <__iterator/iterator_traits.h> // iter_value_t
 #include <__variant/monostate.h>
 #include <array>
+#include <string>
 #include <string_view>
 
 #ifndef _LIBCPP_HAS_NO_LOCALIZATION
 #include <locale>
 #endif
 
 #if !defined(_LIBCPP_HAS_NO_PRAGMA_SYSTEM_HEADER)
 #  pragma GCC system_header