Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Format all files and add clang-format configuration file #241

Closed
SamVanheer opened this issue Nov 28, 2021 · 0 comments
Closed

Format all files and add clang-format configuration file #241

SamVanheer opened this issue Nov 28, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@SamVanheer
Copy link
Collaborator

Format all files to use the same layout. Currently code written at different times has a variety of formatting that makes reading code harder and more importantly makes searching for certain patterns of code more difficult, requiring complicated regular expressions to account for additional whitespace.

Use clang-format to configure the formatting settings in a way that ensures consistency between users.

See https://devblogs.microsoft.com/cppblog/clangformat-support-in-visual-studio-2017-15-7-preview-1/ and https://docs.microsoft.com/en-us/visualstudio/ide/reference/options-text-editor-c-cpp-formatting?view=vs-2019 for Visual Studio integration information.

See https://releases.llvm.org/13.0.0/tools/clang/docs/ClangFormatStyleOptions.html for ClangFormat 13.0 documentation.

Use this Visual Studio extension to apply formatting rules to all files in a project at once (note: will take some time to apply): https://marketplace.visualstudio.com/items?itemName=munyabe.FormatAllFiles

@SamVanheer SamVanheer self-assigned this Nov 28, 2021
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 28, 2021
@SamVanheer SamVanheer changed the title Format all files Format all files and add clang-format configuration file Nov 28, 2021
@SamVanheer SamVanheer reopened this Nov 29, 2021
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 29, 2021
@SamVanheer SamVanheer reopened this Nov 29, 2021
SamVanheer referenced this issue in twhl-community/halflife-updated Nov 29, 2021
SamVanheer referenced this issue in twhl-community/halflife-op4-updated Dec 6, 2021
SamVanheer/halflife-updated#84
SamVanheer referenced this issue in twhl-community/halflife-op4-updated Dec 6, 2021
@SamVanheer SamVanheer transferred this issue from twhl-community/halflife-updated Dec 26, 2021
@SamVanheer SamVanheer added the enhancement New feature or request label Dec 26, 2021
@SamVanheer SamVanheer added this to To do in Upgrades via automation Dec 26, 2021
@SamVanheer SamVanheer added this to To do in Code cleanup Jun 1, 2022
@SamVanheer SamVanheer removed this from To do in Upgrades Jun 1, 2022
@SamVanheer SamVanheer moved this from To do to In progress in Code cleanup Jun 1, 2022
Code cleanup automation moved this from In progress to Done Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Development

No branches or pull requests

1 participant