Comment 1 Jeff Douglas 2012-03-26 11:41:34 EDT

package acme.bugs;

library tim0001
	function n() returns(int)
		if (getArguments() == null)
			return (0);
		else
			return (getArguments().getInt("cc", 1));
		end
	end
	function getArguments() returns(tim0001h)
		return (new tim0001h);
	end
end

package acme.bugs;

handler tim0001h
	function getInt(a string in, i int in) returns(int)
		return (0);
	end
end

Produces unnecessary temporary variables.

Comment 2 Brian Svihovec 2012-03-26 16:13:03 EDT

Are we targeting these changes for .8 Final or for a future milestone?  I am thinking 'Future' at this point, based on the fact that this is the last week of SVT for .8.

Comment 3 Lisa Lasher 2012-03-29 11:44:26 EDT

Jeff, I think this is the item you brought up in the status meeting.   If you have 100% regression tested these changes, then they can go into the build today (or are they already in?).   If you are expecting SVT to cover the testing, then it is too late.  In that case, don't commit the code, and set the target for Future.

Comment 4 Jeff Douglas 2012-03-30 12:07:49 EDT

This defect it to covert general optimizations. Part of this was implemented in 080 and part will be in the next release.

The part in 080 was to use less temporary variables when invoking functions that did not need them, based on the parameter type of in/inout/out etc. Prior to this fix, some expressions that invoked functions would get temporary variables being used, instead of being able to directly invoke the function.

Comment 5 Lisa Lasher 2012-04-04 22:54:01 EDT

changing from defect to enhancement.

Comment 6 Matt Heitz 2013-01-03 14:43:23 EST

Setting the target milestone to Future for bugs that won't be addressed in 0.8.2.