On 22/12/12 05:42, Sam Spilsbury wrote: > Be careful when making changes like these, especially when the > compiler is warning that you are no longer using data you once did. In > addition, when doing refactoring, please try and do either of the > following: > > 1. Add tests for the unit under test (manual test do not count, and > use coverage tools to make sure your tests actually run the code you > think they do) > 2. Write a sample program excersizing the system's behaviour with that > function (this applies especially to toolkits) > 3. Run the system under test with a coverage build to make sure that > the changes you've made are actually being excersized. > > The other thing to consider is that when writing code which has that > one crucial line required to tie the whole thing together, put a > comment there to say what it does so that other people don't > accidentally remove it.
Thanks Sam, good catch!
signature.asc
Description: OpenPGP digital signature
-- Mailing list: https://launchpad.net/~unity-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~unity-dev More help : https://help.launchpad.net/ListHelp
