2007/7/4, Alexei Zakharov <[EMAIL PROTECTED]>:

Spark Chen wrote:
> > If no objection, I will go to implement those missing persistence
> > functionality.

FYI we already have number of persistence delegates classes located in
org.apache.harmony.beans package, and FieldPersistenceDelegate is one
of them. It contains the algorithm that is very close to one you've
implemented in the patch for HARMONY-4321. So I don't think we should
add new FieldPersistenceDelegate, let's fix the existing one instead
(if it needs fixing). BTW, we also have a special folder
(src/test/java-internal/org/apache/harmony/beans) where tests for
persistence delegates are located. Shouldn't we create something like
FieldPersistenceDelegateTest there?

Thanks,


BTW, I find some beans test cases are excluded. I plan to include and fix
them first.

2007/7/3, Tony Wu <[EMAIL PROTECTED]>:
> On 7/3/07, Spark Shen <[EMAIL PROTECTED]> wrote:
> > I find beans.XMLEncoder does not persist java.lang.reflect.Fieldproperly.
> > And I suspect there are more classes not properly handled.
> >
>
> Yes, I think so.
> > I have reported a JIRA:
> > https://issues.apache.org/jira/browse/HARMONY-4321
> >
> > If no objection, I will go to implement those missing persistence
> > functionality.
> >
>
> please go head:)



--
Alexei Zakharov,
Intel ESSD




--
Spark Shen
China Software Development Lab, IBM

Reply via email to