xyris icon indicating copy to clipboard operation
xyris copied to clipboard

Update Code to Reflect Style Guide

Open Kfeavel opened this issue 2 years ago • 1 comments

Areas of Concern:

  • [ ] Namespaces (CamelCase)
  • [ ] Variable naming (lowerCamelCase)
  • [ ] Functions that should be static
  • [ ] Missing function prototypes
  • [ ] Single-line while loops should use continue (i.e. while (bool) continue;)
    • Add AllowShortLoopsOnASingleLine: true to .clang-format
  • [ ] Visit Clang Format Docs and check for other options to enable

These can be separate pull requests, so format the branch names as 287-code-cleanup-(insert category here)

Kfeavel avatar Aug 07 '21 15:08 Kfeavel

---
Language: Cpp
BasedOnStyle: WebKit

NamespaceIndentation: None
IndentPPDirectives: AfterHash
SpaceAfterTemplateKeyword: false
BraceWrapping:
    AfterFunction: true

BreakBeforeInheritanceComma: true
BreakConstructorInitializers: BeforeComma
BreakBeforeBraces: Custom

AllowShortBlocksOnASingleLine: Always

AlignConsecutiveAssignments: AcrossComments
AlignConsecutiveBitFields: AcrossEmptyLinesAndComments
AlignConsecutiveMacros: AcrossComments
AlignTrailingComments: true
AlignEscapedNewlines: Left
AlignOperands: AlignAfterOperator

Kfeavel avatar Aug 09 '21 01:08 Kfeavel