maybe it's a good idea to log the real exception first? Otherwise
we'll use the stacktrace of the original exception if only giving the
message to the IO exception.

Kind regards,
Andreas

On Thu, Mar 22, 2012 at 16:38,  <[email protected]> wrote:
> Author: jgoodyear
> Date: Thu Mar 22 15:38:38 2012
> New Revision: 1303843
>
> URL: http://svn.apache.org/viewvc?rev=1303843&view=rev
> Log:
> [KARAF-1279] IOExceptions used in an incompatible way with Java 5.
>
> Changing IOException call to IOException(String) signature.
>
>
> Modified:
>    
> karaf/branches/karaf-2.2.x/management/mbeans/config/src/main/java/org/apache/karaf/management/mbeans/config/internal/ConfigMBeanImpl.java
>    
> karaf/branches/karaf-2.2.x/shell/config/src/main/java/org/apache/karaf/shell/config/ConfigCommandSupport.java
>
> Modified: 
> karaf/branches/karaf-2.2.x/management/mbeans/config/src/main/java/org/apache/karaf/management/mbeans/config/internal/ConfigMBeanImpl.java
> URL: 
> http://svn.apache.org/viewvc/karaf/branches/karaf-2.2.x/management/mbeans/config/src/main/java/org/apache/karaf/management/mbeans/config/internal/ConfigMBeanImpl.java?rev=1303843&r1=1303842&r2=1303843&view=diff
> ==============================================================================
> --- 
> karaf/branches/karaf-2.2.x/management/mbeans/config/src/main/java/org/apache/karaf/management/mbeans/config/internal/ConfigMBeanImpl.java
>  (original)
> +++ 
> karaf/branches/karaf-2.2.x/management/mbeans/config/src/main/java/org/apache/karaf/management/mbeans/config/internal/ConfigMBeanImpl.java
>  Thu Mar 22 15:38:38 2012
> @@ -187,7 +187,7 @@ public class ConfigMBeanImpl extends Sta
>                         storageFile = new File(new URL((String) val).toURI());
>                     }
>                 } catch (Exception e) {
> -                    throw new IOException(e);
> +                    throw new IOException(e.getMessage());
>                 }
>             }
>             Properties p = new Properties(storageFile);
>
> Modified: 
> karaf/branches/karaf-2.2.x/shell/config/src/main/java/org/apache/karaf/shell/config/ConfigCommandSupport.java
> URL: 
> http://svn.apache.org/viewvc/karaf/branches/karaf-2.2.x/shell/config/src/main/java/org/apache/karaf/shell/config/ConfigCommandSupport.java?rev=1303843&r1=1303842&r2=1303843&view=diff
> ==============================================================================
> --- 
> karaf/branches/karaf-2.2.x/shell/config/src/main/java/org/apache/karaf/shell/config/ConfigCommandSupport.java
>  (original)
> +++ 
> karaf/branches/karaf-2.2.x/shell/config/src/main/java/org/apache/karaf/shell/config/ConfigCommandSupport.java
>  Thu Mar 22 15:38:38 2012
> @@ -162,7 +162,7 @@ public abstract class ConfigCommandSuppo
>                 storageFile = new File(new URL((String) val).toURI());
>             }
>             } catch (Exception e) {
> -                throw new IOException(e);
> +                throw new IOException(e.getMessage());
>             }
>         }
>         Properties p = new Properties(storageFile);
>
>

Reply via email to