Skip to content

Commit

Permalink
Showing 1 changed file with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
@@ -82,7 +82,7 @@ public Object execute(VirtualFrame frame, String expression, Object... arguments
"number of arguments doesn't match number of parameters");
}

ensureConstantExpressionParameters(expression, arguments);
assert ensureConstantExpressionParameters(expression, arguments);

final Object[] callArguments = RubyArguments.pack(
parentFrame(frame, arguments),
@@ -97,12 +97,12 @@ public Object execute(VirtualFrame frame, String expression, Object... arguments
return directCallNode.call(frame, callArguments);
}

@ExplodeLoop
private void ensureConstantExpressionParameters(String expression, Object[] arguments) {
assert this.expression == expression;
private boolean ensureConstantExpressionParameters(String expression, Object[] arguments) {
boolean test = this.expression == expression;
for (int n = 0; n < parameters.length; n++) {
assert parameters[n] == arguments[2 * n];
test = test && parameters[n] == arguments[2 * n];

This comment has been minimized.

Copy link
@eregon

eregon Apr 25, 2016

Member

Nested asserts are OK and useful, as they allow to observe the first failure in the debugger.

}
return test;
}

@ExplodeLoop

0 comments on commit 1fde05b

Please sign in to comment.