LegalizeAdulthood/ClangContributions.md

Last active May 22, 2024 18:08

Star () You must be signed in to star a gist
Fork () You must be signed in to fork a gist

Learn more about clone URLs
Clone this repository at <script src="https://gist.github.com/LegalizeAdulthood/933c867c563303ce047f.js"></script>
Save LegalizeAdulthood/933c867c563303ce047f to your computer and use it in GitHub Desktop.

Download ZIP

My Clang Contributions

Raw

ClangContributions.md

My Clang Contributions

clang tooling library:
- AST matchers:
  - extend hasType to work with TypedefDecl nodes
  - add functionProtoType matcher for matching FunctionProtoType nodes
  - extend parameterCountIs matcher to FunctionProtoType nodes
clang-tidy checks:
- modernize-macro-to-enum converts groups of macros to unscoped anonymous enums
- modernize-raw-string-literal converts string literals with escaped characters to raw string literals
- modernize-redundant-void-arg removes void tokens from T f(void) function declarations, definitions, typedefs, pointers to functions, etc.
- readability-duplicate-include looks for duplicate includes
- readability-redundant-control-flow eliminates redundant return statements in functions returning void and continue statements in loops
- readability-simplify-boolean-expr simplifies boolean expressions

In Review

clang-tidy improvements:
- allow tests to verify changes to headers

To Do

clang-tidy checks:
- readability-redundant-fwd-decl check
- readability-redundant-control-flow
  - Improve test case
  - Handle redundant control flow statements inside an if statement at the end of a control flow scope
- readability-simplify-boolean-expr
  - Apply fixes to header files
  - Handle macro expansion better (iterated-dynamics case)
  - Simplify !(a < b) to (a >= b) and similar for other comparison operators
  - Improve handling of integral types implicitly converted to bool (case with std::vector::size_type implicitly converted)
- modernize-redundant-void-arg
  - Apply fixes to header files
  - Don't apply fixes to macros
- modernize-raw-string-literal
  - Enhance to support non-ASCII string literals: wchar_t, char16_t, char32_t, u8"", u"", U"", L""

Accepted Submissions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment