Patch frustration

[Na'Tosha Bard]

On Thu, Jun 7, 2012 at 9:05 PM, Matt Mackall <mpm at selenic.com> wrote:

> On Thu, 2012-06-07 at 19:00 +0000, David Soria Parra wrote:
> > On 2012-06-07, Matt Mackall <mpm at selenic.com> wrote:
> > > On Wed, 2012-06-06 at 17:31 -0700, Bryan O'Sullivan wrote:
> > >> On Wed, Jun 6, 2012 at 5:19 PM, Adrian Buehlmann <adrian at cadifra.com>
> wrote:
> > >>
> > > If you want lower latency, you have to let the queue empty, folks. If
> > > you want more throughput, you need to start reviewing other people's
> > > patches.
> > >
> > I'd like to help with that but so far I avoided reviewing it, as my
> understandig
> > is, either Matt or Crew reviews it. there is no point in other people
> > stepping up.
>
> If people with the relevant expertise review and ACK patches, other
> people can sweep them up.


Do you prefer if people comment on patches saying they think they are good?
 I've always followed the "only say something if I have reason to believe
the patch is bad" policy, and suspect most others do as well.

Na'Tosha
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20120607/42f5fcf2/attachment.html>