directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Felix Knecht <fel...@apache.org>
Subject Re: svn commit: r656527 - in /directory/apacheds/branches/bigbang: core-avl/src/main/java/org/apache/directory/server/core/avltree/DefaultMarshaller.java jdbm-store/src/main/java/org/apache/directory/server/core/partition/impl/btree/jdbm/JdbmIndex.java
Date Thu, 15 May 2008 07:21:39 GMT
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" );
>          }
>  
>          /*
>
>
>   


Mime
View raw message