Re: [RFA v2 02/17] Use scoped_restore for ui_file

Tom Tromey
2016-10-14T21:33:25Z

>>>>> "Pedro" == Pedro Alves <palves@redhat.com> writes:

Pedro> I wonder if aligning like:
Pedro> scoped_restore save_stdout
Pedro> = make_scoped_restore (&gdb_stdout, str_file);
Pedro> scoped_restore save_stderr
Pedro> = make_scoped_restore (&gdb_stderr, str_file);
Pedro> scoped_restore save_stdlog
Pedro> = make_scoped_restore (&gdb_stdlog, str_file);
Pedro> scoped_restore save_stdtarg
Pedro> = make_scoped_restore (&gdb_stdtarg, str_file);
Pedro> scoped_restore save_stdtargerr
Pedro> = make_scoped_restore (&gdb_stdtargerr, str_file);
Pedro> would read better. Can't tell on the mail client.
Pedro> Up to you; just a thought.

I think so. I went ahead and made this change.

Tom