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.

Sign up for GitHub

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

Jump to bottom

Fix issues reported by clang-tidy #473

Merged
merged 2 commits into from
Jun 26, 2023
Merged

Fix issues reported by clang-tidy #473

merged 2 commits into from
Jun 26, 2023

Conversation

Danielius1922
Copy link
Member

@Danielius1922 Danielius1922 commented Jun 22, 2023 •
edited by jkralik
Loading

Fixes #467

@Danielius1922 Danielius1922 changed the title Adam/bugfix/clang tidy 4 Fix issues reported by clang-tidy Jun 22, 2023
@Danielius1922 Danielius1922 force-pushed the adam/bugfix/clang-tidy-4 branch 2 times, most recently from 61bfade to f91994b Compare June 22, 2023 15:38
@Danielius1922
Handle oc_message_t buffer size
e9c5732 
Depending on compilation options the buffer of oc_message_t might
be static or dynamic, and of various sizes. Functions that work
with the buffer must receive correct size of the buffer.
@Danielius1922 Danielius1922 force-pushed the adam/bugfix/clang-tidy-4 branch from f91994b to 73b0216 Compare June 23, 2023 17:15
@Danielius1922 Danielius1922 added the OCF Conformance Testing OCF Conformance Testing required label Jun 23, 2023
@Danielius1922 Danielius1922 force-pushed the adam/bugfix/clang-tidy-4 branch from 8ebcc6a to 346cc02 Compare June 24, 2023 14:35
@ocf-conformance-test-tool ocf-conformance-test-tool bot removed the OCF Conformance Testing OCF Conformance Testing required label Jun 24, 2023
@Danielius1922 Danielius1922 added the OCF Conformance Testing OCF Conformance Testing required label Jun 24, 2023
@Danielius1922 Danielius1922 force-pushed the adam/bugfix/clang-tidy-4 branch from 7f3b4a4 to afe469a Compare June 25, 2023 09:18
@ocf-conformance-test-tool ocf-conformance-test-tool bot removed the OCF Conformance Testing OCF Conformance Testing required label Jun 25, 2023
@Danielius1922 Danielius1922 force-pushed the adam/bugfix/clang-tidy-4 branch 5 times, most recently from f64cc38 to fd66e49 Compare June 25, 2023 18:29
@Danielius1922 Danielius1922 force-pushed the adam/bugfix/clang-tidy-4 branch from fd66e49 to 7979e6c Compare June 25, 2023 20:30
@Danielius1922 Danielius1922 added the OCF Conformance Testing OCF Conformance Testing required label Jun 25, 2023
@sonarcloud
Copy link

sonarcloud bot commented Jun 25, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 1 Security Hotspot
Code Smell A 13 Code Smells

75.9% 75.9% Coverage
1.1% 1.1% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@Danielius1922 Danielius1922 marked this pull request as ready for review June 26, 2023 07:19
@Danielius1922 Danielius1922 merged commit 664e718 into master Jun 26, 2023
@Danielius1922 Danielius1922 deleted the adam/bugfix/clang-tidy-4 branch June 26, 2023 07:25
@github-actions github-actions bot locked and limited conversation to collaborators Jun 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
OCF Conformance Testing OCF Conformance Testing required
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Coverity issues
1 participant
@Danielius1922