-
Notifications
You must be signed in to change notification settings - Fork 1.5k
improved errorhandling of analyzer information loading #8159
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
base: main
Are you sure you want to change the base?
Conversation
b6e7ae6 to
1653983
Compare
|
The This is to catch issues with generating this data during run-time. This is done since we do not (can) use the XML library to write it. I need to modify that data after it has already been written to inject the |
34606a2 to
5e8739c
Compare
5e8739c to
129d43a
Compare
|
This exposes an issue with I filed https://trac.cppcheck.net/ticket/14456 about this. |
d5c3b7f to
5b83c1c
Compare
This issue is not limited to |
| if (error == tinyxml2::XML_ERROR_FILE_NOT_FOUND) { | ||
| // TODO: bailout? | ||
| //return "'" + xmlfile + "' from '" + filesTxt + "' not found"; | ||
| continue; | ||
| } | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We were bailing out here before and thus might have skipping any subsequent files. It would be better if we have a test which triggers this and add a test before merging this.
593357a to
4519cbb
Compare
There are various causes for this including the one with |
1684c9a to
1fa361a
Compare
…provided by the user the filename is never handed out and thus the file is never created
1fa361a to
a77ea6d
Compare
|



No description provided.