From b032e647f7cab590191428bbac320c39855221df Mon Sep 17 00:00:00 2001 From: Wojciech Litewka Date: Thu, 29 Feb 2024 11:56:25 +0100 Subject: [PATCH] [IR] Move assert in IrSetValueImpl to factory function #KT-65773 In Progress --- .../ir/expressions/impl/IrSetValueImpl.kt | 29 +++++++++---------- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/compiler/ir/ir.tree/src/org/jetbrains/kotlin/ir/expressions/impl/IrSetValueImpl.kt b/compiler/ir/ir.tree/src/org/jetbrains/kotlin/ir/expressions/impl/IrSetValueImpl.kt index 99f7ee95a40..fd3b85e275f 100644 --- a/compiler/ir/ir.tree/src/org/jetbrains/kotlin/ir/expressions/impl/IrSetValueImpl.kt +++ b/compiler/ir/ir.tree/src/org/jetbrains/kotlin/ir/expressions/impl/IrSetValueImpl.kt @@ -25,12 +25,6 @@ class IrSetValueImpl internal constructor( ) : IrSetValue() { override var attributeOwnerId: IrAttributeContainer = this override var originalBeforeInline: IrAttributeContainer? = null - - init { - if (symbol.isBound) { - assert(symbol.owner.isAssignable) { "Only assignable IrValues can be set" } - } - } } fun IrSetValueImpl( @@ -40,12 +34,17 @@ fun IrSetValueImpl( symbol: IrValueSymbol, value: IrExpression, origin: IrStatementOrigin?, -) = IrSetValueImpl( - constructorIndicator = null, - startOffset = startOffset, - endOffset = endOffset, - type = type, - symbol = symbol, - value = value, - origin = origin, -) +): IrSetValueImpl { + if (symbol.isBound) { + assert(symbol.owner.isAssignable) { "Only assignable IrValues can be set" } + } + return IrSetValueImpl( + constructorIndicator = null, + startOffset = startOffset, + endOffset = endOffset, + type = type, + symbol = symbol, + value = value, + origin = origin, + ) +}