Opened 14 years ago

Closed 14 years ago

#334 closed defect (fixed)

Simplification of function application

Reported by: zirkel Owned by:
Priority: major Milestone:
Component: symbolic Version:
Keywords: Cc:

Description

In IdealSimplifier.simplifyApply(EvaluatedFunctionExpression apply), there is a loop to handle the arguments to the apply.

for (int i = 0; i < numArgs; i++) {
	Simplification argumentSimplification = simplifyTree(apply
			.argument(i));
	change = change || argumentSimplification.success();
	newArguments[i] = argumentSimplification.result();
}

However, the method EvaluatedFunctionExpression.argument(int index) returns the function for index=0, and starts the function arguments at index=1. Thus, the call to this method should probably be either

apply.argument(i+1)

or else

apply.functionArgument(i)

I can make the change, I just wanted to confirm that my interpretation is correct.

Change History (1)

comment:1 by zirkel, 14 years ago

Resolution: fixed
Status: newclosed

Changed it to use functionArgument().

Note: See TracTickets for help on using tickets.