https://github.com/manifold-systems/manifold/issues/349

- support calls to non-public extension methods

manifold-deps-parent/manifold-ext-test/src/test/java/manifold/ext/NonPublicExtensionMethodTest.java

@@ -0,0 +1,47 @@
+/*
+ * Copyright (c) 2022 - Manifold Systems LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package manifold.ext;
+
+import junit.framework.TestCase;
+
+public class NonPublicExtensionMethodTest extends TestCase
+{
+  public void testProtectedMethodDirectly()
+  {
+    Foo foo = new Foo();
+    assertEquals( "protected method", foo.callDirectly() );
+  }
+
+  public void testProtectedMethodIndirectly()
+  {
+    Foo foo = new Foo();
+    assertEquals( "protected method", foo.callIndirectly() );
+  }
+
+  static class Foo
+  {
+    public String callDirectly()
+    {
+      return myProtectedMethod();
+    }
+
+    public String callIndirectly()
+    {
+      return new Foo().myProtectedMethod();
+    }
+  }
+}

manifold-deps-parent/manifold-ext-test/src/test/java/manifold/ext/extensions/java/lang/Object/MyObjectExt.java

@@ -0,0 +1,29 @@
+/*
+ * Copyright (c) 2022 - Manifold Systems LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package manifold.ext.extensions.java.lang.Object;
+
+import manifold.ext.rt.api.Extension;
+import manifold.ext.rt.api.This;
+
+@Extension
+public class MyObjectExt
+{
+  protected static String myProtectedMethod( @This Object thiz )
+  {
+    return "protected method";
+  }
+}

manifold-deps-parent/manifold-ext/src/main/java/manifold/ext/ExtensionTransformer.java

@@ -2520,6 +2520,13 @@ private JCTree.JCMethodInvocation replaceExtCall( JCTree.JCMethodInvocation tree
         newArgs.add( 0, thisArg );
         tree.args = List.from( newArgs );
       }
+
+      boolean isPublic = (method.flags_field & Flags.AccessFlags) == PUBLIC;
+      if( !isPublic )
+      {
+        tree = replaceWithReflection( tree );
+      }
+
       return tree;
     }
     else if( methodSelect instanceof JCTree.JCIdent )
@@ -2547,6 +2554,13 @@ else if( methodSelect instanceof JCTree.JCIdent )
       newMethodSelect.type = method.type;
       newMethodSelect.pos = tree.pos;
       assignTypes( newMethodSelect.selected, method.owner );
+
+      boolean isPublic = (method.flags_field & Flags.AccessFlags) == PUBLIC;
+      if( !isPublic )
+      {
+        extCall = replaceWithReflection( extCall );
+      }
+
       return extCall;
     }
     return tree;