[NONE] Add TODOs to indicate label computation has to be enhanced.

eclipse · Aug 17, 2015 · 2577aeb · 2577aeb
1 parent dce2d5f
commit 2577aeb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/org.eclipse.gef4.mvc/src/org/eclipse/gef4/mvc/operations/ForwardUndoCompositeOperation.java b/org.eclipse.gef4.mvc/src/org/eclipse/gef4/mvc/operations/ForwardUndoCompositeOperation.java
@@ -11,6 +11,7 @@
  *******************************************************************************/
 package org.eclipse.gef4.mvc.operations;
 
+//TODO: init label when adding nested operations
 public class ForwardUndoCompositeOperation extends AbstractCompositeOperation {
 
 	public ForwardUndoCompositeOperation(String label) {

diff --git a/org.eclipse.gef4.mvc/src/org/eclipse/gef4/mvc/operations/ReverseUndoCompositeOperation.java b/org.eclipse.gef4.mvc/src/org/eclipse/gef4/mvc/operations/ReverseUndoCompositeOperation.java
@@ -7,7 +7,7 @@
  *
  * Contributors:
  *     Alexander Nyßen (itemis AG) - initial API and implementation
- *     
+ *
  *******************************************************************************/
 package org.eclipse.gef4.mvc.operations;
 
@@ -20,6 +20,7 @@
 import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Status;
 
+// TODO: init label when adding nested operations
 public class ReverseUndoCompositeOperation extends AbstractCompositeOperation {
 
 	public ReverseUndoCompositeOperation(String label) {
@@ -30,8 +31,8 @@ public ReverseUndoCompositeOperation(String label) {
 	public IStatus undo(IProgressMonitor monitor, IAdaptable info)
 			throws ExecutionException {
 		IStatus status = Status.OK_STATUS;
-		ListIterator<IUndoableOperation> li = getOperations().listIterator(
-				getOperations().size());
+		ListIterator<IUndoableOperation> li = getOperations()
+				.listIterator(getOperations().size());
 		while (li.hasPrevious()) {
 			status = combine(status, li.previous().undo(monitor, info));
 		}