Comment 9 for bug 389413

I've decided that a good immediate fix is to apply @need_read_lock to RemoteBranch.get_rev_id, as this is consistent with Branch.get_rev_id.

There's a broader issue that we really shouldn't be repeatedly locking and unlocking these objects just to do diff and log, but that's a separate problem to making it work.

Also, Benoit PIERRE @5: I think you have a different bug to everyone else on this bug report, as yours doesn't involve the smart server. I can reproduce it, so I'll file a separate bug about it.