You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by sm...@apache.org on 2006/08/04 12:30:24 UTC
svn commit: r428686 - /incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/DefaultPersistenceDelegate.java

Author: smishura
Date: Fri Aug  4 03:30:24 2006
New Revision: 428686

URL: http://svn.apache.org/viewvc?rev=428686&view=rev
Log:
Apply patch for HARMONY-975([classlib][beans] fixes for DefaultPersistenceDelegateTest failures (1))

Modified:
    incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/DefaultPersistenceDelegate.java

Modified: incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/DefaultPersistenceDelegate.java
URL: http://svn.apache.org/viewvc/incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/DefaultPersistenceDelegate.java?rev=428686&r1=428685&r2=428686&view=diff
==============================================================================
--- incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/DefaultPersistenceDelegate.java (original)
+++ incubator/harmony/enhanced/classlib/trunk/modules/beans/src/main/java/java/beans/DefaultPersistenceDelegate.java Fri Aug  4 03:30:24 2006
@@ -87,9 +87,10 @@
                     }
                 }
             }
+        } catch (RuntimeException e) {
+            throw e;
         } catch (Exception e) {
-            System.out.println("in DefaultPersistenceDelegate initialize() "
-                    + e.getClass() + " :" + e.getMessage());
+            out.getExceptionListener().exceptionThrown(e);
         }
     }
 
@@ -125,7 +126,7 @@
                                 found = true;
                                 break;
                             } else {
-                                throw new Exception("no getter for "
+                                throw new NoSuchMethodException("no getter for "
                                         + pds[j].getName() + " property");
                             }
 
@@ -133,15 +134,15 @@
                     } // for j
 
                     if (found == false) {
-                        throw new Exception("no property for name "
+                        throw new NoSuchMethodException("no property for name "
                                 + constructorPropertyNames[i] + " is found");
                     }
 
                 } // for i
+            } catch (RuntimeException e) {
+                throw e;
             } catch (Exception e) {
-                System.out
-                        .println("in DefaultPersistenceDelegate instantiate() "
-                                + e.getClass() + " :" + e.getMessage());
+                out.getExceptionListener().exceptionThrown(e);
             }
 
         }
@@ -165,6 +166,7 @@
                     return ((Boolean) result).booleanValue();
                 }
             } catch (Exception e) {
+                // XXX bad style of logging
                 System.out.println("in DefaultPersistenceDelegate.mutatesTo() "
                         + e.getClass() + " :" + e.getMessage());