Repository navigation
Diagnostics needed for the silently omitted non-unused attribute on the delimiter of delimited_list parser (%) #70
Description
Currently the L53 of list parser passes the hard-coded unused attribute for the underlying parser. This leads to surprising result at runtime, where the content of the RHS is silently omitted even if the RHS has some explicit attribute.
x4/include/iris/x4/operator/list.hpp
Lines 47 to 55 in 9de1d0c
This looks like a bug to me, but fixing this would be a breaking change. We should re-check the unit tests.
A workaround is to rewrite this:
body % delimto this:
body >> *(delim >> body)but I think the current behavior is very awkward in the first place.
Activity
Update README
Update README
Update README
saki7 commentedon Sep 13, 2026
This is not a defect, it is the correct semantics; see the updated README.
The only issue we have here is the lack of the diagnostics. We definitely need some compiler warning for this pattern, otherwise we end up into the similar situation like #79.
unusedandunusedparsers in a parent parser with a non-unusedexposed attribute leads to incomplete parse result at runtime #48[-]list parser (`%`) should not omit the RHS if the attribute of delimiter is not an `unused_type`[/-][+]Diagnostics needed for the silently omitted non-unused attribute on the delimiter of delimited_list parser (`%`)[/+]