procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

libstdc++: Replace global std::string objects in tzdb.cc

Browse source 
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.
This commit is contained in:
Jonathan Wakely 2023-08-17 20:39:02 +01:00
parent aad83d61d2
commit d82a85b616
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
libstdc++-v3/src/c++20/tzdb.cc
View file

@ -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[]