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: [patch 1/3] make prompt_for_continue call throw_quit directly


On 12/18/2015 07:30 PM, Sandra Loosemore wrote:
> This is the patch previously suggested by Pedro here:
> 
> https://sourceware.org/ml/gdb-patches/2015-09/msg00637.html
> 
> to fix a problem I ran into when testing on a remote Windows host via 
> SSH without a terminal or job control.  As Pedro said, when typing 'q' 
> in response to a pagination prompt, there's no possibility of a SIGINT 
> so we can bypass the parts of quit() handling that are intended to 
> intercept ^C instead.
> 
> I confirmed that this fixes the bogus quit message I observed previously 
> on Windows host, and did full regression testing on Linux host for 
> nios2-linux-gnu target.
> 
> OK to commit?

OK.  Could you add a little comment to prevent someone from deciding later
that we should using quit() instead?

Thanks,
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]