Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8b886c6f authored by Eugene Susla's avatar Eugene Susla
Browse files

Fix genHiddenConstructor missing @hide 

Test: run on AppOpsManager, ensure ctr hidden
Change-Id: I691628d04664b5d9e871fdeadb32455bfda8b158
parent 9a654b28

tools/codegen/src/com/android/codegen/Generators.kt

+6 −3
Original line number Diff line number Diff line
@@ -5,8 +5,6 @@ import com.github.javaparser.ast.body.MethodDeclaration 
import com.github.javaparser.ast.body.VariableDeclarator

import com.github.javaparser.ast.expr.AnnotationExpr

import com.github.javaparser.ast.expr.ArrayInitializerExpr

import com.github.javaparser.ast.expr.LiteralExpr

import com.github.javaparser.ast.expr.UnaryExpr

import java.io.File
@@ -163,7 +161,12 @@ fun ClassPrinter.generateCopyConstructor() { 
        return

    }



    +"/** Copy constructor */"

    +"/**"

    +" * Copy constructor"

    if (FeatureFlag.COPY_CONSTRUCTOR.hidden) {

        +" * @hide"

    }

    +" */"

    +GENERATED_MEMBER_HEADER

    "public $ClassName(@$NonNull $ClassName orig)" {

        fields.forEachApply {

tools/codegen/src/com/android/codegen/SharedConstants.kt

+1 −1
Original line number Diff line number Diff line 
package com.android.codegen



const val CODEGEN_NAME = "codegen"

const val CODEGEN_VERSION = "1.0.20"

const val CODEGEN_VERSION = "1.0.21"



const val CANONICAL_BUILDER_CLASS = "Builder"

const val BASE_BUILDER_CLASS = "BaseBuilder"