[gdb/cli] Skip string copy in source_cache::ensure
authorTom de Vries <tdevries@suse.de>
Tue, 17 Oct 2023 09:38:06 +0000 (11:38 +0200)
committerTom de Vries <tdevries@suse.de>
Tue, 17 Oct 2023 09:38:06 +0000 (11:38 +0200)
commit7e5649156720ec6c3aa5e2036b27811bc8195ab9
treedd39720a56af95be020e6aea1b393377e1465bdf
parent3233ad1e4e44530fa1088cef4790bdab6ee14944
[gdb/cli] Skip string copy in source_cache::ensure

In function source_cache::ensure we have:
...
        std::ostringstream output;
      ...
      contents = output.str ();
...
The last line causes an unnecessary string copy.

C++20 allows us to skip it, like this:
...
      contents = std::move (output).str ();
...

Use the more efficient solution.

Tested on x86_64-linux.

Reviewed-By: Lancelot Six <lancelot.six@amd.com>
gdb/source-cache.c