Author: rob.stryker(a)jboss.com Date: 2008-01-24 18:03:44 -0500 (Thu, 24 Jan 2008) New Revision: 5959 Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/views/server/extensions/JBossServerViewExtension.java Log: JBIDE-1487 removed try/catch. If an error is now thrown we should know. Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/views/server/extensions/JBossServerViewExtension.java =================================================================== --- trunk/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/views/server/extensions/JBossServerViewExtension.java 2008-01-24 22:22:42 UTC (rev 5958) +++ trunk/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/views/server/extensions/JBossServerViewExtension.java 2008-01-24 23:03:44 UTC (rev 5959) @@ -113,25 +113,22 @@ Runnable r = new Runnable() { public void run() { if( isEnabled() ) { - try { - if( o == null || o == provider ) { - JBossServerView.getDefault().getExtensionFrame().getViewer().refresh(provider); - } else { - ExtensionTableViewer viewer = JBossServerView.getDefault().getExtensionFrame().getViewer(); - ContentWrapper wrapped = new ContentWrapper(o, provider); - if( viewer.elementInTree(wrapped)) - viewer.refresh(new ContentWrapper(o, provider)); - else - viewer.refresh(provider); - } - } catch(Exception e) { - // non-critical error, ignore + if( o == null || o == provider ) { + JBossServerView.getDefault().getExtensionFrame().getViewer().refresh(provider); + } else { + ExtensionTableViewer viewer = JBossServerView.getDefault().getExtensionFrame().getViewer(); + ContentWrapper wrapped = new ContentWrapper(o, provider); + if( viewer.elementInTree(wrapped)) + viewer.refresh(new ContentWrapper(o, provider)); + else + viewer.refresh(provider); } } } }; - if( JBossServerView.getDefault() == null ) return; + if( JBossServerView.getDefault() == null ) + return; if( Display.getCurrent() == null ) Display.getDefault().asyncExec(r);