ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 25777] - [PATCH] [REGRESSION] Cannot pass composite path reference between scripts
Date Mon, 29 Dec 2003 11:17:52 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25777>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25777

[PATCH] [REGRESSION] Cannot pass composite path reference between scripts





------- Additional Comments From j_a_fernandez@yahoo.com  2003-12-29 11:17 -------
I defenitely think that Peter removal of the toString() was the right thing to 
do. This was not just some optimization, it was a superflous call causing a 
20% time increase in some builds. Not peanuts at all. (And maskerading a real 
bug, the one you have found).

I would like to find a solution that applies to all references, not only Paths.
It seem to me what is missing is for <ant/> to call some form of "resolve" 
method, when passing a reference. So, instead of just a call to clone() we 
would do a call to something like cloneAndResolve() that will perhaps use 
something simillar to dieOnCircularReference to resolve all inner references 
of the object.

I guess in principle you only need to do this when references are passed 
explicitly and inheritRefs=false; when inheritRefs=true, it may not be needed.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Mime
View raw message