Hi Joel,
> In the past, I believe Bison has printed "warning" in front of every line > that's part of a warning. I'm not sure we should depart from that > practice. > This is how it does now, as far I have seen from the code (for warnings). > > Can you explain your motivation a little more? Is it that you want to > reuse the same *_at invocations to list variants in both the warning and > complaint cases? You could add to note_at a bool argument that specifies > whether to invoke warn_at or complain_at. > My motivation was the following: 1) Keep overall *number* of errors/warnings issued by bison reasonable. When bison issues an "ambiguity" error with 10 sub-messages, overall number of errors (if one does "grep" on output, or, alternatively bison itself will count them and print a summary message) should be one. 2) Not to see the "warning" word in each sub-message. Alex -- Best regards, Alex Rozenman ([email protected]).
