On 25/02/2013 15:55, Mike Duigou wrote:
:

The webrev suggests you've replaced Map/Collisions.java and assume that is not 
the intention. Otherwise the new test looks okay, an alternative would have 
been to expand MOAT.
Webrev doesn't properly recognize 'hg copy' and sees it as a rename. I started 
ToArray.java by copying Collisions.java

Are you sure you want to do "hg copy" here? This means that hg will track the copy and I assume we don't want that here.

-Alan.

Reply via email to