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

Some bug fixes #10809

Merged
merged 6 commits into from
Sep 7, 2022
Merged

Some bug fixes #10809

merged 6 commits into from
Sep 7, 2022

Conversation

@Nir-Az
Copy link
Collaborator

Nir-Az commented Aug 24, 2022

@baderouaich Nice catch! Thanks for your contribution..
We will look into this fixes, may I ask how did you spotted them? manually or with some static analysis tools?

Thanks!

@baderouaich
Copy link
Contributor Author

Hello @Nir-Az, I did the static analyzing with cppcheck, you can also have it running as a Github Action with cppcheck-action to spot issues in the future and report them automatically into a file...
I hope this helps.
Have a nice day!

@Nir-Az Nir-Az requested a review from maloel September 7, 2022 12:40
@maloel
Copy link
Collaborator

maloel commented Sep 7, 2022

Nice!

@maloel maloel merged commit 5b6b818 into IntelRealSense:development Sep 7, 2022
maloel added a commit that referenced this pull request Sep 8, 2022
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