Roman Ivanov
Roman Ivanov
@rnveach , we already support some cases with `return` - https://github.com/checkstyle/checkstyle/blob/master/src/main/java/com/puppycrawl/tools/checkstyle/checks/coding/FallThroughCheck.java#L157 Should we update Check to search of `return` in whole code of CASE ?
> I don't see OUTER in the example. I just used sources from https://github.com/checkstyle/checkstyle/issues/3885#issuecomment-283454252 > To detect your case, we need to identify the infinite loop. yes, we can support...
@mohitsatr , Issue is approved, but lets treat with caution. If implementation will go wild, we can abandon this issue. Lets try to support only simple cases that shown in...
@rdiachenko , please continue review. CI should be green.
I restarted few, passed. Some no-exception build failures are not related to update of PR
https://checkstyle.org/idea.html this usually works for all.
I think this issue should be reported to intelijidea plugin Might be related https://github.com/jshiell/checkstyle-idea/pull/210
Did you consider contributing this Check to aevntu-checkstyle project?
It is not dead. I don't see any functional fix at https://github.com/sevntu-checkstyle/sevntu.checkstyle/commits/master/ So nothing to release for 2 years, ok :). We keep a lot of Checks in it, to...
I am closing this PR. As we have no agreement in issue. Even we agreed on something, each Check will be in separate PR.