svn commit: r911098 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilObject.java

[do...@apache.org]

Author: doogie
Date: Wed Feb 17 17:28:18 2010
New Revision: 911098

URL: http://svn.apache.org/viewvc?rev=911098&view=rev
Log:
Add comments about unreachable blocks.

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilObject.java

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilObject.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilObject.java?rev=911098&r1=911097&r2=911098&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilObject.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/UtilObject.java Wed Feb 17 17:28:18 2010
@@ -83,11 +83,18 @@
                     oos.close();
                 }
             } catch (IOException e) {
+                // I don't know how to force an error during flush or
+                // close of ObjectOutputStream; since OOS is wrapping
+                // BAOS, and BAOS does not throw IOException during
+                // write, I don't think this can happen.
                 Debug.logError(e, module);
             } finally {
                 bos.close();
             }
         } catch (IOException e) {
+            // How could this ever happen?  BAOS.close() is listed as
+            // throwing the exception, but I don't understand why this
+            // is.
             Debug.logError(e, module);
         }
 
@@ -136,6 +143,9 @@
                 bis.close();
             }
         } catch (IOException e) {
+            // How could this ever happen?  BAIS.close() is listed as
+            // throwing the exception, but I don't understand why this
+            // is.
             Debug.logError(e, module);
         }
         return obj;