Dan Kegel wrote: > On Mon, Aug 25, 2008 at 2:05 PM, Kai Blin <[EMAIL PROTECTED]> wrote: > >>>> #patchwatcher approve >>>> >>>> Such a directive would only be recognized if it's alone on a line. >>>> >>> The usual positive reply is 'ACK'. I believe it would be simplest to >>> grep the reply message for ACK to see if a reply is positive. >>> >> Dunno, I find myself replying to patches that fix bugs I worked on myself or >> that were in my code with things like "good catch". I agree with Alexandre >> that cases like that are seldom enough that we can probably ignore them. >> > > Around where I work, LGTM (looks good to me) is the reply that > denotes approval. And then there's the Apache convention of +1. > Anyway, I'll recognize one or more of those if and when I get around > to adding a wine-devel listener. Might be a couple weeks. > > Dan:
Thank you for adding this. James McKenzie