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