Re: [Qemu-devel] [PATCH 0/5] CODING_STYLE amendments

[Anthony Liguori <anthony@codemonkey.ws>]

On 08/22/2010 03:09 PM, Avi Kivity wrote:
>  On 08/22/2010 09:56 PM, Blue Swirl wrote:
>>
>>> Can someone point to a bug in QEMU that's been caused because of
>>> CODING_STYLE or the fact that some patches don't adhere to it?
>> 7b1df88f284f462ecb236931ad863a815f243195
>
> How was this bug caused by CODING_STYLE?  In fact, if CODING_STYLE was 
> applied correctly then this bug would stand out much more:
>
>     if (...) {
>         /* .... */;
>     }
>         return;

"or the fact that some patches don't adhere to it".  If CODING_STYLE was 
enforced at commit time, this bug would not have happened most likely.

But it's such an odd case that I'd say it's just the exception that 
proves the rule that a loose CODING_STYLE isn't significantly impacting 
overall quality.

I also don't believe that it's creating distress in contributors.  I 
believe timely patch review/commit is probably a more important issue 
for contributors than whether some people get by without CODING_STYLE 
being strictly enforced.

Regards,

Anthony Liguori

>
> or be completely eliminated:
>
>     if (...) {
>         /* .... */;
>         return;
>     }
>
>