Hi Felix,

I committed your patch for Tuscany-927. Thanks a lot! I reformatted it slightly to pass checkstyle. On the QName, issue, yes we can add those to SCAObject but perhaps we should add it to PolicyAttachable instead since not all SCAObjects may have QNames?

A  couple of small comments:

1. Do you think it would be better to name PolicyModel "PolicyObject" since it is an object that pertains to policies as opposed to a model for them?

2. I noticed the is a comment on Intent which says that class is not referenced by model objects but it is in the "model" package. Should this class get moved under the same package as IntentRegistry?


For next steps, let's see if we can start getting the policy infrastructure hooked into the runtime.

Jim
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to