From dccc9091c8f8f0911c79f7f6662c70c627249997 Mon Sep 17 00:00:00 2001 From: Yong He Date: Thu, 29 Jun 2023 13:11:49 -0700 Subject: Warn on semicolon after `if`. (#2948) * Warn on semicolon after `if`. * add test result --------- Co-authored-by: Yong He --- tests/diagnostics/if-empty-body.slang | 11 +++++++++++ tests/diagnostics/if-empty-body.slang.expected | 8 ++++++++ 2 files changed, 19 insertions(+) create mode 100644 tests/diagnostics/if-empty-body.slang create mode 100644 tests/diagnostics/if-empty-body.slang.expected (limited to 'tests') diff --git a/tests/diagnostics/if-empty-body.slang b/tests/diagnostics/if-empty-body.slang new file mode 100644 index 000000000..de8983352 --- /dev/null +++ b/tests/diagnostics/if-empty-body.slang @@ -0,0 +1,11 @@ +//DIAGNOSTIC_TEST(windows):SIMPLE: + +// Test that use of empty statement after an `if` leads to an warning. + +struct S {} + +void test() +{ + if (1+1==2); + return; +} \ No newline at end of file diff --git a/tests/diagnostics/if-empty-body.slang.expected b/tests/diagnostics/if-empty-body.slang.expected new file mode 100644 index 000000000..30915e39d --- /dev/null +++ b/tests/diagnostics/if-empty-body.slang.expected @@ -0,0 +1,8 @@ +result code = 0 +standard error = { +tests/diagnostics/if-empty-body.slang(9): warning 20101: potentially unintended empty statement at this location; use {} instead. + if (1+1==2); + ^ +} +standard output = { +} -- cgit v1.2.3