On Wed, Sep 10, 2008 at 01:41:06AM +0200, Miklos Vajna wrote: > test_expect_success \ > 'validate git-rev-list output.' \ > - 'diff current expected' > + 'test_cmp current expected' We seem to use the convention of test_cmp <expected> <actual> elsewhere, rather than test_cmp <actual> <expected> as you have here. Most noticeably, that means the diff will show deviations from expected, rather "what should be done to make this as expected". But it is also possible that in the future test_cmp could be enhanced to do something more interesting. I don't think it is worth it to go fix all such instances, but while you are here... -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
