API review/cleanup

5 files changed, 43 insertions, 8 deletions

diff --git a/org.eclipse.debug.ui/schema/breakpointOrganizers.exsd b/org.eclipse.debug.ui/schema/breakpointOrganizers.exsd
index 3b2b3c627..7f48c34f2 100644
--- a/org.eclipse.debug.ui/schema/breakpointOrganizers.exsd
+++ b/org.eclipse.debug.ui/schema/breakpointOrganizers.exsd
@@ -73,7 +73,7 @@ Organizers with the specified name will be automatically created by the Debug Pl
          <attribute name="icon" type="string">
             <annotation>
                <documentation>
-                  Path to an optional icon which can be shown for this organizer.
+                  Optional path to an icon which can be shown for this organizer.
                </documentation>
             </annotation>
          </attribute>
@@ -109,7 +109,7 @@ Organizers with the specified name will be automatically created by the Debug Pl
 &lt;extension
     point=&quot;org.eclipse.debug.ui.breakpointOrganizers&quot;&gt;
     &lt;breakpointOrganizer
-  class=&quot;com.example.BreakpointOrganizer&quot;
+  	 class=&quot;com.example.BreakpointOrganizer&quot;
      id=&quot;com.example.BreakpointOrganizer&quot;
      label=&quot;Example Organizer&quot;
      icon=&quot;icons/full/obj16/example_org.gif&quot;/&gt;
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/actions/RemoveBreakpointAction.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/actions/RemoveBreakpointAction.java
index d02dbdd58..aac5f2155 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/actions/RemoveBreakpointAction.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/actions/RemoveBreakpointAction.java
@@ -78,7 +78,6 @@ public class RemoveBreakpointAction extends AbstractRemoveActionDelegate {
                             	Runnable r = new Runnable() {
 								
 									public void run() {
-										// TODO Auto-generated method stub
 										((BreakpointsView) getView()).preserveSelectionState(state);
 									}
 								};
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/variables/VariablesView.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/variables/VariablesView.java
index 469b3066c..1d869059a 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/variables/VariablesView.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/internal/ui/views/variables/VariablesView.java
@@ -860,7 +860,7 @@ public class VariablesView extends AbstractDebugEventHandlerView implements ISel
 			public void run() {
 				IAction findAction= null;
 				if (getDetailViewer().getTextWidget().isFocusControl()) {
-					//XXX Still using "old" resource access
+					//TODO: Still using "old" resource access
 					ResourceBundle bundle= ResourceBundle.getBundle("org.eclipse.debug.internal.ui.views.variables.VariablesViewMessages"); //$NON-NLS-1$
 					findAction= new FindReplaceAction(bundle, "find_replace_action.", VariablesView.this);	 //$NON-NLS-1$
 				} else {
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/IBreakpointOrganizerDelegate.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/IBreakpointOrganizerDelegate.java
index 3b42e6a3c..0ce0b567a 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/IBreakpointOrganizerDelegate.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/IBreakpointOrganizerDelegate.java
@@ -25,7 +25,26 @@ import org.eclipse.jface.util.IPropertyChangeListener;
  * Organizers may optionally support breakpoint recategorization. 
  * </p>
  * <p>
- * TODO: plug-in XML
+ * Following is example plug-in XML for contributing a breakpoint organizer.
+ * <pre>
+ * &lt;extension point="org.eclipse.debug.ui.breakpointOrganizers"&gt;
+ * 	&lt;breakpointOrganizer
+ * 		class="com.example.BreakpointOrganizer"
+ *      id="com.example.BreakpointOrganizer"
+ *      label="Example Organizer"
+ *      icon="icons/full/obj16/example_org.gif"/&gt;
+ * &lt;/extension&gt;
+ * </pre>
+ * The attributes are specified as follows:
+ * <ul>
+ * <li><code>class</code> Fully qualified name of a Java class that implements
+ * {@link IBreakpointOrganizerDelegate}.</li>
+ * <li><code>id</code> Unique identifier for this breakpoint organizer.</li>
+ * <li><code>label</code> Label for this organizer which is suitable for
+ * presentation to the user.</li>
+ * <li><code>icon</code> Optional path to an icon which can be shown for this
+ * organizer</li>
+ * </ul>
  * </p>
  * <p>
  * Clients contributing a breakpoint organizer are intended to implement
diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/actions/IVariableValueEditor.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/actions/IVariableValueEditor.java
index 6809cad95..0fc8f9813 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/actions/IVariableValueEditor.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/actions/IVariableValueEditor.java
@@ -17,9 +17,26 @@ import org.eclipse.swt.widgets.Shell;
  * A variable value editor allows the user to edit a variable's value.
  * Variable value editors are contributed for a debug model via the
  * <code>org.eclipse.debug.ui.variableValueEditors</code> extension point.
- * 
- * TODO: example plug-in XML
- * 
+ * <p>
+ * Following is example plug-in XML for contributing a variable value editor.
+ * <pre>
+ * &lt;extension point="org.eclipse.debug.ui.variableValueEditors"&gt;
+ *    &lt;variableEditor
+ *       modelId="com.examples.myDebugModel"
+ *       class="com.examples.variables.MyVariableValueEditor"/&gt;
+ * &lt;/extension&gt;
+ * </pre>
+ * The attributes are specified as follows:
+ * <ul>
+ * <li><code>modelId</code> the debug model identifier for which the given
+ * variable value editor is applicable</li>
+ * <li><code>class</code> fully qualified name of a class that implements 
+ * {@link IVariableValueEditor}</li>
+ * </ul>
+ * </p>
+ * <p>
+ * Clients may implement this interface.
+ * </p>
  * @since 3.1
  */
 public interface IVariableValueEditor {