-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
update tinyxml to version 10 #22475
update tinyxml to version 10 #22475
Conversation
This seems to introduce a new |
@@ -23,9 +23,7 @@ distribution. | |||
|
|||
#include "tinyxml2.h" | |||
|
|||
#include "mu_patch.h" |
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.
This seems to indicate that we patched it at some point. See d6a5864. We should check if that patch is still needed.
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.
I think this is the reason for the failing tests. I added the line again.
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.
But note that it's not only that line, that's why I mentioned d6a5864
Small update to tinyxml, see https://github.com/leethomason/tinyxml2/releases/tag/10.0.0