Skip to content

Commit 7ab04c7

Browse files
committed
Make the object wrapping generated Kotlin accessors be internal
This prevents issues with leaking internal classes.
1 parent eb0ceb8 commit 7ab04c7

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ public final class MyClassTestAccessors {
5454
The processor-kotlin artifact generates Kotlin extension methods for the class that owns the
5555
annotated fields for more idiomatic accessor usage.
5656
```kotlin
57-
object MyClassTestAccessors {
57+
internal object MyClassTestAccessors {
5858
fun <T> MyClass.myField(): T
5959

6060
fun <T> MyClass.myField(newValue: T)

processor-kotlin/src/main/kotlin/testaccessors/internal/AccessorWriter.kt

+2
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import com.squareup.kotlinpoet.ClassName
66
import com.squareup.kotlinpoet.CodeBlock
77
import com.squareup.kotlinpoet.FileSpec
88
import com.squareup.kotlinpoet.FunSpec
9+
import com.squareup.kotlinpoet.KModifier
910
import com.squareup.kotlinpoet.ParameterSpec
1011
import com.squareup.kotlinpoet.TypeSpec
1112
import com.squareup.kotlinpoet.TypeVariableName
@@ -41,6 +42,7 @@ internal class AccessorWriter(
4142
elementUtils.getPackageOf(enclosingClassElement).qualifiedName.toString(), fileAndObjectName)
4243
.indent(" ")
4344
val objectSpecBuilder = TypeSpec.objectBuilder(fileAndObjectName)
45+
.addModifiers(KModifier.INTERNAL)
4446
annotatedElements.flatMap(object : (Element) -> Iterable<FunSpec> {
4547
override fun invoke(element: Element) =
4648
element.getAnnotation(RequiresAccessor::class.java).requires.map {

0 commit comments

Comments
 (0)