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

Modernize C++ sources using clang-tidy #7621

Open
jcfr opened this issue Mar 5, 2024 · 0 comments
Open

Modernize C++ sources using clang-tidy #7621

jcfr opened this issue Mar 5, 2024 · 0 comments
Labels
Type: Enhancement Improvement to functionality

Comments

@jcfr
Copy link
Member

jcfr commented Mar 5, 2024

Is your feature request related to a problem? Please describe.

Following the update of the minimum C++ version required to build Slicer (now C++171), existing code can be improved to use simpler and more modern construct now built-in the c++ languages.

Describe the solution you'd like

The modernize- checks offered by clang-tidy2 should be selectively applied to update the sources and then systematically executed through a pre-commit hook.

Describe alternatives you've considered

NA

Additional context

Footnotes

  1. https://github.com/Slicer/Slicer/pull/6237

  2. https://clang.llvm.org/extra/clang-tidy/checks/list.html

@jcfr jcfr added the Type: Enhancement Improvement to functionality label Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Improvement to functionality
Development

No branches or pull requests

1 participant