Jump statements should not be redundant (squid:S3626) FP in try block


(Alix Warnke) #1

SonarQube version: 6.7.6 LTS (build 38781)
Java Plugin version: 5.11 (build 17289)

    void foo(boolean b) {
        try {
            if (b) {
                System.out.println("work block A");
            } else {
                return;
            }
        } finally {
            System.out.println("work block B");
        }
        System.out.println("work block C");
    }

squid:S3626 triggers on the line with the return statement.
If the return statement is removed then work block C will be executed.