[RFA]: Fix to watchthreads test case

Jeff Johnston jjohnstn@redhat.com
Tue Oct 12 22:50:00 GMT 2004


Thanks Michael.  Patch checked in and tested on x86-linux and ia64-linux.

-- Jeff J.

Michael Chastain wrote:
> Jeff Johnston writes:
> 
>>After discussing with Andrew, it was suggested that watchthreads.exp be
>>given its own C file which is a copy of schedlock.c.
> 
> 
> Sounds good to me, I like it that way too.
> 
> It looks like I broke this.  :(  When I added a copyright notice to
> schedlock.c, I tested schedlock.exp, but I did not look for any other
> *.exp files that shared schedlock.c.  So I just plain broke it.
> 
> 
>>This new patch replaces the old one below.
>>Hopefully, someone will get a chance to look at it.
> 
> 
> I'm still catching up on my e-mail so maybe Michael Snyder already
> reviewed this patch.  But if not -- I approve this, if you say how
> you tested it.
> 
> Michael
> 
> ===
> 
> 2004-10-04  Jeff Johnston  <jjohnstn@redhat.com>
> 
> 	* gdb.threads/watchthreads.c: New file for use by watchthreads.exp. 
> 
> 	* gdb.threads/watchthreads.exp: Use gdb_get_line_number to find
> 	breakpoint lines and switch to use watchthreads.c.
> 



More information about the Gdb-patches mailing list