Felix you're absolutely right - there's also the option of setting the cause after instantiating before throwing.  That's what I usually do but I was rushing to fix this to get some sleep :).

I'll fix it.

Thanks,
Alex

On Thu, May 15, 2008 at 3:21 AM, Felix Knecht <felixk@apache.org> wrote:
Hi Alex

Your loosing information about e.g. the class not found this way.
I'd propose something like

catch ( ClassNotFoundException e )
{
  throw new IOException( "Could not find class. " + e.getMessage( ) );
}


Regards
Felix



Author: akarasulu
Date: Thu May 15 00:16:28 2008
New Revision: 656527

URL: http://svn.apache.org/viewvc?rev=656527&view=rev
Log:
make it work with jdk 5

Modified:
   directory/apacheds/branches/bigbang/core-avl/src/main/java/org/apache/directory/server/core/avltree/DefaultMarshaller.java
   directory/apacheds/branches/bigbang/jdbm-store/src/main/java/org/apache/directory/server/core/partition/impl/btree/jdbm/JdbmIndex.java

Modified: directory/apacheds/branches/bigbang/core-avl/src/main/java/org/apache/directory/server/core/avltree/DefaultMarshaller.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/bigbang/core-avl/src/main/java/org/apache/directory/server/core/avltree/DefaultMarshaller.java?rev=656527&r1=656526&r2=656527&view=diff
==============================================================================
--- directory/apacheds/branches/bigbang/core-avl/src/main/java/org/apache/directory/server/core/avltree/DefaultMarshaller.java (original)
+++ directory/apacheds/branches/bigbang/core-avl/src/main/java/org/apache/directory/server/core/avltree/DefaultMarshaller.java Thu May 15 00:16:28 2008
@@ -61,7 +61,7 @@
        }
        catch ( ClassNotFoundException e )
        {
-            throw new IOException( "Could not find class", e );
+            throw new IOException( "Could not find class" );
        }
          return object;

Modified: directory/apacheds/branches/bigbang/jdbm-store/src/main/java/org/apache/directory/server/core/partition/impl/btree/jdbm/JdbmIndex.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/bigbang/jdbm-store/src/main/java/org/apache/directory/server/core/partition/impl/btree/jdbm/JdbmIndex.java?rev=656527&r1=656526&r2=656527&view=diff
==============================================================================
--- directory/apacheds/branches/bigbang/jdbm-store/src/main/java/org/apache/directory/server/core/partition/impl/btree/jdbm/JdbmIndex.java (original)
+++ directory/apacheds/branches/bigbang/jdbm-store/src/main/java/org/apache/directory/server/core/partition/impl/btree/jdbm/JdbmIndex.java Thu May 15 00:16:28 2008
@@ -176,7 +176,7 @@
        }
        catch ( NamingException e )
        {
-            throw new IOException( "Failed to find an equality matching rule for attribute type", e );
+            throw new IOException( "Failed to find an equality matching rule for attribute type" );
        }
          /*