Skip to content

Commit abe7aaf

Browse files
committed
Small refactor to use pattern switch
1 parent 48ef8b0 commit abe7aaf

File tree

1 file changed

+8
-15
lines changed

1 file changed

+8
-15
lines changed

src/main/java/org/jruby/prism/builder/IRBuilderPrism.java

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1775,10 +1775,8 @@ private void buildMultiAssignment(Node[] pre, Node rest, Node[] post, Operand va
17751775
private void processReads(Variable result, List<Tuple<Node, ResultInstr>> assigns, Map<Node, Operand> reads,
17761776
Node node) {
17771777
switch(node) {
1778-
case CallTargetNode call:
1779-
reads.put(call.receiver, build(call.receiver));
1780-
break;
1781-
case IndexTargetNode index:
1778+
case CallTargetNode call -> reads.put(call.receiver, build(call.receiver));
1779+
case IndexTargetNode index -> {
17821780
reads.put(index.receiver, build(index.receiver));
17831781
Node[] arguments = index.arguments == null ? Node.EMPTY_ARRAY : index.arguments.arguments;
17841782
int[] flags = new int[] { 0 };
@@ -1787,20 +1785,15 @@ private void processReads(Variable result, List<Tuple<Node, ResultInstr>> assign
17871785
System.arraycopy(args, 0, hackyArgs, 0, args.length);
17881786
hackyArgs[args.length] = new Integer(flags[0]);
17891787
reads.put(index.arguments, new Array(hackyArgs));
1790-
break;
1791-
case ClassVariableTargetNode cvar:
1792-
reads.put(node, classVarDefinitionContainer());
1793-
break;
1794-
case ConstantPathTargetNode constpath:
1795-
reads.put(node, buildModuleParent(constpath.parent));
1796-
break;
1797-
case MultiTargetNode multi:
1788+
}
1789+
case ClassVariableTargetNode _cvar -> reads.put(node, classVarDefinitionContainer());
1790+
case ConstantPathTargetNode constpath -> reads.put(node, buildModuleParent(constpath.parent));
1791+
case MultiTargetNode multi -> {
17981792
var subRet = temp();
17991793
assigns.add(new Tuple<>(multi, new ToAryInstr(subRet, result)));
18001794
buildMultiAssignment(multi.lefts, multi.rest, multi.rights, subRet, reads, assigns);
1801-
break;
1802-
default:
1803-
break;
1795+
}
1796+
default -> {}
18041797
}
18051798
}
18061799

0 commit comments

Comments
 (0)