<div dir="ltr"><div dir="ltr"><div>There has been no more votes either way so no space wins. Here's a patch to codify it in our code style guidelines: <a href="https://bugs.webkit.org/show_bug.cgi?id=204021">https://bugs.webkit.org/show_bug.cgi?id=204021</a></div><div><br></div><div>- R. Niwa</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Nov 2, 2019 at 8:26 PM Ryosuke Niwa <<a href="mailto:rniwa@webkit.org" target="_blank">rniwa@webkit.org</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Nov 2, 2019 at 10:16 AM Caitlin Potter <<a href="mailto:caitp@igalia.com" target="_blank">caitp@igalia.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="auto">Not that anybody asked me, but I also prefer to not include a space between captures and parameter, for similar reasons.<div><br></div><div>If I’m not mistaken, v8/chromium tends to omit the space as well. If that’s still true and WebKit adopted that style, context switching between both codebases would be marginally easier for me.<br><div dir="ltr"><br><blockquote type="cite">On Nov 2, 2019, at 4:19 AM, Antti Koivisto <<a href="mailto:koivisto@iki.fi" target="_blank">koivisto@iki.fi</a>> wrote:<br><br></blockquote></div><blockquote type="cite"><div dir="ltr"></div></blockquote></div></div><div dir="auto"><div><blockquote type="cite"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Nov 1, 2019 at 10:50 PM Yusuke Suzuki <<a href="mailto:ysuzuki@apple.com" target="_blank">ysuzuki@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
> On Nov 1, 2019, at 11:53, Michael Catanzaro <<a href="mailto:mcatanzaro@gnome.org" target="_blank">mcatanzaro@gnome.org</a>> wrote:<br>
> <br>
> On Fri, Nov 1, 2019 at 11:19 am, Ryosuke Niwa <<a href="mailto:rniwa@webkit.org" target="_blank">rniwa@webkit.org</a>> wrote:<br>
>> Namely, some people write a lambda as:<br>
>> auto x = [] () { }<br>
>> with a space between [] and () while others would write it as:<br>
>> auto x = []() { }<br>
> <br>
> 🔧: I omit the () when there are no parameters, as in these examples.<br>
> <br>
> No preference on spacing.<br>
<br>
I like having a space here, because this rule is simpler to me.<br>
If we always have a space between them, this is clear that the above case is written in `[] { }` instead of `[]{ }`.<br></blockquote><div><br></div><div>I prefer not having the redundant space in [](). It also makes logical sense to me to keep the lambda signature together. I started using lambdas with space there, dropped it later, and suffered no adverse consequences.</div><div><br></div><div>As for existing practice, WebCore favors spaceless ]( about 2:1 but across the entire WebKit it is closer to 1:1.</div><div><br></div><div>We always put space before { } block, I don't think that is really in question here, or creating any inconsistencies.</div><div><br></div><div><br></div><div>   antti</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
-Yusuke<br>
<br>
> <br>
> <br>
> _______________________________________________<br>
> webkit-dev mailing list<br>
> <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
> <a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
<br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
</blockquote></div></div></div>
<span>_______________________________________________</span><br><span>webkit-dev mailing list</span><br><span><a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a></span><br><span><a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a></span><br></div></blockquote></div></div>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
</blockquote></div></div>-- <br><div dir="ltr">- R. Niwa</div>
</blockquote></div></div></div>