-
Notifications
You must be signed in to change notification settings - Fork 63
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: initial changes for clang-tidy recommendations, clang-format, a…
…nd use of safe_ functions A lot of initial changes to implement a clang-format and clang-tidy files and the recommended changes to reduce possibility of bugs and improve memory safety. Signed-off-by: Tyler Erickson <[email protected]>
- Loading branch information
1 parent
607e985
commit 2805734
Showing
45 changed files
with
12,524 additions
and
9,229 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
# Use the Microsoft coding style | ||
# This is the style our code already mostly followed-TJE | ||
BasedOnStyle: Microsoft | ||
|
||
# Additional Customizations | ||
PointerAlignment: Left | ||
AlignConsecutiveAssignments: true | ||
AlignConsecutiveDeclarations: true | ||
AlignTrailingComments: true | ||
AlignConsecutiveMacros: true | ||
AlignAfterOpenBracket: true | ||
BinPackParameters: false | ||
AllowAllParametersOfDeclarationOnNextLine: false | ||
AlignConsecutiveMacros: 'AcrossEmptyLines' | ||
BraceWrapping: | ||
AfterControlStatement: true | ||
AllowShortBlocksOnASingleLine: Never | ||
AllowShortIfStatementsOnASingleLine: Never | ||
IndentPPDirectives: AfterHash | ||
BreakBeforeBraces: Allman |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
Checks: '-*,modernize-*,cert-*,bugprone-*,-bugprone-easily-swappable-parameters' | ||
#other available checks: clang-analyzer-*, performance-*, readability-*, misc-* | ||
#Disabled checks: | ||
# -bugprone-easily-swappable-parameters - warns a LOT. Sometimes on functions meant to look like standardized C11 annex k functions | ||
# While a overall useful thing to consider, it's too noisy right now and it may make the API | ||
# harder to use to resolve all of these. | ||
WarningsAsErrors: '' | ||
HeaderFilterRegex: '.*' | ||
AnalyzeTemporaryDtors: false | ||
FormatStyle: 'file' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.