On Mon, Dec 21, 2015 at 02:10:17PM -0700, Jeff Law wrote: > On 12/18/2015 01:21 PM, David Malcolm wrote: > > >I don't think there's a way to fix -Wmisleading-indentation if we're > >in this state, so the first part of the following patch detects if > >this has happened, and effectively turns off -Wmisleading-indentation > >from that point onwards. To avoid a false sense of security, the > >patch issues a "sorry" at the that point, currently with this wording: > >location-overflow-test-1.c:17:0: sorry, unimplemented: > >-Wmisleading-indentation is disabled from this point onwards, since > >column-tracking was disabled due to the size of the code/headers > Seems reasonable. I can't see any way to get indentation warnings if we > don't have column info.
sorry will set sorrycount to non-zero though, so seen_error () will be true and the compiler will exit with non-zero exit status. That is IMHO not appripriate for warning (at least unless -Werror=misleading-indentation). Jakub