Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

Update clang format version + cmake format + minor warnings fixes #229

Merged
merged 4 commits into from
Sep 29, 2021

Conversation

jrmadsen
Copy link
Collaborator

  • updated clang-format from v6 to v9
    • this resulted in a lot of whitespace changes
  • updated workflows using (deprecated) ubuntu-16.04 to use ubuntu-20.04
  • added cmake-format workflow
  • removed use of TIMEMORY_NOCLONE
  • removed overloads of new and delete in singleton class
    • caused mismatched new/delete warnings in GCC 11

@jrmadsen jrmadsen force-pushed the update-clang-format-version branch 2 times, most recently from ec86c39 to 4ab1eaf Compare September 29, 2021 19:17
- clang-format v9
- cmake-format
- formatting workflow
- Removed TIMEMORY_NOCLONE usage
- Removed new/delete overloads from singleton (warnings)
@jrmadsen jrmadsen merged commit b13a58d into develop Sep 29, 2021
@jrmadsen jrmadsen deleted the update-clang-format-version branch September 29, 2021 19:26
@PointKernel
Copy link
Contributor

Have to say, cmake-format is still a bit ugly.

@jrmadsen
Copy link
Collaborator Author

Haha, yeah I don't really disagree. It is a relatively primitive tool at this point but at least it helps making things consistent.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants