This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA v2 12/17] Remove make_cleanup_restore_current_uiout

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
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]