Code Optimization and Refactoring

[TakanoTaiga]

Abstract

This pull request introduces code optimizations and improvements, including the removal of unused variables, added override qualifiers, standardization of declarations, and adjustments to lambda captures and explicit casts.

Background

To improve code clarity, performance, and maintainability, minor refactors and cleanup were conducted. These changes aim to ensure consistency and efficiency in the codebase.

Details

• Unused Variable Removal: Unused variables were removed to optimize memory usage.
• Inline Directives: Added missing inline directives where needed.
• Virtual Destructor: A virtual destructor was added for polymorphic classes to ensure proper resource management.
• Lambda Capture Adjustments: Unnecessary captures in lambdas were removed for performance.
• Override Qualifiers: Missing override qualifiers were added to functions to improve code clarity.
• Standardizing Declarations: Standardized declaration patterns for better readability and consistency.
• Explicit Casts: Implemented explicit casts to prevent implicit conversion issues.

References

This PR fixes errors or warnings that appeared during Clang builds.

Destructive Changes

N/A

Known Limitations

N/A

[github-actions]

Checklist for reviewers ☑️

All references to "You" in the following text refer to the code reviewer.

• Is this pull request written in a way that is easy to read from a third-party perspective?
• Is there sufficient information (background, purpose, specification, algorithm description, list of disruptive changes, and migration guide) in the description of this pull request?
• If this pull request contains a destructive change, does this pull request contain the migration guide?
• Labels of this pull request are valid?
• All unit tests/integration tests are included in this pull request? If you think adding test cases is unnecessary, please describe why and cross out this line.
• The documentation for this pull request is enough? If you think adding documents for this pull request is unnecessary, please describe why and cross out this line.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
25.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud