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 - Resolving C26439 & type formatting warnings #64

Merged
merged 1 commit into from
Sep 14, 2024

Conversation

dpasukhi
Copy link
Member

Sprintf with %s always convert values into char*, not safety from int.
Move operators and constructors can be marked as noexcept

@dpasukhi dpasukhi added the 2. Enhancement New feature or request label Sep 10, 2024
@dpasukhi dpasukhi self-assigned this Sep 10, 2024
@dpasukhi dpasukhi added the 1. Coding Coding rules, trivial changes and misprints label Sep 10, 2024
Sprintf  with %s always convert values into char*, not safety from int.
Move operators and constructors can be marked as noexcept
Copy link

@ikochetkova ikochetkova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No remarks.

@dpasukhi dpasukhi merged commit acf0a7f into Open-Cascade-SAS:IR Sep 14, 2024
4 checks passed
@dpasukhi dpasukhi deleted the formattion_ql branch September 14, 2024 19:57
@dpasukhi dpasukhi added this to the Release 7.9 milestone Dec 2, 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
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants