Code review comment for lp:~beuno/loggerhead/bundles

On Tue, Aug 12, 2008 at 9:27 PM, Michael Hudson
<email address hidden> wrote:
> I think the conditions and the diff links should be done by inspecting
> the change object -- show the diff against the left-most parent of the
> change, something like 'h.get_revno(change.parents[0].revid)', rather
> than the navigation. The effect is the same, but this would cause fewer
> wtfs :)

Done.

> Hm, actually I think the diff should be to just diff/<revno> in the
> default case and to diff/<compare_revid>/<cur revno> when there is a
> compare_revid -- this should be an easy change to make, I think.

Done.

Thanks for the review!

--
Martin