r214722 - Highlight the offending function parameter when the format_arg argument refers to an invalid function parameter type.

David Blaikie dblaikie at gmail.com
Mon Aug 4 09:56:06 PDT 2014


On Mon, Aug 4, 2014 at 9:36 AM, Aaron Ballman <aaron at aaronballman.com> wrote:
> On Mon, Aug 4, 2014 at 12:34 PM, David Blaikie <dblaikie at gmail.com> wrote:
>> Test case? (I know we don't rigorously test the ranges specified in a
>> diagnostic, but if we're fixing a bug in it, it might be worthwhile -
>> I think they're sometimes tested with the machine readable diagnostic
>> format)
>
> I asked about this last week, but there was no conclusion on whether
> this sort of change needs tests or how to effectively test it. Can you
> point me to a testcase to model after?

Looks like we have no existing test cases for this, nor even any test
cases for the feature (that already exists) that can be used to test
this. "-fdiagnostics-print-source-range-info" should be able to be
used to test this (we have lots of test cases that test /fixits/ using
a similar feature "-fdiagnostics-parseable-fixits" - for example
test/Sema/warn-main.c)

- Dave



More information about the cfe-commits mailing list