Mikhail,

Alexei Z, do you agree with what Alexey V. suggests?

Yes, I've sent  the reply to JIRA already, please see below.

Thanks,
Alexei

---------- Forwarded message ----------
From: Alexei Zakharov (JIRA) <[EMAIL PROTECTED]>
Date: 11.09.2006 13:35
Subject: [jira] Commented: (HARMONY-1408) [classlib][beans]
implementation of ProxyPersistenceDelegate
To: [EMAIL PROTECTED]


  [ 
http://issues.apache.org/jira/browse/HARMONY-1408?page=comments#action_12433809
]

Alexei Zakharov commented on HARMONY-1408:
------------------------------------------

Ok, I agree. In fact I cannot imagine the situation when we've got two
different Class'es loaded by different classloaders. At least there
are no comments in java beans spec on how we should handle such
situations. I think we may also replace

          } else if (methodName.equals("newInstance") //$NON-NLS-1$
                  && target instanceof Class
-                    && ((Class) target).getName().equals(
-                            "java.lang.reflect.Array")) { //$NON-NLS-1$
+                   && target == Array.class)) {
              Class<?> componentType = (Class) arguments[0];

earlier in the same method.
Thank you Alexei.

[classlib][beans] implementation of ProxyPersistenceDelegate
------------------------------------------------------------

                Key: HARMONY-1408
                URL: http://issues.apache.org/jira/browse/HARMONY-1408
            Project: Harmony
         Issue Type: Improvement
         Components: Classlib
        Environment: ws2003
           Reporter: Alexei Zakharov
        Assigned To: Mikhail Loenko
        Attachments: CustomizedPDTest4.patch


The attached patch contains implementation of persistence delegate for all 
dynamic proxy objects that can be created by means of the 
java.lang.reflect.Proxy class methods. In the process of development of this 
persistence delegate a few bugs in Statement and 
DefaultPersistenceDelegatesFactory were also fixed. After applying this patch 
all tests from 
org.apache.harmony.beans.tests.java.beans.CustomizedPersistenceDelegateTest 
will be enabled so we can remove this class from the exclude list.
Thanks, Alexei

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the
administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira




--
Alexei Zakharov,
Intel Middleware Product Division

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to