New J2K: Assume specifyLocalVariableTypeByDefault in for loops initializers

This commit is contained in:
Ilya Kirillov
2019-01-28 14:08:30 +03:00
committed by Ilya Kirillov
parent 1e7b1f8716
commit 936c43d2bc
2 changed files with 10 additions and 9 deletions
@@ -35,7 +35,6 @@ class ForConversion(private val context: ConversionContext) : RecursiveApplicabl
val whileStatement = JKWhileStatementImpl(condition, whileBody)
if (loopStatement.initializer is JKEmptyStatement) return whileStatement
//TODO check for error conflict
val convertedFromForLoopSyntheticWhileStatement =
JKKtConvertedFromForLoopSyntheticWhileStatementImpl(
@@ -131,13 +130,13 @@ class ForConversion(private val context: ConversionContext) : RecursiveApplicabl
else -> return null
}
val range = forIterationRange(start, right, reversed, inclusive, loopVarPsi)
//TODO for loop explicitType
// val explicitType = if (context.converter.settings.specifyLocalVariableTypeByDefault)
// JKJavaPrimitiveTypeImpl.INT
// else null
val explicitType =
if (context.converter.settings.specifyLocalVariableTypeByDefault)
JKJavaPrimitiveTypeImpl.INT
else JKNoTypeImpl
val loopVarDeclaration =
JKForLoopVariableImpl(
JKTypeElementImpl(JKNoTypeImpl),
JKTypeElementImpl(explicitType),
loopVar::name.detached(),
JKStubExpressionImpl()
)
@@ -13,12 +13,14 @@ import org.jetbrains.kotlin.lexer.KtTokens
import org.jetbrains.kotlin.name.ClassId
class ForInConversion : RecursiveApplicableConversionBase() {
class ForInConversion(private val context: ConversionContext) : RecursiveApplicableConversionBase() {
override fun applyToElement(element: JKTreeElement): JKTreeElement {
if (element !is JKForInStatement) return recurse(element)
val parameterDeclaration = element.declaration as? JKParameter
if (parameterDeclaration != null) {
val parameterDeclaration = element.declaration as? JKVariable
if (parameterDeclaration != null
&& !context.converter.settings.specifyLocalVariableTypeByDefault
) {
parameterDeclaration.type = JKTypeElementImpl(JKNoTypeImpl)
}