<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 23, 2020 at 00:20 Pavel Labath <<a href="mailto:pavel@labath.sk">pavel@labath.sk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On 22/07/2020 22:56, Jonas Devlieghere via lldb-commits wrote:<br>
> -  ConstString operator=(ConstString rhs) {<br>
> +  ConstString operator=(const ConstString &rhs) {<br>
<br>
ConstString is trivially copyable (well, it has a user defined copy<br>
constructor but it is trivial...).<br>
<br>
Plus, not so long ago (D59030) we changed all `const ConstString &`s to<br>
plain `ConstString`. It would be good to avoid flipping this back and<br>
forth all the time.</blockquote><div dir="auto"><br></div><div dir="auto">Thanks for reminding me, I had totally forgotten about that patch. I’ll take care of that tomorrow. </div></div></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature">Sent from my iPhone</div>