Skip to content

Commit 1ff756f

Browse files
committed
Fix codegen bug: _elseif triggers generation at statement creation time
1 parent 3318041 commit 1ff756f

File tree

3 files changed

+5
-8
lines changed

3 files changed

+5
-8
lines changed

errai-codegen/src/main/java/org/jboss/errai/codegen/builder/impl/IfBlockBuilderImpl.java

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ public BlockBuilder<ElseBlockBuilder> if_(BooleanOperator op, Statement rhs) {
6969

7070
@Override
7171
public BlockBuilder<ElseBlockBuilder> if_(BooleanOperator op, Object rhs) {
72-
Statement rhsStatement = GenUtil.generate(context, rhs);
72+
final Statement rhsStatement = GenUtil.generate(context, rhs);
7373
return if_(new BooleanExpressionBuilder(rhsStatement, op));
7474
}
7575

@@ -113,10 +113,7 @@ public BlockBuilder<ElseBlockBuilder> elseif_(Statement lhs) {
113113

114114
@Override
115115
public BlockBuilder<ElseBlockBuilder> elseif_(Statement lhs, BooleanOperator op, Statement rhs) {
116-
if (lhs.getType() == null)
117-
lhs.generate(context);
118-
119-
IfBlock elseIfBlock = new IfBlock(new BooleanExpressionBuilder(lhs, rhs, op));
116+
final IfBlock elseIfBlock = new IfBlock(new BooleanExpressionBuilder(lhs, rhs, op));
120117
ifBlock.setElseIfBlock(elseIfBlock);
121118
return elseif_(elseIfBlock);
122119
}

errai-data-binding/src/main/java/org/jboss/errai/databinding/rebind/BindableProxyGenerator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -428,7 +428,7 @@ private Statement generateDeepUnwrapMethodBody(final String methodName) {
428428
)
429429
)
430430
.finish()
431-
.elseif_(Bool.expr(Stmt.invokeStatic(BindableProxyFactory.class, "isBindableType", target().invoke(readMethod))))
431+
.elseif_(Stmt.invokeStatic(BindableProxyFactory.class, "isBindableType", target().invoke(readMethod)))
432432
.append(Stmt.loadVariable(cloneVar).invoke(writeMethod,
433433
Cast.to (
434434
readMethod.getReturnType(),

errai-marshalling/src/main/java/org/jboss/errai/marshalling/rebind/MarshallerGeneratorFactory.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -247,8 +247,8 @@ private ElseBlockBuilder generateGetMarshallerHelperMethods() {
247247
elseBlockBuilder = If.objEquals(Stmt.loadLiteral(typeName), Stmt.loadVariable("a0")).append(stmt).finish();
248248
}
249249
else {
250-
elseBlockBuilder = elseBlockBuilder.elseif_(Stmt.create(Context.create().addVariable("a0", String.class))
251-
.loadLiteral(typeName).invoke("equals", Stmt.loadVariable("a0"))).append(stmt).finish();
250+
elseBlockBuilder = elseBlockBuilder.elseif_(Stmt.loadLiteral(typeName).invoke("equals",
251+
Stmt.loadVariable("a0"))).append(stmt).finish();
252252
}
253253

254254
typeIndex += 1;

0 commit comments

Comments
 (0)