[sw,dv] %c format specifier not supported by SW-DV log
Nobody has claimed this yet.
- Dominant language
- SystemVerilog
- Stars
- 38
- Forks
- 21
- Avg merge
- 2d 23h
- Merged PRs (30d)
- 14
Description
Follow-up from PR #600.
The upstream format-string cleanup rewrites %c to %0c (the same way %d becomes %0d), but SV $sformatf rejects %0c (*W,SYSFMW: Illegal format ignored) — the log emits a warning even though the character still prints (https://github.com/lowRISC/mocha/pull/600#discussion_r3589039182).
PR #600 dropped %c from the dv_log_smoketest rather than fix this, since %0c is invalid SV anyway.
Options:
- Fix
cleanup_formatto leave%cuntouched. - Leave
%cunsupported and document it as such.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the cleanup_format entry point and the dv_log_smoketest referenced in the issue, then review PR #600's handling of format specifiers. Reproduce the %c case through the SW-DV log path and confirm the chosen behavior avoids an illegal %0c warning while preserving a clear test or documentation outcome.
Written by the indexing model from the issue text.
Assessment
- Domain
- testing-qa
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100