Author: dgolovin
Date: 2011-09-13 19:15:11 -0400 (Tue, 13 Sep 2011)
New Revision: 34703
Modified:
trunk/common/plugins/org.jboss.tools.common.model/src/org/jboss/tools/common/model/filesystems/impl/JarAccess.java
Log:
Exception handling clean up.
JarAccess.java should not just report zip related exceptions, but include affected zip
file name in the log message as well.
Modified:
trunk/common/plugins/org.jboss.tools.common.model/src/org/jboss/tools/common/model/filesystems/impl/JarAccess.java
===================================================================
---
trunk/common/plugins/org.jboss.tools.common.model/src/org/jboss/tools/common/model/filesystems/impl/JarAccess.java 2011-09-13
23:14:50 UTC (rev 34702)
+++
trunk/common/plugins/org.jboss.tools.common.model/src/org/jboss/tools/common/model/filesystems/impl/JarAccess.java 2011-09-13
23:15:11 UTC (rev 34703)
@@ -242,8 +242,8 @@
}
return sb.toString();
} catch (IOException e) {
- errors.add(e.getClass().getName() + ": " + e.getMessage());
- ModelPlugin.getPluginLog().logError(e);
+ errors.add(e.getClass().getName() + " occurs when reading " + jar.getName()
+ " : " + e.getMessage()); //$NON-NLS-1$//$NON-NLS-2$
+ ModelPlugin.getPluginLog().logError("Exception occurs when reading " +
jar.getName(), e); //$NON-NLS-1$
return ""; //$NON-NLS-1$
} finally {
unlockJar();
Show replies by date