Author: dgolovin
Date: 2010-11-05 18:17:43 -0400 (Fri, 05 Nov 2010)
New Revision: 26311
Modified:
trunk/common/plugins/org.jboss.tools.common.el.core/src/org/jboss/tools/common/el/core/resolver/ELResolverFactoryManager.java
Log:
catch (InvalidRegistryObjectException e) block removed from ELResolverFactoryManager:
1. It is runtime exception
2. natures[k].createExecutableExtension("resolver-class"); is actually has
catch block for this exception which converts InvalidRegistryObjectException into
CoreException
so catch for CoreException is enough here
Modified:
trunk/common/plugins/org.jboss.tools.common.el.core/src/org/jboss/tools/common/el/core/resolver/ELResolverFactoryManager.java
===================================================================
---
trunk/common/plugins/org.jboss.tools.common.el.core/src/org/jboss/tools/common/el/core/resolver/ELResolverFactoryManager.java 2010-11-05
18:11:36 UTC (rev 26310)
+++
trunk/common/plugins/org.jboss.tools.common.el.core/src/org/jboss/tools/common/el/core/resolver/ELResolverFactoryManager.java 2010-11-05
22:17:43 UTC (rev 26311)
@@ -61,7 +61,7 @@
// return result;
// }
Set<ELResolver> resolverSet = new HashSet<ELResolver>();
- IExtensionRegistry registry = Platform.getExtensionRegistry();
+ IExtensionRegistry registry = Platform.getExtensionRegistry();
IExtensionPoint extensionPoint =
registry.getExtensionPoint("org.jboss.tools.common.el.core.elResolver");
//$NON-NLS-1$
IExtension[] extensions = extensionPoint.getExtensions();
for (int i=0; i<extensions.length; i++) {
@@ -80,8 +80,6 @@
ELCorePlugin.getPluginLog().logError(resolver.getClass().getName() + " must
be instance of org.jboss.tools.common.el.core.resolver.ELResolver"); //$NON-NLS-1$
}
}
- } catch (InvalidRegistryObjectException e) {
- ELCorePlugin.getPluginLog().logError(e);
} catch (CoreException e) {
ELCorePlugin.getPluginLog().logError(e);
}
@@ -107,6 +105,6 @@
// result = resolverSet.toArray(new ELResolver[resolverSet.size()]);
// resolvers.put(project.getName(), result);
// return result;
- return resolverSet.toArray(new ELResolver[0]);
+ return resolverSet.toArray(new ELResolver[resolverSet.size()]);
}
}
\ No newline at end of file
Show replies by date