[jbosstools-commits] JBoss Tools SVN: r17804 - in trunk/seam/plugins: org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/internal/reveng and 1 other directory.

jbosstools-commits at lists.jboss.org jbosstools-commits at lists.jboss.org
Tue Sep 29 12:48:18 EDT 2009


Author: dgolovin
Date: 2009-09-29 12:48:17 -0400 (Tue, 29 Sep 2009)
New Revision: 17804

Modified:
   trunk/seam/plugins/org.jboss.tools.seam.core/src/org/jboss/tools/seam/internal/core/project/facet/Seam2ProjectCreator.java
   trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/internal/reveng/JDBCTablesColumnsReader.java
Log:
Code clean up, catch(Exception) blocks were fixed

Modified: trunk/seam/plugins/org.jboss.tools.seam.core/src/org/jboss/tools/seam/internal/core/project/facet/Seam2ProjectCreator.java
===================================================================
--- trunk/seam/plugins/org.jboss.tools.seam.core/src/org/jboss/tools/seam/internal/core/project/facet/Seam2ProjectCreator.java	2009-09-29 16:46:33 UTC (rev 17803)
+++ trunk/seam/plugins/org.jboss.tools.seam.core/src/org/jboss/tools/seam/internal/core/project/facet/Seam2ProjectCreator.java	2009-09-29 16:48:17 UTC (rev 17804)
@@ -283,7 +283,10 @@
 		}
 		try {
 			new FormatProcessorXML().formatFile((IFile) jbossAppXml);
-		} catch (Exception ignore) {
+		} catch (IOException e) {
+			
+		} catch (CoreException e) {
+			
 		}
 	}
 	

Modified: trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/internal/reveng/JDBCTablesColumnsReader.java
===================================================================
--- trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/internal/reveng/JDBCTablesColumnsReader.java	2009-09-29 16:46:33 UTC (rev 17803)
+++ trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/internal/reveng/JDBCTablesColumnsReader.java	2009-09-29 16:48:17 UTC (rev 17804)
@@ -100,12 +100,8 @@
 				}
 			}
 		} finally {
-			try {
-				if (null != tableIterator) {
-					metadataDialect.close(tableIterator);
-				}
-			} catch (Exception ignore) {
-				SeamGuiPlugin.getPluginLog().logError(ignore);
+			if (null != tableIterator) {
+				metadataDialect.close(tableIterator);
 			}
 		}
 	}
@@ -148,12 +144,8 @@
 				tcc.addColumnName(catalogName, schemaName, quote(tableName), columnName);
 			}
 		} finally {
-			try {
-				if (null != columnIterator) {
-					metadataDialect.close(columnIterator);
-				}
-			} catch (Exception ignore) {
-				SeamGuiPlugin.getPluginLog().logError(ignore);
+			if (null != columnIterator) {
+				metadataDialect.close(columnIterator);
 			}
 		}
 	}



More information about the jbosstools-commits mailing list