Re: [RFA v2 12/17] Remove make_cleanup_restore_current_uiout

Pedro Alves
2016-10-13T22:49:32Z

On 10/13/2016 10:10 PM, Tom Tromey wrote:

> - print_stop_location (&last);
> + {
> + scoped_restore save_uiout = make_scoped_restore (&current_uiout);
> + current_uiout = uiout;

I wonder if you had a reasoning that led to deciding when
to use two-argument form? A couple places looked like obvious
candidates. My reasoning would be: if the assignment is right
after the scoped_restore creation.

Anyway, this is OK.

Thanks,
Pedro Alves