unity-dev team mailing list archive

[Mark Shuttleworth <mark@xxxxxxxxxx>]

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!

Attachment: signature.asc
Description: OpenPGP digital signature