Skip to content

fixed #13485 - symbol names in error messages were not serialized#7142

Merged
firewave merged 1 commit intocppcheck-opensource:mainfrom
firewave:symbol-xml
Jan 2, 2025
Merged

fixed #13485 - symbol names in error messages were not serialized#7142
firewave merged 1 commit intocppcheck-opensource:mainfrom
firewave:symbol-xml

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

@firewave firewave changed the title Symbol xml fixed #13485 - symbol names in error messages were not serialized Dec 27, 2024
Comment thread test/cli/other_test.py
assert not os.path.exists(str(test_file) + '.dump')


@pytest.mark.xfail(strict=True)
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

xfail because it not fully fixed until #7141 is merged.

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Jan 1, 2025

I botched the Python test in this but since it will be failing anyways it should not matter that much (it still helped with testing this specific issue). It is also already resolved in the follow-up #7141 which builds on top of this.

@firewave firewave merged commit a3cb101 into cppcheck-opensource:main Jan 2, 2025
@firewave firewave deleted the symbol-xml branch January 2, 2025 15:35
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.

2 participants