----- Original Message ----- > From: "Stefan Hajnoczi" <[email protected]> > To: "Federico Simoncelli" <[email protected]> > Cc: [email protected], [email protected], [email protected] > Sent: Thursday, February 23, 2012 5:18:41 PM > Subject: Re: [Qemu-devel] [PATCH 1/3] Add blkmirror block driver > > On Thu, Feb 23, 2012 at 4:14 PM, Stefan Hajnoczi <[email protected]> > wrote: > > By the way, this code does not build against qemu.git/master. It > > uses > > interfaces which have been dropped. > > Ah, I see you changed that in Patch 2/3. Please don't do that, it's > hard to review and breaks git-bisect.
Squashing is easy (anyone could do it at the commit phase), reviewing is hard (if you hide your changes in someone else's code). -- Federico
