Author: vrubezhny Date: 2007-09-11 13:14:53 -0400 (Tue, 11 Sep 2007) New Revision: 3570 Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/BreakpointImageProvider.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/ToggleBreakpointAction.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/properties/WebDataPreferencePage.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/properties/xpl/WebDataProperties.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionFormValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionForwardValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionMappingValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionMessagesVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayListValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayListVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayPartitionVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EmptyVariable.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationEntryValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationEntryVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ExceptionConfigValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/FilteredVariablesEnumeration.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapEntryValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapEntryVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashtableValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashtableVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestAttributesVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestHeadersVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestParametersVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletResponseVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpSessionAttributesVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpSessionVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/PageContextAttributesVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/PageContextVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextAttributesVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextInitParametersVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextVariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/StackFrameWrapper.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/StrutsData.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/TypeSupport.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ValueProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ValueProxyFactory.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/VariableProxy.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/VariableProxyFactory.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataView.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewContentProvider.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewEventHandler.java trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewInput.java Log: http://jira.jboss.org/jira/browse/JBIDE-894 Clean out jst component Code clean up is performed Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/BreakpointImageProvider.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/BreakpointImageProvider.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/BreakpointImageProvider.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -16,6 +16,7 @@ import org.eclipse.debug.core.model.IBreakpoint; import org.eclipse.jdt.debug.core.IJavaBreakpoint; import org.eclipse.ui.internal.ide.IMarkerImageProvider; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -51,7 +52,7 @@ if (installed) return iconPath + "brkpid_obj.gif"; //$NON-NLS-1$ return iconPath + "brkpd_obj.gif";//$NON-NLS-1$ } catch (CoreException e) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(e); } } return null; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/ToggleBreakpointAction.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/ToggleBreakpointAction.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/ToggleBreakpointAction.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -187,6 +187,7 @@ } catch (BadLocationException ex) { + WebDebugUIPlugin.getPluginLog().logError(ex); } } return false; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/properties/WebDataPreferencePage.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/properties/WebDataPreferencePage.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/properties/WebDataPreferencePage.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -24,6 +24,7 @@ import org.jboss.tools.common.model.ui.attribute.editor.IFieldEditor; import org.jboss.tools.common.model.ui.widgets.IWidgetSettings; import org.jboss.tools.common.model.ui.widgets.WhiteSettings; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.messages.xpl.WebUIMessages; /** @@ -142,7 +143,7 @@ gd = new GridData(GridData.FILL_BOTH); control[1].setLayoutData(gd); } catch (Exception e) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(e); } Control button = buttonController.createButton(treeButtonComposite); Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/properties/xpl/WebDataProperties.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/properties/xpl/WebDataProperties.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/properties/xpl/WebDataProperties.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -20,6 +20,7 @@ import org.eclipse.jface.preference.IPreferenceStore; import org.eclipse.jface.util.IPropertyChangeListener; import org.eclipse.jface.util.PropertyChangeEvent; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -77,7 +78,7 @@ try { fStore.addPropertyChangeListener(this); } catch (Exception x) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(x); } } @@ -149,7 +150,7 @@ } } } catch (Exception x) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(x); } } @@ -225,7 +226,7 @@ try { listeners[i].propertyChanged(property); } catch (Exception x) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(x); } } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionFormValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionFormValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionFormValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -22,6 +22,7 @@ import org.eclipse.jdt.debug.eval.IEvaluationResult; import org.eclipse.jdt.internal.debug.core.model.JDIDebugTarget; import org.eclipse.jdt.internal.debug.core.model.JDIValue; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; import com.sun.jdi.ClassType; @@ -101,7 +102,7 @@ } } catch (Exception e) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(e); } } } @@ -110,6 +111,7 @@ fVariables = (IVariable[])variables.toArray(new IVariable[variables.size()]); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionForwardValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionForwardValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionForwardValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -50,6 +51,7 @@ fVariables[2] = VariableProxyFactory.createVariable(fStackFrameWrapper, redirect, VariableProxy.class); fVariables[3] = VariableProxyFactory.createVariable(fStackFrameWrapper, contextRelative, VariableProxy.class); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionMappingValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionMappingValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionMappingValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -18,6 +18,7 @@ import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -100,6 +101,7 @@ fVariables = (IVariable[])list.toArray(new IVariable[list.size()]); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -114,6 +116,7 @@ globalForwards = EvaluationSupport.findVariableForName(mcVars, "forwards"); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return globalForwards; } @@ -129,6 +132,7 @@ globalExceptions = EvaluationSupport.findVariableForName(mcVars, "exceptions"); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return globalExceptions; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionMessagesVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionMessagesVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionMessagesVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -20,6 +20,7 @@ import org.eclipse.jdt.debug.eval.IEvaluationResult; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.eclipse.jdt.internal.debug.core.model.JDIThread; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -29,22 +30,19 @@ ActionMessagesVariableProxy(StackFrameWrapper frameWrapper, IVariable origin) { super(frameWrapper, origin); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } + IValue value = getOriginValue(origin); initValue(value); } ActionMessagesVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias) { super(frameWrapper, origin, alias); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } + IValue value = getOriginValue(origin); initValue(value); } ActionMessagesVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias, String type) { super(frameWrapper, origin, alias, type); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } + IValue value = getOriginValue(origin); initValue(value); } @@ -52,11 +50,23 @@ super(frameWrapper, result, alias, type); IValue value = null; if (result != null && !result.hasErrors()) { - try { value = result.getValue(); } catch (Exception e) { } + try { + value = result.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } initValue(value); } + private IValue getOriginValue(IVariable origin) { + try { + return origin.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + return null; + } + } private void initValue (IValue value) { fValue = (value == null || value instanceof JDINullValue ? null : new ActionMessagesValueProxy(fStackFrameWrapper, value)); } @@ -85,7 +95,9 @@ if(thisFrame == null || thisFrame.isTerminated()) return null; IVariable[] stackVars = null; try { stackVars = fStackFrameWrapper.getFrameVariables(); } - catch (Exception e) {} + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } IVariable variable = EvaluationSupport.findVariableForName(stackVars, "errors"); if (variable != null) return variable; @@ -99,7 +111,10 @@ IStackFrame currentFrame = (IStackFrame)frames.get(i); if (!thisFrame.equals(currentFrame)) { try { stackVars = fStackFrameWrapper.getFrameVariables(); } - catch (Exception e) {} + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } + variable = EvaluationSupport.findVariableForName(stackVars, "errors"); if (variable != null) return variable; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ActionValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -20,6 +20,7 @@ import org.eclipse.jdt.debug.core.IJavaValue; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.jboss.tools.jst.web.debug.ui.internal.views.properties.xpl.WebDataProperties; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -68,6 +69,7 @@ } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayListValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayListValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayListValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -18,6 +18,7 @@ import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -65,6 +66,7 @@ } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -81,6 +83,7 @@ int size = (fVariables == null ? 0 : fVariables.length); text = text.append(type).append('(').append(size).append(')'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayListVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayListVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayListVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -42,6 +43,7 @@ fValue = (value == null ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, ArrayListValueProxy.class)); ((ArrayListValueProxy)fValue).fVariables = null; } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } fHasValueChanged = false; return fValue; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayPartitionVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayPartitionVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayPartitionVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -35,6 +36,7 @@ value = getOriginValue(); fValue = (value == null ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, ArrayPartitionValueProxy.class)); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } fHasValueChanged = false; return fValue; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ArrayValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -19,6 +19,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.core.IJavaVariable; import org.eclipse.jdt.internal.debug.core.model.JDIValue; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -50,6 +51,7 @@ } fVariables = (IVariable[])variables.toArray(new IVariable[variables.size()]); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -88,6 +90,7 @@ // text = text.append(type).append('(').append(fVariables.size()).append(')'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EmptyVariable.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EmptyVariable.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EmptyVariable.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -23,6 +23,7 @@ import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.eclipse.jdt.internal.debug.core.model.JDIStackFrame; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.messages.xpl.WebUIMessages; /** @@ -57,6 +58,7 @@ if (showTypeName) text = text.append(getReferenceTypeName()).append(' '); text = text.append(getName()).append(WebUIMessages.VALUE_NOT_RESOLVED); } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return text.toString(); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationEntryValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationEntryValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationEntryValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -15,6 +15,7 @@ import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -62,6 +63,7 @@ fVariables[1] = VariableProxyFactory.createVariable(fStackFrameWrapper, value); ((VariableProxy)fVariables[1]).setOrigin(value); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -78,6 +80,7 @@ String value = fVariables[1].getValue().getValueString(); text = text.append('<').append(key).append("> : <").append(value).append('>'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationEntryVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationEntryVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationEntryVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -42,6 +43,7 @@ fValue = (value == null ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, EnumerationEntryValueProxy.class)); // ((HeadersEntryValueProxy)fValue).fVariables = null; } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } fHasValueChanged = false; return fValue; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/EnumerationValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -17,6 +17,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -61,6 +62,7 @@ fVariables = (IVariable[])list.toArray(new IVariable[list.size()]); } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -76,6 +78,7 @@ int size = (vars == null ? 0 : vars.length); text = text.append(type).append('(').append(size).append(')'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ExceptionConfigValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ExceptionConfigValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ExceptionConfigValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -49,6 +50,7 @@ fVariables[2] = new VariableProxy(fStackFrameWrapper, path); fVariables[3] = new VariableProxy(fStackFrameWrapper, scope); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/FilteredVariablesEnumeration.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/FilteredVariablesEnumeration.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/FilteredVariablesEnumeration.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -107,6 +107,7 @@ return false; } } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } } @@ -117,6 +118,7 @@ return true; } } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } } return false; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapEntryValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapEntryValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapEntryValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -46,6 +47,7 @@ fVariables[1] = VariableProxyFactory.createVariable(fStackFrameWrapper, value); ((VariableProxy)fVariables[1]).setOrigin(value); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -62,6 +64,7 @@ String value = fVariables[1].getValue().getValueString(); text = text.append('<').append(key).append("> : <").append(value).append('>'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapEntryVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapEntryVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapEntryVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -48,6 +49,7 @@ IValue value = getOriginValue(); fValue = (value == null ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, HashMapEntryValueProxy.class)); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } fHasValueChanged = false; return fValue; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -19,6 +19,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.core.IJavaVariable; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -62,6 +63,7 @@ } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -97,6 +99,7 @@ count++; } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } return count; @@ -113,6 +116,7 @@ int size = getMapSize(); text = text.append(type).append('(').append(size).append(')'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashMapVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -43,6 +44,7 @@ fValue = (value == null ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, HashMapValueProxy.class)); ((HashMapValueProxy)fValue).fVariables = null; } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } fHasValueChanged = false; return fValue; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashtableValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashtableValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashtableValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -19,6 +19,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.core.IJavaVariable; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -62,6 +63,7 @@ } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } private int addVariablesToList(List list, int count, IVariable[] vars) { @@ -80,6 +82,7 @@ count++; } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } return count; @@ -96,6 +99,7 @@ int size = (fVariables == null ? 0 : fVariables.length); text = text.append(type).append('(').append(size).append(')'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashtableVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashtableVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HashtableVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -14,6 +14,7 @@ import org.eclipse.debug.core.DebugException; import org.eclipse.debug.core.model.IValue; import org.eclipse.debug.core.model.IVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -42,6 +43,7 @@ fValue = (value == null ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, HashtableValueProxy.class)); ((HashtableValueProxy)fValue).fVariables = null; } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } fHasValueChanged = false; return fValue; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestAttributesVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestAttributesVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestAttributesVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -13,6 +13,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.eval.IEvaluationResult; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -47,6 +48,7 @@ } return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestHeadersVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestHeadersVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestHeadersVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -13,6 +13,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.eval.IEvaluationResult; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -52,6 +53,7 @@ } return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestParametersVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestParametersVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestParametersVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -13,6 +13,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.eval.IEvaluationResult; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -46,6 +47,7 @@ } return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -22,6 +22,7 @@ import org.eclipse.jdt.debug.eval.IEvaluationResult; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.jboss.tools.jst.web.debug.ui.internal.views.properties.xpl.WebDataProperties; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -78,7 +79,9 @@ IEvaluationResult resultValue = evaluateSessionVariable(); HttpSessionVariableProxy variable = new HttpSessionVariableProxy(fStackFrameWrapper, resultValue, "session", "javax.servlet.http.HttpSession"); list.add(variable); - } catch (Exception x) {} + } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); + } } } String stopWords = "inputStream reader"; @@ -86,7 +89,9 @@ try { IVariable var = (IVariable)list.get(i); stopWords += " " + var.getName(); - } catch (Exception x) {} + } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); + } } FilteredVariablesEnumeration filtered = @@ -117,6 +122,7 @@ } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -135,6 +141,7 @@ if (result == null || result.hasErrors()) return null; return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } @@ -153,6 +160,7 @@ if (result == null || result.hasErrors()) return null; return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletRequestVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -20,6 +20,7 @@ import org.eclipse.jdt.debug.eval.IEvaluationResult; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.eclipse.jdt.internal.debug.core.model.JDIThread; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -29,34 +30,41 @@ HttpServletRequestVariableProxy(StackFrameWrapper frameWrapper, IVariable origin) { super(frameWrapper, origin); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } HttpServletRequestVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias) { super(frameWrapper, origin, alias); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } HttpServletRequestVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias, String type) { super(frameWrapper, origin, alias, type); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } HttpServletRequestVariableProxy(StackFrameWrapper frameWrapper, IEvaluationResult result, String alias, String type) { super(frameWrapper, result, alias, type); IValue value = null; if (result != null && !result.hasErrors()) { - try { value = result.getValue(); } catch (Exception e) { } + try { + value = result.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } initValue(value); } + private IValue getOriginValue(IVariable origin) { + try { + return origin.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + return null; + } + } + private void initValue (IValue value) { fValue = (value == null || value instanceof JDINullValue ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, HttpServletRequestValueProxy.class)); @@ -93,7 +101,9 @@ if(thisFrame == null || thisFrame.isTerminated()) return null; IVariable[] stackVars = null; try { stackVars = stackFrameWrapper.getFrameVariables(); } - catch (Exception e) {} + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } IVariable variable = EvaluationSupport.findVariableForName(stackVars, "request"); if (variable != null) { @@ -109,7 +119,9 @@ IStackFrame currentFrame = (IStackFrame)frames.get(i); if (!thisFrame.equals(currentFrame)) { try { stackVars = stackFrameWrapper.getFrameVariables(); } - catch (Exception e) {} + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } variable = EvaluationSupport.findVariableForName(stackVars, "request"); if (variable != null) return substByInternalVariable(variable, "request", "javax.servlet.ServletRequest"); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletResponseVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletResponseVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpServletResponseVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -23,6 +23,7 @@ import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.eclipse.jdt.internal.debug.core.model.JDIThread; import org.jboss.tools.jst.web.debug.ui.internal.views.properties.xpl.WebDataProperties; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -32,34 +33,41 @@ HttpServletResponseVariableProxy(StackFrameWrapper frameWrapper, IVariable origin) { super(frameWrapper, origin); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } HttpServletResponseVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias) { super(frameWrapper, origin, alias); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } HttpServletResponseVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias, String type) { super(frameWrapper, origin, alias, type); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } HttpServletResponseVariableProxy(StackFrameWrapper frameWrapper, IEvaluationResult result, String alias, String type) { super(frameWrapper, result, alias, type); IValue value = null; if (result != null && !result.hasErrors()) { - try { value = result.getValue(); } catch (Exception e) { } + try { + value = result.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } initValue(value); } + private IValue getOriginValue(IVariable origin) { + try { + return origin.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + return null; + } + } + private void initValue (IValue value) { fValue = (value == null || value instanceof JDINullValue ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, HttpServletResponseValueProxy.class)); @@ -96,7 +104,9 @@ if(thisFrame == null || thisFrame.isTerminated()) return null; IVariable[] stackVars = null; try { stackVars = fStackFrameWrapper.getFrameVariables(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } IVariable variable = EvaluationSupport.findVariableForName(stackVars, "response"); if (variable != null) { @@ -112,7 +122,9 @@ IStackFrame currentFrame = (IStackFrame)frames.get(i); if (!thisFrame.equals(currentFrame)) { try { stackVars = fStackFrameWrapper.getFrameVariables(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } variable = EvaluationSupport.findVariableForName(stackVars, "response"); if (variable != null) return substByInternalVariable(variable, "response", "javax.servlet.ServletResponse"); } @@ -164,6 +176,7 @@ fVariables = (IVariable[])list.toArray(new IVariable[list.size()]); } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpSessionAttributesVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpSessionAttributesVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpSessionAttributesVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -13,6 +13,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.eval.IEvaluationResult; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -47,6 +48,7 @@ } return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpSessionVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpSessionVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/HttpSessionVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -24,6 +24,7 @@ import org.eclipse.jdt.debug.eval.IEvaluationResult; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.jboss.tools.jst.web.debug.ui.internal.views.properties.xpl.WebDataProperties; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -34,35 +35,42 @@ HttpSessionVariableProxy(StackFrameWrapper frameWrapper, IVariable origin) { super(frameWrapper, origin); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); - + initValue(getOriginValue(origin)); } HttpSessionVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias) { super(frameWrapper, origin, alias); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } HttpSessionVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias, String type) { super(frameWrapper, origin, alias, type); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } HttpSessionVariableProxy(StackFrameWrapper frameWrapper, IEvaluationResult result, String alias, String type) { super(frameWrapper, result, alias, type); IValue value = null; if (result != null && !result.hasErrors()) { - try { value = result.getValue(); } catch (Exception e) { } + try { + value = result.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } initValue(value); } + + private IValue getOriginValue(IVariable origin) { + try { + return origin.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + return null; + } + } + private void initValue (IValue value) { fValue = (value == null || value instanceof JDINullValue ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, HttpSessionValueProxy.class)); @@ -101,6 +109,7 @@ if (result == null || result.hasErrors()) return null; return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } @@ -143,7 +152,9 @@ try { IVariable var = (IVariable)list.get(i); stopWords += " " + var.getName(); - } catch (Exception x) {} + } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); + } } FilteredVariablesEnumeration filtered = @@ -174,6 +185,7 @@ } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/PageContextAttributesVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/PageContextAttributesVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/PageContextAttributesVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -13,6 +13,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.eval.IEvaluationResult; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -44,6 +45,7 @@ } return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/PageContextVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/PageContextVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/PageContextVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -25,6 +25,7 @@ import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.eclipse.jdt.internal.debug.core.model.JDIThread; import org.jboss.tools.jst.web.debug.ui.internal.views.properties.xpl.WebDataProperties; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -34,34 +35,41 @@ PageContextVariableProxy(StackFrameWrapper frameWrapper, IVariable origin) { super(frameWrapper, origin); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } PageContextVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias) { super(frameWrapper, origin, alias); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } PageContextVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias, String type) { super(frameWrapper, origin, alias, type); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } PageContextVariableProxy(StackFrameWrapper frameWrapper, IEvaluationResult result, String alias, String type) { super(frameWrapper, result, alias, type); IValue value = null; if (result != null && !result.hasErrors()) { - try { value = result.getValue(); } catch (Exception e) { } + try { + value = result.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } initValue(value); } + private IValue getOriginValue(IVariable origin) { + try { + return origin.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + return null; + } + } + private void initValue (IValue value) { fValue = (value == null || value instanceof JDINullValue ? null : ValueProxyFactory.createValueProxy(fStackFrameWrapper, value, PageContextValueProxy.class)); } @@ -97,7 +105,9 @@ if(thisFrame == null || thisFrame.isTerminated()) return null; IVariable[] stackVars = null; try { stackVars = fStackFrameWrapper.getFrameVariables(); } - catch (Exception e) {} + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } IVariable variable = EvaluationSupport.findVariableForName(stackVars, "pageContext"); if (variable != null) return variable; @@ -111,7 +121,9 @@ IStackFrame currentFrame = (IStackFrame)frames.get(i); if (!thisFrame.equals(currentFrame)) { try { stackVars = fStackFrameWrapper.getFrameVariables(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } variable = EvaluationSupport.findVariableForName(stackVars, "pageContext"); if (variable != null) return variable; } @@ -157,7 +169,9 @@ try { IVariable var = (IVariable)list.get(i); stopWords += " " + var.getName(); - } catch (Exception x) {} + } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); + } } FilteredVariablesEnumeration filtered = @@ -188,6 +202,7 @@ } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextAttributesVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextAttributesVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextAttributesVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -13,6 +13,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.eval.IEvaluationResult; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -48,6 +49,7 @@ } return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextInitParametersVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextInitParametersVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextInitParametersVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -13,6 +13,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.eval.IEvaluationResult; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -48,6 +49,7 @@ } return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextVariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextVariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ServletContextVariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -24,6 +24,7 @@ import org.eclipse.jdt.debug.eval.IEvaluationResult; import org.eclipse.jdt.internal.debug.core.model.JDINullValue; import org.jboss.tools.jst.web.debug.ui.internal.views.properties.xpl.WebDataProperties; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; import org.jboss.tools.jst.web.debug.xpl.EvaluationSupport; /** @@ -35,34 +36,41 @@ ServletContextVariableProxy(StackFrameWrapper frameWrapper, IVariable origin) { super(frameWrapper, origin); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } ServletContextVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias) { super(frameWrapper, origin, alias); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } ServletContextVariableProxy(StackFrameWrapper frameWrapper, IVariable origin, String alias, String type) { super(frameWrapper, origin, alias, type); - IValue value = null; - try { value = origin.getValue(); } catch (Exception e) { } - initValue(value); + initValue(getOriginValue(origin)); } ServletContextVariableProxy(StackFrameWrapper frameWrapper, IEvaluationResult result, String alias, String type) { super(frameWrapper, result, alias, type); IValue value = null; if (result != null && !result.hasErrors()) { - try { value = result.getValue(); } catch (Exception e) { } + try { + value = result.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } initValue(value); } + private IValue getOriginValue(IVariable origin) { + try { + return origin.getValue(); + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + return null; + } + } + private void initValue (IValue value) { fValue = (value == null || value instanceof JDINullValue ? null : new ServletContextValueProxy(fStackFrameWrapper, value)); } @@ -101,6 +109,7 @@ if (result == null || result.hasErrors()) return null; return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } @@ -148,7 +157,9 @@ try { IVariable var = (IVariable)list.get(i); stopWords += " " + var.getName(); - } catch (Exception x) {} + } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); + } } FilteredVariablesEnumeration filtered = @@ -179,6 +190,7 @@ } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/StackFrameWrapper.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/StackFrameWrapper.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/StackFrameWrapper.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -94,14 +94,18 @@ if (fHasValueChanged) fWebData.updateVariables(); vars[count] = fWebData; count++; - } catch (Exception e) { } + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } if (wdp.isEnabledFilter(WebDataProperties.SHOW_STRUTS_VARIABLES_PROPERTY)) { try { if (fHasValueChanged) fStrutsData.updateVariables(); vars[count] = fStrutsData; count++; - } catch (Exception e) { } + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } IVariable[] newVars = new IVariable[count]; for (int i = 0; i < count; i++) newVars[i] = vars[i]; Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/StrutsData.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/StrutsData.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/StrutsData.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -111,7 +111,9 @@ if (resultValue instanceof IVariable) { var.setOrigin((IVariable)resultValue); try { var.fAlias = ((IVariable)resultValue).getName(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } } else { var.fAlias = "mapping"; //$NON-NLS-1$ var.fType = "org.apache.struts.action.ActionMapping"; //$NON-NLS-1$ @@ -262,6 +264,7 @@ if (result == null || result.hasErrors()) return null; return result; } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } return null; } @@ -272,7 +275,9 @@ if(thisFrame == null || thisFrame.isTerminated()) return null; IVariable[] stackVars = null; try { stackVars = parent.getFrameVariables(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } IVariable variable = EvaluationSupport.findVariableForName(stackVars, "mapping"); //$NON-NLS-1$ if (variable != null) return variable; @@ -291,7 +296,9 @@ IStackFrame currentFrame = (IStackFrame)frames.get(i); if (!thisFrame.equals(currentFrame)) { try { stackVars = parent.getFrameVariables(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } variable = EvaluationSupport.findVariableForName(stackVars, "mapping"); //$NON-NLS-1$ if (variable != null) return variable; variable = EvaluationSupport.findVariableForName(stackVars, "actionMapping"); //$NON-NLS-1$ @@ -307,7 +314,9 @@ if(thisFrame == null || thisFrame.isTerminated()) return null; IVariable[] stackVars = null; try { stackVars = parent.getFrameVariables(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } IVariable variable = EvaluationSupport.findVariableForName(stackVars, "form"); //$NON-NLS-1$ if (variable != null) return variable; @@ -320,6 +329,7 @@ return thisVariable; } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } JDIThread thread = (JDIThread)thisFrame.getThread(); @@ -331,7 +341,9 @@ IStackFrame currentFrame = (IStackFrame)frames.get(i); if (!thisFrame.equals(currentFrame)) { try { stackVars = parent.getFrameVariables(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } variable = EvaluationSupport.findVariableForName(stackVars, "form"); //$NON-NLS-1$ if (variable != null) return variable; } @@ -348,6 +360,7 @@ if (result == null || result.hasErrors() || !result.getValue().isAllocated()) return null; return result; } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return null; } @@ -358,7 +371,9 @@ if(thisFrame == null || thisFrame.isTerminated()) return null; IVariable[] stackVars = null; try { stackVars = parent.getFrameVariables(); } - catch (Exception e) { } + catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } IVariable thisVariable = EvaluationSupport.findVariableForName(stackVars, "this"); //$NON-NLS-1$ if(thisVariable == null) return null; @@ -367,6 +382,7 @@ return thisVariable; } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return null; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/TypeSupport.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/TypeSupport.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/TypeSupport.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -18,6 +18,7 @@ import org.eclipse.jdt.debug.core.IJavaClassType; import org.eclipse.jdt.debug.core.IJavaInterfaceType; import org.eclipse.jdt.debug.core.IJavaType; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -42,6 +43,7 @@ try { return SIMPLE_TYPES.contains(javaType.getName()); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return false; } @@ -52,7 +54,9 @@ if (isClassType(javaType)) { return SIMPLE_TYPE_WRAPPERS.contains(javaType.getName()); } - } catch (Exception x) { } + } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); + } return false; } @@ -118,6 +122,7 @@ } } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return false; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ValueProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ValueProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ValueProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -67,6 +67,7 @@ fVariables = (IVariable[])list.toArray(new IVariable[list.size()]); } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } } @@ -141,7 +142,9 @@ if (!TypeSupport.isSimpleTypeOrWrapper(((IJavaValue)fOrigin).getJavaType())) { return fOrigin.hasVariables(); } - } catch (Exception e) { } + } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); + } return false; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ValueProxyFactory.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ValueProxyFactory.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/ValueProxyFactory.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -13,6 +13,7 @@ import org.eclipse.debug.core.model.IValue; import org.eclipse.jdt.internal.debug.core.model.JDIValue; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -40,6 +41,7 @@ try { vp = (IValue)cls.getDeclaredConstructor(new Class[]{StackFrameWrapper.class, IValue.class}).newInstance(new Object[]{frameWrapper, value}); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } // if(value != null && vp != null) valueProxies.put(value, vp); } @@ -96,6 +98,7 @@ return new ValueProxy(frameWrapper, value); } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return new ValueProxy(frameWrapper, proxy); @@ -165,6 +168,7 @@ return new ValueProxy(frameWrapper, value); } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return new ValueProxy(frameWrapper, proxy); Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/VariableProxy.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/VariableProxy.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/VariableProxy.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -139,6 +139,7 @@ getReferenceTypeName())); } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } fHasValueChanged = false; setChangeCount(fStackFrameWrapper.getChangeCount()); @@ -149,6 +150,7 @@ try { return fResult.getValue(); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return null; } @@ -157,6 +159,7 @@ try { return fOrigin.getValue(); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return null; } @@ -276,6 +279,7 @@ return v; } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return variable; } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/VariableProxyFactory.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/VariableProxyFactory.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/VariableProxyFactory.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -18,6 +18,7 @@ import org.eclipse.debug.core.model.IVariable; import org.eclipse.jdt.debug.core.IJavaType; import org.eclipse.jdt.debug.core.IJavaVariable; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * @author Jeremy @@ -44,6 +45,7 @@ try { vp = (VariableProxy)cls.getDeclaredConstructor(new Class[]{StackFrameWrapper.class, IVariable.class}).newInstance(new Object[]{frameWrapper, origin}); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } if(origin != null && vp != null) variableProxies.put(origin, vp); } @@ -76,6 +78,7 @@ return new ActionMessagesVariableProxy(frameWrapper, origin); } } catch (Exception ex) { + WebDebugUIPlugin.getPluginLog().logError(ex); } return new VariableProxy(frameWrapper, origin); } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataView.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataView.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataView.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -208,6 +208,7 @@ return getVariableProxyText((IVariable) element, true); } } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return super.getText(element); } @@ -247,6 +248,7 @@ if (notNull && quoteValue) text = text.append('"'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } @@ -280,6 +282,7 @@ if (notNull && quoteValue) text = text.append('"'); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } @@ -293,6 +296,7 @@ ' '); text = text.append(variable.getName()); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return text.toString(); } @@ -334,6 +338,7 @@ try { origin = ((ActionMappingValueProxy) value).getOrigin(); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } if (origin != null) { ProxyValueDetailListener pl = new ProxyValueDetailListener( @@ -350,6 +355,7 @@ try { origin = ((ValueProxy) value).getOrigin(); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } if (origin != null) { ProxyValueDetailListener pl = new ProxyValueDetailListener( @@ -670,7 +676,7 @@ loadTree(); } catch (Exception x) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(x); } } @@ -728,6 +734,7 @@ try { t = d.getThreads(); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } for (int i = 0; i < t.length; i++) fExpandedVariables.remove(t[i]); Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewContentProvider.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewContentProvider.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewContentProvider.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -27,6 +27,7 @@ import org.eclipse.jface.viewers.IContentProvider; import org.eclipse.jface.viewers.ITreeContentProvider; import org.eclipse.jface.viewers.Viewer; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; /** * Provide the contents for a variables viewer. @@ -78,7 +79,7 @@ try { children = ((IVariable)parent).getValue().getVariables(); } catch (Exception ee) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(ee); } } if (children != null) { @@ -86,7 +87,7 @@ return children; } } catch (Exception e) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(e); } return new Object[0]; } @@ -208,7 +209,7 @@ clearCache(); this.fViewer = viewer; } catch (Exception x) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(x); } } @@ -216,7 +217,7 @@ try { fWrapper.doChange(); } catch (Exception x) { - //ignore + WebDebugUIPlugin.getPluginLog().logError(x); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewEventHandler.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewEventHandler.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewEventHandler.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -114,6 +114,7 @@ // TODO-3.3: WTP 2.0 // ((WebDataViewContentProvider) getWebDataView().getWebDataViewer().getContentProvider()).postChange(); } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } // TODO replace with valid code /// refresh(); @@ -169,6 +170,7 @@ /// refresh(); getWebDataView().populateDetailPane(); } catch (Exception x) { + WebDebugUIPlugin.getPluginLog().logError(x); } } Modified: trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewInput.java =================================================================== --- trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewInput.java 2007-09-11 17:11:55 UTC (rev 3569) +++ trunk/jst/plugins/org.jboss.tools.jst.web.debug.ui/src/org/jboss/tools/jst/web/debug/ui/internal/views/xpl/WebDataViewInput.java 2007-09-11 17:14:53 UTC (rev 3570) @@ -17,6 +17,7 @@ import org.eclipse.debug.core.model.IStackFrame; import org.eclipse.debug.core.model.IThread; +import org.jboss.tools.jst.web.debug.ui.xpl.WebDebugUIPlugin; public class WebDataViewInput { @@ -27,6 +28,7 @@ try { if(frame != null && !frame.isTerminated()) thread = frame.getThread(); } catch (Exception e) { + WebDebugUIPlugin.getPluginLog().logError(e); } return create(thread); }