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

Add tidy check for const and ref members #1578

Open
godexsoft opened this issue Aug 2, 2024 · 0 comments
Open

Add tidy check for const and ref members #1578

godexsoft opened this issue Aug 2, 2024 · 0 comments
Labels
enhancement New feature or request refactoring Code improvements without logical changes
Milestone

Comments

@godexsoft
Copy link
Collaborator

Summary

We should enforce https://clang.llvm.org/extra/clang-tidy/checks/cppcoreguidelines/avoid-const-or-ref-data-members.html#cppcoreguidelines-avoid-const-or-ref-data-members

I tried to add this check locally and it yields a lot of warnings.

Motivation

Using const and reference members is bad practice.

Solution

Add a clang-tidy check and fix all errors manually.

@godexsoft godexsoft added enhancement New feature or request refactoring Code improvements without logical changes labels Aug 2, 2024
@godexsoft godexsoft added this to the Future milestone Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactoring Code improvements without logical changes
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant