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: Sergio Durigan Junior <sergiodj at redhat dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: Yao Qi <yao at codesourcery dot com>, gdb-patches at sourceware dot org
- Date: Mon, 01 Jul 2013 08:19:11 -0300
- Subject: Re: [patch 1/2] Code cleanup: remote.c *->{async,sync}*
- References: <20130630181059 dot GA29548 at host2 dot jankratochvil dot net> <51D150A9 dot 6070806 at codesourcery dot com> <20130701111056 dot GA30121 at host2 dot jankratochvil dot net>
On Monday, July 01 2013, Jan Kratochvil wrote:
> A matter of opionion. While debugging the sync and async parts I had to write
> a help text along which function gets used in which mode. Which I find the
> {a,}sync_* prefies easier, moreover where already some functions/variables
> contain *{a,}sync_* therein.
FWIW, I have looked at this code a lot during the last weeks, and I
agree that renaming the functions along with more documentation would
help future hackers.
--
Sergio