[SLC] do not filter out private interface functions

This was accidentally added for regular functions
in 8350f95, but its
initial purpose is to filter out declarations with value
classes

^KT-80690 Fixed

Author: dimonchik0036

analysis/symbol-light-classes/src/org/jetbrains/kotlin/light/classes/symbol/classes/SymbolLightClassForInterface.kt

@@ -58,9 +58,7 @@ internal open class SymbolLightClassForInterface : SymbolLightClassForInterfaceO
         }
     }
 
-    protected open fun acceptCallableSymbol(symbol: KaCallableSymbol): Boolean {
-        return symbol !is KaNamedFunctionSymbol || symbol.visibility != KaSymbolVisibility.PRIVATE
-    }
+    protected open fun acceptCallableSymbol(symbol: KaCallableSymbol): Boolean = true
 
     override fun copy(): SymbolLightClassForInterface =
         SymbolLightClassForInterface(classOrObjectDeclaration, classSymbolPointer, ktModule, manager)

compiler/testData/asJava/lightClasses/lightClassByFqName/compilationErrors/PrivateInTrait.java

@@ -1,11 +1,15 @@
 public abstract interface PrivateInTrait /* PrivateInTrait*/ {
+  private @org.jetbrains.annotations.Nullable() java.lang.String bar(@org.jetbrains.annotations.NotNull() java.lang.String, @org.jetbrains.annotations.Nullable() java.lang.String);//  bar(@org.jetbrains.annotations.NotNull() java.lang.String, @org.jetbrains.annotations.Nullable() java.lang.String)
+
   private abstract @org.jetbrains.annotations.NotNull() java.lang.String getNn();//  getNn()
 
   private abstract @org.jetbrains.annotations.Nullable() java.lang.String getN();//  getN()
 
   private abstract void setNn(@org.jetbrains.annotations.NotNull() java.lang.String);//  setNn(@org.jetbrains.annotations.NotNull() java.lang.String)
 
   public static final class DefaultImpls /* PrivateInTrait.DefaultImpls*/ {
+    private static @org.jetbrains.annotations.Nullable() java.lang.String bar(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() PrivateInTrait, @org.jetbrains.annotations.NotNull() java.lang.String, @org.jetbrains.annotations.Nullable() java.lang.String);//  bar(@org.jetbrains.annotations.NotNull() PrivateInTrait, @org.jetbrains.annotations.NotNull() java.lang.String, @org.jetbrains.annotations.Nullable() java.lang.String)
+
     private static final @org.jetbrains.annotations.NotNull() java.lang.String getNn(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() PrivateInTrait);//  getNn(@org.jetbrains.annotations.NotNull() PrivateInTrait)
 
     private static final @org.jetbrains.annotations.Nullable() java.lang.String getN(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() PrivateInTrait);//  getN(@org.jetbrains.annotations.NotNull() PrivateInTrait)

compiler/testData/asJava/lightClasses/lightClassByFqName/privateInterfaceCallable.java

@@ -2,11 +2,16 @@ public abstract interface MyInterface /* MyInterface*/ {
   @kotlin.jvm.JvmExposeBoxed()
   private abstract @org.jetbrains.annotations.NotNull() java.lang.String getMangledVariable(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() StringWrapper);//  getMangledVariable(@org.jetbrains.annotations.NotNull() StringWrapper)
 
+  @kotlin.jvm.JvmExposeBoxed()
+  private abstract void mangledMethod(@org.jetbrains.annotations.NotNull() StringWrapper);//  mangledMethod(@org.jetbrains.annotations.NotNull() StringWrapper)
+
   @kotlin.jvm.JvmExposeBoxed()
   private abstract void setMangledVariable(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() StringWrapper, @org.jetbrains.annotations.NotNull() java.lang.String);//  setMangledVariable(@org.jetbrains.annotations.NotNull() StringWrapper, @org.jetbrains.annotations.NotNull() java.lang.String)
 
   private abstract int getRegularVariable();//  getRegularVariable()
 
+  private abstract void regularMethod();//  regularMethod()
+
   private abstract void setRegularVariable(int);//  setRegularVariable(int)
 
   public abstract int getPublicRegularVariable();//  getPublicRegularVariable()
@@ -22,10 +27,15 @@ public static final class DefaultImpls /* MyInterface.DefaultImpls*/ {
     @kotlin.jvm.JvmExposeBoxed()
     private static final void setMangledVariable(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() StringWrapper, @org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() MyInterface, @org.jetbrains.annotations.NotNull() java.lang.String);//  setMangledVariable(@org.jetbrains.annotations.NotNull() StringWrapper, @org.jetbrains.annotations.NotNull() MyInterface, @org.jetbrains.annotations.NotNull() java.lang.String)
 
+    @kotlin.jvm.JvmExposeBoxed()
+    private static void mangledMethod(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() MyInterface, @org.jetbrains.annotations.NotNull() StringWrapper);//  mangledMethod(@org.jetbrains.annotations.NotNull() MyInterface, @org.jetbrains.annotations.NotNull() StringWrapper)
+
     private static final int getRegularVariable(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() MyInterface);//  getRegularVariable(@org.jetbrains.annotations.NotNull() MyInterface)
 
     private static final void setRegularVariable(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() MyInterface, int);//  setRegularVariable(@org.jetbrains.annotations.NotNull() MyInterface, int)
 
+    private static void regularMethod(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() MyInterface);//  regularMethod(@org.jetbrains.annotations.NotNull() MyInterface)
+
     public static int getPublicRegularVariable(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() MyInterface);//  getPublicRegularVariable(@org.jetbrains.annotations.NotNull() MyInterface)
 
     public static void publicRegularMethod(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() MyInterface);//  publicRegularMethod(@org.jetbrains.annotations.NotNull() MyInterface)

compiler/testData/asJava/lightClasses/lightClassByPsi/jvmExposeBoxed/directive/defaultInInterface.java

@@ -16,10 +16,16 @@ public final class IC /* IC*/ {
 }
 
 public abstract interface Test /* Test*/ {
+  @kotlin.jvm.JvmExposeBoxed()
+  private abstract @org.jetbrains.annotations.NotNull() IC foo(@org.jetbrains.annotations.NotNull() IC);//  foo(@org.jetbrains.annotations.NotNull() IC)
+
   @kotlin.jvm.JvmExposeBoxed()
   private abstract @org.jetbrains.annotations.Nullable() IC getBar();//  getBar()
 
   public static final class DefaultImpls /* Test.DefaultImpls*/ {
+    @kotlin.jvm.JvmExposeBoxed()
+    private static @org.jetbrains.annotations.NotNull() IC foo(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() Test, @org.jetbrains.annotations.NotNull() IC);//  foo(@org.jetbrains.annotations.NotNull() Test, @org.jetbrains.annotations.NotNull() IC)
+
     @kotlin.jvm.JvmExposeBoxed()
     private static final @org.jetbrains.annotations.Nullable() IC getBar(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() Test);//  getBar(@org.jetbrains.annotations.NotNull() Test)
   }

compiler/testData/asJava/lightClasses/lightClassByPsi/jvmExposeBoxed/directive/defaultInInterface.kmp.java

@@ -16,10 +16,16 @@ public final class IC /* IC*/ {
 }
 
 public abstract interface Test /* Test*/ {
+  @kotlin.jvm.JvmExposeBoxed()
+  private abstract @org.jetbrains.annotations.NotNull() IC foo(@org.jetbrains.annotations.NotNull() IC);//  foo(@org.jetbrains.annotations.NotNull() IC)
+
   @kotlin.jvm.JvmExposeBoxed()
   private abstract @org.jetbrains.annotations.Nullable() IC getBar();//  getBar()
 
   public static final class DefaultImpls /* Test.DefaultImpls*/ {
+    @kotlin.jvm.JvmExposeBoxed()
+    private static @org.jetbrains.annotations.NotNull() IC foo(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() Test, @org.jetbrains.annotations.NotNull() IC);//  foo(@org.jetbrains.annotations.NotNull() Test, @org.jetbrains.annotations.NotNull() IC)
+
     @kotlin.jvm.JvmExposeBoxed()
     private static final @org.jetbrains.annotations.Nullable() IC getBar(@org.jetbrains.annotations.NotNull() @org.jetbrains.annotations.NotNull() Test);//  getBar(@org.jetbrains.annotations.NotNull() Test)
   }