Re: [Xen-devel] [PATCH 14/15] libxl: New API for providing OS events to libxl

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Stefano Stabellini writes ("Re: [Xen-devel] [PATCH 14/15] libxl: New API for 
providing OS events to libxl"):
> On Fri, 9 Dec 2011, Ian Jackson wrote:
> > Firstly, Stefano, please trim your quotes.  You don't need to quote
> > the whole zillion-line patch.  Just quote the bits that are relevant.
> > Otherwise people may miss your comments as they page through trying to
> > find them.
> 
> I personally prefer to keep the full quote so that I can search for
> references without having to go back to the other email. However I do
> understand that some people don't like this so I'll trim my comment to
> your patches in the future.

Everyone on the list needs to be able to read these review comments,
not just the author of the original patch.  I see Ian C agrees with
me.  The standard approach should always be to trim the patch to the
relevant parts only.

Thanks,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel