You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When building Coral.Managed within my project, some build warnings are given (mainly about null references) that should be handled e.g. by error reporting, establishing default handling, etc.
Yes, I know that warnings != errors :)
(with decades of experience as sw-dev and sw-arch in automotive and consumer-electronics industry and with hobby-projects)
Why I reported the warnings:
It is good practice to avoid build errors and warnings wherever possible (avoid future issues; think through your code once more; keep overview of errors/warnings in large projects)
Each build-warning may hint to a potential future runtime bug (here clearly the case at least for some null-reference warnings.
First level of defense; we even didn't start talking about static-code-analysis
Just wanted to help the project.
Whether or not the warnings will be fixed is actually your decision.
My recommendation would be to fix them.
Branch: dev
Commit: 9f071d7
When building Coral.Managed within my project, some build warnings are given (mainly about null references) that should be handled e.g. by error reporting, establishing default handling, etc.
Used tools:
BuildWarningsCoralManaged.txt
The text was updated successfully, but these errors were encountered: