harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexei Zakharov (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-1408) [classlib][beans] implementation of ProxyPersistenceDelegate
Date Mon, 11 Sep 2006 09:35:24 GMT
    [ 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

        

Mime
View raw message