Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ReturnCountExtendedCheck takes into account 'returns' in lambdas #463

Closed
baratali opened this issue Jul 15, 2016 · 1 comment
Closed

ReturnCountExtendedCheck takes into account 'returns' in lambdas #463

baratali opened this issue Jul 15, 2016 · 1 comment
Milestone

Comments

@baratali
Copy link
Contributor

@baratali baratali commented Jul 15, 2016

com.github.sevntu.checkstyle.checks.coding.ReturnCountExtendedCheck: Return count for 'load' method is 2 (max allowed is 1).

// code is not compilable
private static Integer load() {
    query((rs, rownum) -> {
        System.out.println();
        return p;
    });
    return a;
}

Similar issue for ReturnCountCheck: checkstyle/checkstyle#1068

rnveach added a commit to rnveach/sevntu.checkstyle that referenced this issue Dec 3, 2016
rnveach added a commit to rnveach/sevntu.checkstyle that referenced this issue Dec 4, 2016
rnveach added a commit to rnveach/sevntu.checkstyle that referenced this issue Dec 4, 2016
rnveach added a commit to rnveach/sevntu.checkstyle that referenced this issue Dec 4, 2016
rnveach added a commit to rnveach/sevntu.checkstyle that referenced this issue Dec 5, 2016
romani added a commit that referenced this issue Dec 5, 2016
@romani
Copy link
Member

@romani romani commented Dec 5, 2016

Fix is merged

@romani romani closed this Dec 5, 2016
@romani romani added this to the 1.23 milestone Dec 5, 2016
kariem added a commit to kariem/sevntu.checkstyle that referenced this issue Jul 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants