[
https://issues.apache.org/jira/browse/DRILL-6656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16565888#comment-16565888
]
ASF GitHub Bot commented on DRILL-6656:
---------------------------------------
ilooner opened a new pull request #1415: DRILL-6656: Disallow extra semicolons
in import statements.
URL: https://github.com/apache/drill/pull/1415
Due to this jdk bug https://bugs.openjdk.java.net/browse/JDK-8072390,
IntelliJ and openjdk 7 and 8 allow extra semicolons in imports. This is
technically disallowed by the java spec and results in a compilation issue in
eclipse. This resulted in this issue
https://issues.apache.org/jira/browse/DRILL-6650. This change adds some
checkstyle regexes to prevent a similar mistake from being repeated.
This PR adds the following checkstyle checks:
1. Disallow lines starting with a semicolon.
2. Disallow two consecutive semicolons separated by nothing but whitespace.
3. Disallowing empty statements in java statements. This check does not
effect semicolons in imports; however, since we are already fixing semicolons
in imports we might as well fix them everywhere.
The first two checks catch semicolons in imports, but as a side effect also
detect strange usages of semicolons in the body of a java files. As a result
things like the following are disallowed as well:
```
for (;;
) {
}
```
or
```
new Builder()
.someStuff()
.thatStuff()
.build()
;
```
But I think we should disallow these strange usages of imports anyway.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Add Regex To Disallow Extra Semicolons In Imports
> -------------------------------------------------
>
> Key: DRILL-6656
> URL: https://issues.apache.org/jira/browse/DRILL-6656
> Project: Apache Drill
> Issue Type: Improvement
> Reporter: Timothy Farkas
> Assignee: Timothy Farkas
> Priority: Major
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
