Author: slaws
Date: Thu Mar 18 21:16:52 2010
New Revision: 924996
URL: http://svn.apache.org/viewvc?rev=924996&view=rev
Log:
Add comment, re. doubts about location of external attachment processing
Modified:
tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java
Modified:
tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java
URL:
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java?rev=924996&r1=924995&r2=924996&view=diff
==============================================================================
---
tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java
(original)
+++
tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java
Thu Mar 18 21:16:52 2010
@@ -103,6 +103,9 @@ public class ModelBuilderImpl implements
// Apply policy external attachment. Happens before the composite
type
// is created so that suitable promotion and structural processing
is
// applied to the attached policies
+ // TODO - I don't think this is correct. The policy spec
(4.12.1)implies that
+ // policy attachment happens independently of
implementation or
+ // structural inheritance
composite = policyAttachmentBuilder.build(composite, context);
// Process the implementation hierarchy by calculating the
component type
