No need to declare methods as throwing Bioclipse Exception in the int…

…erface. It is caught and handled by AOP during the method call.
egonw · Jul 21, 2009 · 7736746 · 7736746
1 parent f3945aa
commit 7736746
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/plugins/net.bioclipse.cdk.business/src/net/bioclipse/cdk/business/ICDKManager.java b/plugins/net.bioclipse.cdk.business/src/net/bioclipse/cdk/business/ICDKManager.java
@@ -134,8 +134,7 @@ public ICDKMolecule loadMolecule( String path )
      * @throws CoreException
      */
     @Recorded
-    public ICDKMolecule loadMolecule( IFile file )
-                        throws IOException, BioclipseException, CoreException;
+    public ICDKMolecule loadMolecule( IFile file );
 
     @Recorded
     @PublishedMethod( 

diff --git a/plugins/net.bioclipse.cdk.business/src/net/bioclipse/cdk/domain/SDFAdapterFactory.java b/plugins/net.bioclipse.cdk.business/src/net/bioclipse/cdk/domain/SDFAdapterFactory.java
@@ -91,18 +91,15 @@ private ICDKMolecule loadSDFPart(SDFElement element){
         } catch (IOException e) {
             // TODO Auto-generated catch block
             LogUtils.debugTrace( logger,e);
-        } catch (BioclipseException e) {
-            // TODO Auto-generated catch block
-            LogUtils.debugTrace( logger,e);
-        }finally{
+        } 
+        finally {
             try {                
                 if(file!=null) file.delete(true,null);
                 if(folder!=null) folder.delete(true,null );
             }catch (CoreException e) {
                 // TODO Auto-generated catch block
                 LogUtils.debugTrace( logger,e);
             } 
-
         }
 
         return null;