Bug 369937 - DebugOptions are not updated correctly in Trace Preference Page

1 files changed, 5 insertions, 7 deletions

diff --git a/ui/org.eclipse.ui.trace/src/org/eclipse/ui/trace/internal/datamodel/ModifiedDebugOptions.java b/ui/org.eclipse.ui.trace/src/org/eclipse/ui/trace/internal/datamodel/ModifiedDebugOptions.java
index 0f9f2d9..0636820 100644
--- a/ui/org.eclipse.ui.trace/src/org/eclipse/ui/trace/internal/datamodel/ModifiedDebugOptions.java
+++ b/ui/org.eclipse.ui.trace/src/org/eclipse/ui/trace/internal/datamodel/ModifiedDebugOptions.java
@@ -1,5 +1,5 @@
 /*******************************************************************************

- * Copyright (c) 2011 IBM Corporation and others.

+ * Copyright (c) 2011, 2012 IBM Corporation and others.

  * All rights reserved. This program and the accompanying materials

  * are made available under the terms of the Eclipse Public License v1.0

  * which accompanies this distribution, and is available at

@@ -64,10 +64,9 @@ public class ModifiedDebugOptions {
 			if (isBeingRemoved) {

 				// remove it from the list of debug options to remove

 				this.debugOptionsToRemove.remove(option);

-			} else {

-				// add it to the list of debug options to add

-				this.debugOptionsToAdd.add(option);

 			}

+			// add it to the list of debug options to add

+			this.debugOptionsToAdd.add(option);

 		}

 	}

 

@@ -84,10 +83,9 @@ public class ModifiedDebugOptions {
 			if (isBeingAdded) {

 				// remove it from the list of debug options to add

 				this.debugOptionsToAdd.remove(option);

-			} else {

-				// add it to the list of debug options to remove

-				this.debugOptionsToRemove.add(option);

 			}

+			// add it to the list of debug options to remove

+			this.debugOptionsToRemove.add(option);

 		}

 	}