Re: [RFA v2 12/17] Remove make_cleanup_restore_current_uiout

Pedro Alves
2016-10-20T21:57:47Z

On 10/20/2016 10:46 PM, Tom Tromey wrote:
> Pedro> I wonder if you had a reasoning that led to deciding when
> Pedro> to use two-argument form? A couple places looked like obvious
> Pedro> candidates. My reasoning would be: if the assignment is right
> Pedro> after the scoped_restore creation.
>
> Tom> I tried to use them everywhere, but I just forgot to restore it in this
> Tom> patch when I added the two-arg form back. I'll fix this up.
>
> Here's the updated patch.

OK.

Thanks,
Pedro Alves