public inbox for libstdc++-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r14-3309] libstdc++: Replace global std::string objects in tzdb.cc
@ 2023-08-17 23:28 Jonathan Wakely
  0 siblings, 0 replies; only message in thread
From: Jonathan Wakely @ 2023-08-17 23:28 UTC (permalink / raw)
  To: gcc-cvs, libstdc++-cvs

https://gcc.gnu.org/g:d82a85b6161cbeebfe3de5d3d4cdd355e4617e69

commit r14-3309-gd82a85b6161cbeebfe3de5d3d4cdd355e4617e69
Author: Jonathan Wakely <jwakely@redhat.com>
Date:   Thu Aug 17 20:39:02 2023 +0100

    libstdc++: Replace global std::string objects in tzdb.cc
    
    When the library is built with --disable-libstdcxx-dual-abi the only
    type of std::string supported is the COW string, and the two global
    std::string objects in tzdb.cc have to allocate memory. I added them
    thinking they would fit in the SSO string buffer, but that's not the
    case when the library only uses COW strings.
    
    Replace them with string_view objects to avoid any allocations.
    
    libstdc++-v3/ChangeLog:
    
            * src/c++20/tzdb.cc (tzdata_file, leaps_file): Change type to
            std::string_view.

Diff:
---
 libstdc++-v3/src/c++20/tzdb.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libstdc++-v3/src/c++20/tzdb.cc b/libstdc++-v3/src/c++20/tzdb.cc
index 8d27726016ec..0fcbf6a4824e 100644
--- a/libstdc++-v3/src/c++20/tzdb.cc
+++ b/libstdc++-v3/src/c++20/tzdb.cc
@@ -1078,8 +1078,8 @@ namespace std::chrono
     }
 
     // N.B. Leading slash as required by zoneinfo_file function.
-    const string tzdata_file = "/tzdata.zi";
-    const string leaps_file = "/leapseconds";
+    const string_view tzdata_file = "/tzdata.zi";
+    const string_view leaps_file = "/leapseconds";
 
 #ifdef _GLIBCXX_STATIC_TZDATA
 // Static copy of tzdata.zi embedded in the library as tzdata_chars[]

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-08-17 23:28 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-17 23:28 [gcc r14-3309] libstdc++: Replace global std::string objects in tzdb.cc 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).