Merged
Merged
Conversation
Daniel Marjamäki and others added 2 commits
August 5, 2026 14:43Pull request overview
This PR introduces deterministic “warning hash” generation for token-based warnings by computing and emitting a hash attribute in XML output, and updates build dependencies and CLI tests accordingly.
Changes:
- Compute and store a warning hash from token callstacks/error paths in
ErrorMessage. - Emit the
hashattribute in XML output and update CLI XML-output expectations. - Update Makefile dependency lists to reflect new header usage.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file| File | Description |
|---|---|
| test/cli/other_test.py | Updates expected XML output to include the new hash attribute. |
| oss-fuzz/Makefile | Adds missing header dependencies for errorlogger.o in the OSS-Fuzz build. |
| Makefile | Adds missing header dependencies for errorlogger.o in the main build. |
| lib/errorlogger.h | Declares ErrorMessage::calculateWarningHash helper. |
| lib/errorlogger.cpp | Implements warning hash computation and wires it into relevant constructors. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Labels
None yet