On Nov 1, 2010, at 10:00 AM, David Hyatt wrote:
> Yeah I agree with Peter. I think blank lines after { and before } would
> improve the readability of the 2nd example even without indentation.
>
> namespace WebCore {
>
> class AuthenticationChallenge;
> class CachedFrame;
> class HistoryItem;
> class ProtectionSpace;
> class ResourceLoader;
> class ResourceRequest;
>
> }
I agree this also makes it more readable, but...
> This doesn't seem worth making a special case for to me.
If the general sentiment is "I don't have a strong preference either way", then
I would argue that we should allow its continued used simply because so much
code already uses it.
~Brady
>
> dave
> ([email protected])
>
> On Nov 1, 2010, at 11:49 AM, Peter Kasting wrote:
>
>> On Mon, Nov 1, 2010 at 9:40 AM, Brady Eidson <[email protected]> wrote:
>> I think this pattern increases readability of forward declarations in
>> headers and we should change the style guidelines to specify its continued
>> use.
>>
>> Thoughts?
>>
>> I don't find either one significantly better than the other, personally.
>>
>> PK
>> _______________________________________________
>> webkit-dev mailing list
>> [email protected]
>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
_______________________________________________
webkit-dev mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
