Re: [RFA v2 16/17] Add a constructor and destructor to linespec_result

Tom Tromey
2017-04-12T14:30:42Z

>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:

Simon> On 2017-04-11 11:01, Tom Tromey wrote:
>> linespec_result is only ever allocated on the stack, so it's
>> relatively easy to convert to having a constructor and a destructor.
>> This patch makes this change. This removes some cleanups.

Simon> LGTM, though there are still some references in comments to
Simon> destroy_linespec_result that could be cleaned up.

I've changed these to refer to the linespec_result destructor instead.

Tom