zebreus/lix
0
0
Fork 0
forked from lix-project/lix
Code Pull requests 1 Activity

Fix tests on systems with a non-master git defaultBranch #1

Open
zebreus wants to merge 140 commits from fix-tests-without-master into main
pull from: fix-tests-without-master
merge into: zebreus:main
Conversation 0 Commits 140 Files changed 216 +11 -1
1 changed files with 11 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 644176a631 - Show all commits

12
src/libutil/strings.hh
View file

@ -200,8 +200,18 @@ std::string showBytes(uint64_t bytes);
/** /**
* Provide an addition operator between strings and string_views * Provide an addition operator between `std::string` and `std::string_view`
* inexplicably omitted from the standard library. * inexplicably omitted from the standard library.
*
* > The reason for this is given in n3512 string_ref: a non-owning reference
* to a string, revision 2 by Jeffrey Yasskin:
* >
* > > I also omitted operator+(basic_string, basic_string_ref) because LLVM
* > > returns a lightweight object from this overload and only performs the
* > > concatenation lazily. If we define this overload, we'll have a hard time
* > > introducing that lightweight concatenation later.
*
* See: https://stackoverflow.com/a/47735624
*/ */
inline std::string operator + (const std::string & s1, std::string_view s2) inline std::string operator + (const std::string & s1, std::string_view s2)
{ {