Summary: | Add punctuation rules for C++ lambdas | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||
Component: | New Bugs | Assignee: | Ryosuke Niwa <rniwa> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ross.kirsling, webkit-bug-importer, zalan | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Ryosuke Niwa
2019-11-08 13:56:02 PST
Created attachment 383160 [details]
Add punctuation rules for C++ lambdas
Comment on attachment 383160 [details] Add punctuation rules for C++ lambdas View in context: https://bugs.webkit.org/attachment.cgi?id=383160&action=review > Websites/webkit.org/code-style.md:900 > +[](#punctuation-omit-lambda-paren) Omit parantheses for a C++ lambda whenever possible nit: parantheses -> parentheses (In reply to Ross Kirsling from comment #2) > Comment on attachment 383160 [details] > Add punctuation rules for C++ lambdas > > View in context: > https://bugs.webkit.org/attachment.cgi?id=383160&action=review > > > Websites/webkit.org/code-style.md:900 > > +[](#punctuation-omit-lambda-paren) Omit parantheses for a C++ lambda whenever possible > > nit: parantheses -> parentheses (Also a missing period.) (In reply to Ross Kirsling from comment #3) > (In reply to Ross Kirsling from comment #2) > > Comment on attachment 383160 [details] > > Add punctuation rules for C++ lambdas > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=383160&action=review > > > > > Websites/webkit.org/code-style.md:900 > > > +[](#punctuation-omit-lambda-paren) Omit parantheses for a C++ lambda whenever possible > > > > nit: parantheses -> parentheses > > (Also a missing period.) Fixed. Committed r252395: <https://trac.webkit.org/changeset/252395> |