Hi Raymond, I looked thro your changes - just the one thing to say -
Thanks for taking a look and making those changes :)
- Venkat
On 8/6/07, Raymond Feng <[EMAIL PROTECTED]> wrote:
> Hi, Venkat.
>
> I fixed a few related issues and checked the changes into SVN under r562968.
>
> * The IntentImpl.hashCode() should be based on the name
> * Move getRequiredIntents() from Intent to ProfileIntent
> * The resolved intent will be kept in the collection before the clear() is
> called
>
> Thanks,
> Raymond
>
> - Original Message -
> From: "Venkata Krishnan" <[EMAIL PROTECTED]>
> To:
> Sent: Sunday, August 05, 2007 12:57 PM
> Subject: Changes to Policy Module
>
>
> > Hi Raymond,
> >
> > I had wrapped up a local increment to the Policy module and tried to
> > check it in. However, I find that there are modifications that you
> > had checked in. With those modifications, the work that I did in
> > policy-xml broke :).
> >
> > I'd need some help to merge what I have done with what you have
> > checked in. Of particular interest to me is the 'hashcode' method in
> > IntentImpl.java. The trunk has only your checked in versions of the
> > Intent.java and IntentImpl.java. For your referene, my versions for
> > these classes are here http://people.apache.org/~svkrish/policy. If
> > you replace these two files in the trunk, you'd find that the test in
> > policy-xml module runs fine.
> >
> > I am open to any modifications that you might suggest so that
> > implementation is clean. Could you please help in this regard.
> >
> > Thanks
> >
> > - Venkat
> >
> > -
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
>
>
> -
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]