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

Coding - Fixing clang-tidy warnings #45

Closed
wants to merge 2 commits into from

Conversation

dpasukhi
Copy link
Member

Fixed warnings for:

  • TKernel
  • TKMath
  • TKGeomBase Some warnings that are expected marked as NOLINT

TODO:
Needs to check and validate other for loops with double

@dpasukhi dpasukhi self-assigned this Aug 25, 2024
@dpasukhi dpasukhi added 1. Coding Coding rules, trivial changes and misprints 2. Enhancement New feature or request 3. In progress The issue is not complete. Waiting for new patches labels Aug 25, 2024
dpasukhi and others added 2 commits December 21, 2024 11:13
Update size of buffer for the brep parsing according standard
Fixed warnings for:
- TKernel
- TKMath
- TKGeomBase
Some warnings that are expected marked as NOLINT
TODO:
Needs to check and validate other for loops with double
@dpasukhi
Copy link
Member Author

Will be splatted.
First PR is #207

@dpasukhi dpasukhi closed this Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. Coding Coding rules, trivial changes and misprints 2. Enhancement New feature or request 3. In progress The issue is not complete. Waiting for new patches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant