You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@royale.apache.org by jo...@apache.org on 2022/10/27 22:48:52 UTC
[royale-compiler] 05/09: linter: EmptyStatementRule should handle block open before semicolon
This is an automated email from the ASF dual-hosted git repository.
joshtynjala pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-compiler.git
commit 2b5660010d4e88a4bc16c7016f0f626ff340748c
Author: Josh Tynjala <jo...@apache.org>
AuthorDate: Thu Oct 27 14:50:46 2022 -0700
linter: EmptyStatementRule should handle block open before semicolon
---
.../apache/royale/linter/rules/EmptyStatementRule.java | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/linter/src/main/java/org/apache/royale/linter/rules/EmptyStatementRule.java b/linter/src/main/java/org/apache/royale/linter/rules/EmptyStatementRule.java
index a6c63b0cc..6308be5b7 100644
--- a/linter/src/main/java/org/apache/royale/linter/rules/EmptyStatementRule.java
+++ b/linter/src/main/java/org/apache/royale/linter/rules/EmptyStatementRule.java
@@ -48,14 +48,13 @@ public class EmptyStatementRule extends LinterRule {
private void checkSemicolon(IASToken semicolon, TokenQuery tokenQuery, Collection<ICompilerProblem> problems) {
IASToken prevToken = tokenQuery.getSignificantTokenBefore(semicolon);
- if (prevToken == null) {
- return;
- }
- if (prevToken.getType() != ASTokenTypes.TOKEN_SEMICOLON) {
- return;
- }
- if (prevToken.isImplicit()) {
- return;
+ if (prevToken != null) {
+ if (prevToken.getType() != ASTokenTypes.TOKEN_SEMICOLON && prevToken.getType() != ASTokenTypes.TOKEN_BLOCK_OPEN) {
+ return;
+ }
+ if (prevToken.isImplicit()) {
+ return;
+ }
}
problems.add(new EmptyStatementLinterProblem(semicolon));
}