(Desperate) Plea for multi-person code review
Nick Mathewson
nickm at freehaven.net
Mon Feb 15 17:22:06 UTC 2010
On Sun, Feb 14, 2010 at 9:09 PM, Mike Perry <mikeperry at fscked.org> wrote:
> I've now rebased, split, and squashed all the commits into 8
> logically distinct patches in mikeperry/consensus-bw-weights4.
> Should be easier to review for those who prefer to look at things
> one commit at a time.
>
Ouch. This is great for people who want to read your patch series de
novo, and it's good for clarity once we merge it into Tor's main
history, but it makes stuff harder for people who've been reviewing
the old patch series unless you say something like, "BTW, commit X in
consensus-bw-weights4 corresponds to exactly the same changes as you
had looked at up until now in consesus-bw-weights3, just cleaned up a
little."
thanks,
--
Nick
More information about the tor-dev
mailing list