Skip to content

Conversation

mcb5637
Copy link
Collaborator

@mcb5637 mcb5637 commented Jul 18, 2025

close #342

I added filename + line to the assert text in release mode, to make finding the failed assert easier.

@mcb5637 mcb5637 marked this pull request as ready for review July 18, 2025 13:12
@mcb5637 mcb5637 requested a review from liss-h July 18, 2025 13:12
Copy link
Collaborator

@liss-h liss-h left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, I only have some minor comments

@mcb5637 mcb5637 requested a review from liss-h July 24, 2025 11:42
@liss-h liss-h added the not pobr breaking change Not a breaking change for Persisted Object Binary Representation even if the workflow says so label Jul 25, 2025
@liss-h liss-h merged commit e50fe3a into develop Jul 28, 2025
18 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not pobr breaking change Not a breaking change for Persisted Object Binary Representation even if the workflow says so
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add RDF4CPP_ASSERT like we have in other repos
2 participants