[webkit-dev] style question for empty for loops

David Levin levin at chromium.org
Thu Aug 26 13:18:15 PDT 2010


On Thu, Aug 26, 2010 at 1:12 PM, Chris Fleizach <cfleizach at apple.com> wrote:

> webkit-check-style should probably be amended as well
>

Please file a bug. Feel free to cc hamaji at chromium.org, levin at chromium.org,
cjerdonek at webkit.org



>
>
> On Aug 26, 2010, at 12:48 PM, James Robinson wrote:
>
> The style guide currently covers this
> http://webkit.org/coding/coding-style.html:
>
> 4. Control clauses without a body should use empty braces: Right:
>
> for ( ; current; current = current->next) { }
>
> Wrong:
>
> for ( ; current; current = current->next);
>
>
> - James
>
> On Thu, Aug 26, 2010 at 12:22 PM, Chris Fleizach <cfleizach at apple.com>wrote:
>
>>
>> On 26. aug. 2010, at 11.49, Darin Adler wrote:
>>
>> > On Aug 26, 2010, at 11:35 AM, Chris Fleizach wrote:
>> >
>> >>    for (...; ...; ...) { }
>> >
>>
>> So maybe this is the best option. I can add a style guide check for that,
>> unless there are objections
>>
>> _______________________________________________
>> webkit-dev mailing list
>> webkit-dev at lists.webkit.org
>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>>
>
>
>
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20100826/2bfe1fdf/attachment.html>


More information about the webkit-dev mailing list