codeinn/vcs — pull request #102 by lukaszb (2012-01-10 01:22:34+00:00)
Title: Implemented GitRepository.get_diff method - refers #96

This is first approach. I'm not happy with *EmptyChangeset*, maybe simple static at *GitRepository* or *GitChangeset*, *EMPTY_REVISION* (or something) being ``"0" * 40`` would suffice - this way at least we can assume that *get_diff* method can take only *hashes*/*integers*, not objects. We still would need to handle that specific case with empty changeset, but in a lighter way I believe.

Also, if you can, please take a look at new tests for *get_diff* method for mercurial backend (they are failing, and maybe you can fix them easily - if not, please write down your impression too as it may be harder to write generic tests for all backends :)).

Moreover, let's discuss implementation here, not at #96, so we can easily see what's going on in this branch.