I did that on purpose as I see the component builder code seems to add callback references and services during the build. Another loop is needed to capture these contracts, right?

Thanks,
Raymond
--------------------------------------------------
From: "Simon Laws" <[email protected]>
Sent: Wednesday, December 02, 2009 4:07 AM
To: <[email protected]>
Subject: Re: svn commit: r886027 - in /tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl: ComponentBuilderImpl.java ComponentPolicyBuilderImpl.java CompositePolicyBuilderImpl.java

Hi Raymond

I think this change makes the policy processing much clearer. I have a
small aesthetic comment so feel free to ignore if you don't like
it....

As the component builder already loops explicitly over
services/references to calculate the configuration could be jump into
the ComponentPolicyBuilder at that point rather than diving into the
ComponentPolicyBuilder .configure() operation which loops over
services/references again?

Simon

Reply via email to