Re: [PATCH 2/7] Eliminate the two-level data structures behind location_specs

[Pedro Alves <pedro@palves.net>]

On 2022-05-27 19:02, Pedro Alves wrote:
> +	  std::unique_ptr<explicit_location_spec> els
> +	    (new explicit_location_spec ());
> +	  els->source_filename
> +	    = xstrdup (symtab_to_filename_for_display (sal2.symtab));
> +	  els->line_offset.offset = b->loc->line_number;
> +	  els->line_offset.sign = LINE_OFFSET_NONE;
> +
> +	  b->locspec.reset (els.release ());

I realized after sending the patch that even though els's type is
std::unique_ptr<explicit_location_spec>, and b->locspec is
a std::unique_ptr<location_spec>, we can do a move instead of
the reset/release above, like so:

 	  b->locspec = std::move (els);

I've made that change locally, here and in a couple other spots that could
do it.