gerrit lame / non-reviews ...

Hi guys,

On Mon, 2012-07-09 at 10:28 +0000, Gerrit wrote:
> Björn Michaelsen has posted comments on this change.

        Let me pick on Bjoern because he is a big guy :-)

> Change subject: renamed German variables (in sw/source/core/crsr/)
> ......................................................................
> Patch Set 1: Looks good to me, but someone else must approve

        Until we can either turn down the gerrit fire-hose; or work out some
way of improving the flow:

        *please* have some consideration for the people on this list.

        If you would not send a content-free mail of a dozen lines saying "me
too" - then please do not do bogus/empty "looks good to me but doesn't
close or merge the patch" type reviews.

        If you agree a patch looks good; just merge it. If you like the look of
it but are not sure - please consider staying quiet for now :-) I (for
one) don't want to have a +1 infested Apache-style list full of poorly
quoted context and almost no signal :-)

        If a review has been done, and the patch is ok lets merge the patch :-)
it avoids someone else having to do that work, and worse a ton of people
having to read their mailbox full of big mail with ~identical subjects
many with banal content.

        :-)

        Thanks !

                Michael.

-- 
michael.meeks@suse.com  <><, Pseudo Engineer, itinerant idiot