Comment 20 for bug 1197921
Revision history for this message
|
|
#20 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
1b1ed1a
(Get the code!)
Thanks for the test case, that's much appreciated. Just a few comments:
- We might as well extend the existing trap-crasher test instead of adding a new one that is a near duplicate.
- The test should ideally be added in its own commit before the fix so that it's easy to verify that the fix actually fixes the test case
- We need all patches to go through the mailing list, including this updated one. I have added you to the mailman whitelist, so it won't get stuck in moderation if you send it.