[Code Style] Use uniform initializer syntax in member initializer list

• Component: WebKit Website
• Status: RESOLVED WONTFIX

[Daniel Bates]

<https://lists.webkit.org/pipermail/webkit-dev/2017-August/029471.html>

[Daniel Bates]

Created attachment 319261 [details]
Patch

[Daniel Bates]

Comment on attachment 319261 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=319261&action=review

> Websites/webkit.org/code-style.md:-763
> -    : MySuperClass()

We also prefer to omit calling the default base class constructor (as the compiler will emit code to do this for us). I did not change the example to reflect this. Let me know if I should.

[Konstantin Tokarev]

Comment on attachment 319261 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=319261&action=review

>> Websites/webkit.org/code-style.md:-763
>> -    : MySuperClass()
> 
> We also prefer to omit calling the default base class constructor (as the compiler will emit code to do this for us). I did not change the example to reflect this. Let me know if I should.

It may make sense to add arguments to base class ctor to avoid cofusion

[Darin Adler]

Comment on attachment 319261 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=319261&action=review

>>> Websites/webkit.org/code-style.md:-763
>>> -    : MySuperClass()
>> 
>> We also prefer to omit calling the default base class constructor (as the compiler will emit code to do this for us). I did not change the example to reflect this. Let me know if I should.
> 
> It may make sense to add arguments to base class ctor to avoid cofusion

I think we should.

[Daniel Bates]

There appears to be no ❤️ to solidify this in the style guide. Makes me 😢. Maybe one day. Let's just move on.