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

Fix for warns #5

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

alexrayne
Copy link

hallow!
here provided some massive fixes of warning of compilation
and a pair of fixes from other people

@gurux-kurumi
Copy link

Hi,

I'm sorry, but we can't just accept your changes because we have a lot of clients and we have to try to test all changes.
What compiler you have used? We can fix your warnings and check your changes.

BR,
Mikko

@alexrayne
Copy link
Author

i works on gnu arm 9.2.1 now.
i've just try build some of your server project.
anyway, i no need your help, since my work goes much further then this fixes.
just try to share some work, and try to test your participationess.

@alexrayne
Copy link
Author

mem leak fix from ayz was mulfunctional, now it shuld be careful

Copy link

sonarcloud bot commented Dec 28, 2023

Quality Gate Failed Quality Gate failed

Failed conditions

5 Security Hotspots
E Reliability Rating on New Code (required ≥ A)
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants