Skip to content

Conversation

@markos
Copy link

@markos markos commented May 12, 2024

No description provided.

@markos markos requested review from gtsoul-tech and isildur-g May 12, 2024 14:35
@markos markos changed the title Fix false positive truncLongCastAssignment warnings Fix marked as false positive truncLongCastAssignment cppcheck warnings May 12, 2024
@markos
Copy link
Author

markos commented May 12, 2024

Copy link
Collaborator

@gtsoul-tech gtsoul-tech left a comment

Choose a reason for hiding this comment

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

I approve the changes to use long arithmetic for the multiplication, as it prevents potential integer overflow.

@markos markos merged commit cbd831e into develop May 13, 2024
@markos markos deleted the bugfix/cppcheck-truncLongCastAssignment branch May 22, 2024 09:34
markos added a commit that referenced this pull request Oct 24, 2025
…Assignment

Fix marked as false positive truncLongCastAssignment cppcheck warnings
markos added a commit that referenced this pull request Oct 24, 2025
…Assignment

Fix marked as false positive truncLongCastAssignment cppcheck warnings
markos added a commit that referenced this pull request Oct 24, 2025
…Assignment

Fix marked as false positive truncLongCastAssignment cppcheck warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants