AndroidComponents API support #3
Conversation
dkostyrev
requested review from
a team and
satween
and removed request for
a team
| private val isDirtyByDefault: Boolean = false, | ||
| ) : ClassVisitor(asmApi, classVisitor) { | ||
|
|
||
| var isDirty: Boolean = false |
There was a problem hiding this comment.
точно ли должно тут это проперти в public торчать?
There was a problem hiding this comment.
копипастил из colonist, вроде нет причины наружу торчать
| private const val IS_ENABLED_DEPRECATION_WARNING = "paranoid.enabled is deprecated. Use paranoid.applyToBuildTypes" |
There was a problem hiding this comment.
тут можно через ReplaceWith прописать, чтобы в студии автозамену можно было сделать
| ) | ||
|
|
||
| try { | ||
| processor.process() |
There was a problem hiding this comment.
может validate() и cleanOutPut() тоже под try-catch затянуть, чтобы в случае всяких IOException-ов было понятно что за таска?
There was a problem hiding this comment.
Мне кажется оно и так понятно будет какая таска упала, но обернул
| } | ||
| var isCacheable: Boolean = false | ||
| var includeSubprojects: Boolean = false | ||
| var isEnabled: Boolean by deprecatedProperty(true, IS_ENABLED_DEPRECATION_WARNING) |
There was a problem hiding this comment.
может тут кастомный getter-setter сделать, чтобы оно applyToBuildTypes выставляло, тогда можно будет в новом коде уже не использовать isEnabled и избавиться от @Suppress("DEPRECATION")
| @@ -9,6 +9,7 @@ allprojects { | |||
| buildscript { | |||
| repositories { | |||
| google() | |||
| mavenLocal() | |||
There was a problem hiding this comment.
В комментарии к коммиту говориться "Add mavenCentral to repositories..." скорее всего опечатка, и имелось в виду все-таки mavenLocal, но решил обратить внимание на всякий пожарный
…sses processed by paranoid.
… should be processed were indeed processed by paranoid.
…ks up into jar task for java projects.
…at text is displayed.
dkostyrev
force-pushed
the
android-components
branch
from
998c1d5
to
2b46b99
Compare
dkostyrev
force-pushed
the
android-components
branch
from
2b46b99
to
1159eef
Compare
ParanoidPluginso it can be applied to java modules as well as android modules.@Obfuscateannotation.WatermarkClassVisitor), soParanoidProcessorcan validate paranoid plugin usage by checking whether this attribute is present in should-be-transformed classes available in classpath.ParanoidTransformTask.