This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch 1/2] Code cleanup: remote.c *->{async,sync}*
- From: Pedro Alves <palves at redhat dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Fri, 02 Aug 2013 12:06:33 +0100
- Subject: Re: [patch 1/2] Code cleanup: remote.c *->{async,sync}*
- References: <20130630181059 dot GA29548 at host2 dot jankratochvil dot net>
On 06/30/2013 07:10 PM, Jan Kratochvil wrote:
> Code cleanup.
> * remote.c (cleanup_sigint_signal_handler): Rename the declaration
> to ...
> (async_cleanup_sigint_signal_handler): ... this.
> (initialize_sigint_signal_handler): Remove declaration.
> (handle_remote_sigint): Rename the declaration to ...
> (async_handle_remote_sigint): ... this.
> (handle_remote_sigint_twice): Rename the declaration to ...
> (async_handle_remote_sigint_twice): ... this.
> (async_remote_interrupt, async_remote_interrupt_twice)
> (remote_interrupt): Remove the declarations.
> (remote_interrupt_twice): Rename the declaration ...
> (sync_remote_interrupt_twice): ... this.
> (sigint_remote_twice_token): Rename the variable to ...
> (async_sigint_remote_twice_token): ... this.
> (sigint_remote_token): Rename the variable to ...
> (async_sigint_remote_token): ... this.
> (initialize_sigint_signal_handler): Rename the function to ...
> (async_initialize_sigint_signal_handler): ... this. Update the name
> inside.
> (handle_remote_sigint): Rename the function to ...
> (async_handle_remote_sigint): ... this. Update the names inside.
> (handle_remote_sigint_twice): Rename the function to ...
> (async_handle_remote_sigint_twice): ... this. Update the names inside.
> (cleanup_sigint_signal_handler): Rename the function to ...
> (async_cleanup_sigint_signal_handler): ... this.
> (remote_interrupt): Rename the function to ...
> (sync_remote_interrupt): this. Update the names inside.
> (remote_interrupt_twice): Rename the function to ...
> (sync_remote_interrupt_twice): this. Update the names inside.
> (remote_terminal_inferior, remote_terminal_ours, remote_wait_as)
> (_initialize_remote): Update the names inside.
This is fine with me.
Thanks,
--
Pedro Alves