Re: [C PATCH] Fix missing warning (PR c/67730)

2015-09-29 Thread Marc Glisse
On Tue, 29 Sep 2015, Marek Polacek wrote: This fixes missing warning for the attached testcase. In such a case, we must use the expansion point location. I didn't simply add loc = expansion_point_location_if_in_system_header (loc); as might be seen elsewhere in the codebase because we pass LO

Re: [C PATCH] Fix missing warning (PR c/67730)

2015-09-29 Thread Marek Polacek
On Tue, Sep 29, 2015 at 06:04:55PM +0200, Marc Glisse wrote: > On Tue, 29 Sep 2015, Marek Polacek wrote: > > >This fixes missing warning for the attached testcase. In such a case, > >we must use the expansion point location. I didn't simply add > > loc = expansion_point_location_if_in_system_hea

Re: [C PATCH] Fix missing warning (PR c/67730)

2015-09-29 Thread Joseph Myers
On Tue, 29 Sep 2015, Marek Polacek wrote: > This fixes missing warning for the attached testcase. In such a case, > we must use the expansion point location. I didn't simply add > loc = expansion_point_location_if_in_system_header (loc); > as might be seen elsewhere in the codebase because we

[C PATCH] Fix missing warning (PR c/67730)

2015-09-29 Thread Marek Polacek
This fixes missing warning for the attached testcase. In such a case, we must use the expansion point location. I didn't simply add loc = expansion_point_location_if_in_system_header (loc); as might be seen elsewhere in the codebase because we pass LOC down to convert_for_assignment where many