public inbox for libstdc++@gcc.gnu.org
 help / color / mirror / Atom feed
* [committed] libstdc++: Remove obsolete comment in <string>
@ 2022-07-09 12:25 François Dumont
  2022-07-09 15:46 ` Jonathan Wakely
  0 siblings, 1 reply; 2+ messages in thread
From: François Dumont @ 2022-07-09 12:25 UTC (permalink / raw)
  To: libstdc++; +Cc: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 430 bytes --]

    libstdc++: Remove obsolete comment in <string> header

     The comment is obsolete because char_traits.h do not include 
stl_algobase.h
     anymore and stl_algobase.h is included directly from <string> a few 
lines
     below.

     libstdc++-v3/ChangeLog:

             * include/std/string: Remove obsolete comment about 
char_traits.h including
             stl_algobase.h.

François

[-- Attachment #2: string.patch --]
[-- Type: text/x-patch, Size: 507 bytes --]

diff --git a/libstdc++-v3/include/std/string b/libstdc++-v3/include/std/string
index 37a4aaba9cd..62ecdb3af45 100644
--- a/libstdc++-v3/include/std/string
+++ b/libstdc++-v3/include/std/string
@@ -37,7 +37,7 @@
 
 #include <bits/c++config.h>
 #include <bits/stringfwd.h>
-#include <bits/char_traits.h>  // NB: In turn includes stl_algobase.h
+#include <bits/char_traits.h>
 #include <bits/allocator.h>
 #include <bits/cpp_type_traits.h>
 #include <bits/localefwd.h>    // For operators >>, <<, and getline.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [committed] libstdc++: Remove obsolete comment in <string>
  2022-07-09 12:25 [committed] libstdc++: Remove obsolete comment in <string> François Dumont
@ 2022-07-09 15:46 ` Jonathan Wakely
  0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Wakely @ 2022-07-09 15:46 UTC (permalink / raw)
  To: François Dumont; +Cc: libstdc++, gcc-patches

On Sat, 9 Jul 2022 at 13:26, François Dumont via Libstdc++
<libstdc++@gcc.gnu.org> wrote:
>
>     libstdc++: Remove obsolete comment in <string> header
>
>      The comment is obsolete because char_traits.h do not include
> stl_algobase.h
>      anymore and stl_algobase.h is included directly from <string> a few
> lines
>      below.

Nice, thanks


>
>      libstdc++-v3/ChangeLog:
>
>              * include/std/string: Remove obsolete comment about
> char_traits.h including
>              stl_algobase.h.
>
> François


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-07-09 15:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-09 12:25 [committed] libstdc++: Remove obsolete comment in <string> François Dumont
2022-07-09 15:46 ` Jonathan Wakely

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).