Commit 6b95d7cc by Patrick Palka

Fix a -Wmisleading-indentation false-negative

gcc/c-family/ChangeLog:

	* c-indentation.c (should_warn_for_misleading_indentation):
	Compare next_stmt_vis_column with guard_line_first_nws instead
	of with guard_line_vis_column.

gcc/testsuite/ChangeLog:

	* c-c++-common/Wmisleading-indentation.c: Augment test.

From-SVN: r228071
parent 3d5cda12
2015-09-24 Patrick Palka <ppalka@gcc.gnu.org>
* c-indentation.c (should_warn_for_misleading_indentation):
Compare next_stmt_vis_column with guard_line_first_nws instead
of with guard_line_vis_column.
2015-09-23 Manuel López-Ibáñez <manu@gcc.gnu.org>
PR c/49654
......
......@@ -464,7 +464,7 @@ should_warn_for_misleading_indentation (const token_indent_info &guard_tinfo,
{
if (next_stmt_vis_column > guard_line_first_nws
|| (next_tok_type == CPP_OPEN_BRACE
&& next_stmt_vis_column == guard_vis_column))
&& next_stmt_vis_column == guard_line_first_nws))
return true;
}
}
......
2015-09-24 Patrick Palka <ppalka@gcc.gnu.org>
* c-c++-common/Wmisleading-indentation.c: Augment test.
2015-09-23 Ville Voutilainen <ville.voutilainen@gmail.com>
Fix small typos in the coding rule enforcement warnings.
......
......@@ -797,6 +797,15 @@ fn_37 (void)
i++);
foo (i); /* { dg-warning "statement is indented as if" } */
if (flagA)
{
foo (1);
}
else if (flagB); /* { dg-message "8: ...this 'if' clause" } */
{ /* { dg-warning "statement is indented as if" } */
foo (2);
}
#undef EMPTY
#undef FOR_EACH
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment