[ https://issues.apache.org/jira/browse/GROOVY-8509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16477858#comment-16477858 ]
ASF GitHub Bot commented on GROOVY-8509: ---------------------------------------- GitHub user jwagenleitner opened a pull request: https://github.com/apache/groovy/pull/707 GROOVY-8509: SC error call to protected method from same package Because they were closely related, this PR also includes the following: 1. A change to the fix introduced by GROOVY-7883 (b1d1232770aa) to prevent filtering protected methods in the `StaticTypeCheckingVisitor` if caller is in the same package. From the same commit, removes the test `Groovy7883Bug#test3` because it should compile and the new test introduced for GROOVY-8509 in `MethodCallsStaticCompilation` takes it place. 1. A change to revert groovy.sql.Sql#asList (b1d1232770aa) back to `protected`. Fix for GROOVY-7883 enforced visibility for method calls in `TypedChecked` and `CompileStatic` modes. Test was originally put in place to verify the added ClosureParams, so changed test to access method via subclass so that the access modifier for the method can remain protected. I think the spirit of the test is preserved in this case without having to open access to the method under test. You can merge this pull request into a Git repository by running: $ git pull https://github.com/jwagenleitner/groovy 8509-SC-protected-same-package Alternatively you can review and apply these changes as the patch at: https://github.com/apache/groovy/pull/707.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #707 ---- commit c76783966f4f9198498abdf6f4696dc36e449cd5 Author: John Wagenleitner <jwagenleitner@...> Date: 2018-05-16T14:58:23Z Revert change to groovy.sql.Sql#asList in commit b1d1232770aa Fix for GROOVY-7883 enforced visibility for method calls in TypedChecked and CompileStatic modes. Test was originally put in place to verify the added ClosureParams, so changed test to access method via subclass so that the access modifier for the method can remain protected. commit 8821e8406adeec4d69894b739a2d941c51266c87 Author: John Wagenleitner <jwagenleitner@...> Date: 2018-05-16T15:34:48Z GROOVY-8509: SC error call to protected method from same package Also includes a change to the fix introduced by GROOVY-7883 (b1d1232770aa) to prevent filtering protected methods in the StaticTypeCheckingVisitor if caller is in the same package. From the same commit, removes the test Groovy7883Bug#test3 because it should compile and the new test introduced for GROOVY-8509 in MethodCallsStaticCompilation takes it place. ---- > SC: error for call to protected method from same package > -------------------------------------------------------- > > Key: GROOVY-8509 > URL: https://issues.apache.org/jira/browse/GROOVY-8509 > Project: Groovy > Issue Type: Bug > Affects Versions: 2.5.0-beta-2, 2.6.0-alpha-3 > Reporter: Eric Milles > Priority: Major > > Groovy 2.5+ is producing compilation errors for @CompileStatic test class > calling a protected method of class under test (in same package). This > should be allowed as protected is "package-private" plus subtypes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)