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: symbolic debug of loadable modules with kgdb light


Here is the updated patch.
- Use Ctrl-C, BREAK and BREAK-g
- Use tab only in gdb/ChangeLog.
- Add 2 spaces after period.
- cmd_name is initialized on each lookup_cmd().
- add_setshow_boolean_cmd ("remotebreak", ...) has no change except for
calling set_remotebreak and show_remotebreak
- Tested. No regression.
-caz

-----Original Message-----
From: Pedro Alves [mailto:pedro@codesourcery.com] 
Sent: Thursday, October 01, 2009 3:14 PM
To: Daniel Jacobowitz
Cc: Joel Brobecker; Caz Yokoyama; 'Eli Zaretskii';
gdb-patches@sourceware.org
Subject: Re: symbolic debug of loadable modules with kgdb light

On Thursday 01 October 2009 22:10:15, Daniel Jacobowitz wrote:
> On Thu, Oct 01, 2009 at 01:46:24PM -0700, Joel Brobecker wrote:
> > > Actually I support your spellings. Ctrl-C is much popular than
control-c,
> > > BREAK is popular than break for break signal. SysRq-g should be
BREAK-g as
> > > Joel said.
> > 
> > Your arguments make sense and I am fine with your choices. But we
generally
> > use lower-case names for the various GDB settings, so do double-check
> > with Daniel J and Pedro that they are also fine with these choices.
> 
> I have no opinion (and think we've spent more time on these names than
> they required - let's just do it).

Ditto.

-- 
Pedro Alves

Attachment: remotebreak.patch
Description: Binary data


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