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 for invalid hw breakpoints


> Date: Fri, 02 Jun 2006 08:25:59 +0100
> From: Nathan Sidwell <nathan@codesourcery.com>
> CC: Eli Zaretskii <eliz@gnu.org>,  gdb-patches@sourceware.org
> 
> I suspect that there's actually another failure mode which my patch fixes and it 
> would be hard for remote.c to fix, and that is when the watched expression 
> inserts more than one hardware watchpoint.  The current code will attempt to 
> insert multiple hw watchpoints, and remote.c will signal an internal error on 
> inserting the second one
> 
> -> Z2,11110000
> <- [empty: I don't support that.]
> tries to send: -> Z2,abcdef
> [internal error, I already know I don't support that!]

One more reason not to throw an internal error in remote.c, if you ask
me.  Low-level interfaces have no business doing such things, IMHO.


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