[PATCH] D33029: [clang-format] add option for dangling parenthesis

Ryan Stringham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 10 10:44:50 PDT 2017


stringham added a comment.

1. be used in a project of significant size (have dozens of contributors)
  - We are using this at Lucid Software (https://www.lucidchart.com & https://www.lucidpress.com) to format our JS and TS (we are using clang-format to automatically style over 650,000 lines of code)
  - It's not open source, but we do have dozens of contributors

2. have a publicly accessible style guide
  - We don't have a publicly available style guide, but here is Airbnb's Javascript Style guide's style definition which uses the dangling parens:
    - https://github.com/airbnb/javascript#functions--signature-invocation-indentation
    - I think this counts as a project of significant size with a publicly available style guide which would use this style rule.

3. have a person willing to contribute and maintain patches
  - Me
  - We currently have a fork of clang on github (https://github.com/lucidsoftware/clang) with this patch applied


https://reviews.llvm.org/D33029





More information about the cfe-commits mailing list