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: (gdb-6.8) Discard breakpoint address if shared library is unloaded


Nick,

> 2008-02-15  Nick Roberts  <nickrob@snap.net.nz>
> 
> 	* breakpoint.c (print_one_breakpoint_location): Revert Enb field
> 	to old format.  Discard breakpoint address if shared library is
> 	unloaded.
> 	(breakpoint_1): Adjust formatting of table header accordingly.

The patch looks fine, but I assumed that you had run the testsuite
before sending it. From my end, it looks like you didn't, and I find
this quite disappointing. Based on Volodya's email, I ran the testsuite
myself (which I really shouldn't have to), and discovered that your
patch causes ~20 new failures, not just the 3 that Volodya discovered.

Your patch cannot go in until the failures are investigated and fixed.

Please make sure to always confirm in your submissions that you did
run the testsuite, mentioning which architecture it was run on, and
also confirm that it did not introduce any regression.

-- 
Joel


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