anukalp2804 opened a new pull request, #17611:
URL: https://github.com/apache/iotdb/pull/17611
## Summary
This PR resolves SonarCloud rule `java:S128`, switch cases should end with
an unconditional `break`, `return` or `throw` statement. For that we have done
some changes like :
- Added break statements where execution could fall through to the next case
- Removed unnecessary {} blocks from cases that already had unconditional
return statements
- Added {} blocks in a few places to make the case scope clearer for static
analysis tools
This PR has:
- [x] been self-reviewed.
- [x] been built locally with `mvn spotless:apply`.
- [x] been built locally with `mvn clean package -DskipTests`.
- [x] been built locally with `mvn -pl iotdb-core -am test -DskipTests`.
Resolves :
https://sonarcloud.io/project/issues?impactSeverities=BLOCKER&issueStatuses=OPEN%2CCONFIRMED&id=apache_iotdb
`End this switch case with an unconditional break, return or throw
statement.`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
