[jbosstools-commits] JBoss Tools SVN: r22860 - in trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core: impl/definition and 1 other directories.

[jbosstools-commits at lists.jboss.org]

Author: akazakov
Date: 2010-06-17 14:20:18 -0400 (Thu, 17 Jun 2010)
New Revision: 22860

Modified:
   trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/ParametedType.java
   trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition/ParametedTypeFactory.java
   trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/validation/errorList.txt
Log:
Added exception logging.

Modified: trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/ParametedType.java
===================================================================
--- trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/ParametedType.java	2010-06-17 17:56:43 UTC (rev 22859)
+++ trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/ParametedType.java	2010-06-17 18:20:18 UTC (rev 22860)
@@ -168,7 +168,7 @@
 					try {
 						sc = resolveParameters(sc);
 					} catch (Exception e) {
-						e.printStackTrace();
+						CDICorePlugin.getDefault().logError(e);
 					}
 				} else if(!"java.lang.Object".equals(type.getFullyQualifiedName())) {
 					sc = "QObject;";

Modified: trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition/ParametedTypeFactory.java
===================================================================
--- trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition/ParametedTypeFactory.java	2010-06-17 17:56:43 UTC (rev 22859)
+++ trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/impl/definition/ParametedTypeFactory.java	2010-06-17 18:20:18 UTC (rev 22860)
@@ -60,7 +60,7 @@
 		try {
 			ps = type.getTypeParameterSignatures();
 		} catch (JavaModelException e) {
-			//ignore
+			CDICorePlugin.getDefault().logError(e);
 		}
 		if(ps != null && ps.length > 0) {
 			for (int i = 0; i < ps.length; i++) {
@@ -68,9 +68,9 @@
 					ParametedType p = getParametedTypeForParameter(type, ps[i], null);
 					if(p != null) parametedType.addParameter(p);
 				} catch (JavaModelException e) {
-					e.printStackTrace();
+					CDICorePlugin.getDefault().logError(e);
 				} catch (Exception ee) {
-					ee.printStackTrace();
+					CDICorePlugin.getDefault().logError(ee);
 				}
 				
 			}
@@ -150,9 +150,9 @@
 				StringBuffer newParams = new StringBuffer();
 				String[] paramSignatures = null;
 				try {
-				paramSignatures = Signature.getTypeArguments(typeSignature);
+					paramSignatures = Signature.getTypeArguments(typeSignature);
 				} catch (Exception e) {
-					e.printStackTrace();
+					CDICorePlugin.getDefault().logError(e);
 				}
 				if(paramSignatures != null) for (String paramSignature: paramSignatures) {
 					ParametedType param = getParametedType(context, paramSignature);

Modified: trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/validation/errorList.txt
===================================================================
--- trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/validation/errorList.txt	2010-06-17 17:56:43 UTC (rev 22859)
+++ trunk/cdi/plugins/org.jboss.tools.cdi.core/src/org/jboss/tools/cdi/internal/core/validation/errorList.txt	2010-06-17 18:20:18 UTC (rev 22860)
@@ -141,15 +141,11 @@
 5.2.5. Qualifier annotations with members
 - array-valued or annotation-valued member of a qualifier type is not annotated @Nonbinding (Non-Portable behavior)
 
-
-
-
-
 5.5.7. Injection point metadata
 - bean that declares any scope other than @Dependent has an injection point of type 
   InjectionPoint and qualifier @Default
 - Java EE component class supporting injection that is not a bean has an injection 
-  point of type InjectionPoint and qualifier @Default
+  point of type InjectionPoint and qualifier @Default (JBT: let's ignore this rule so far since we don't have any good tool to recognize such components)