Bug 399002 - Remove auto-generated TODOs in synchronized projects

Change-Id: Ibe0d7fe0d8917e443ad29c8bebba3006eab2cfff
Signed-off-by: John Eblen <jeblen@acm.org>

3 files changed, 6 insertions, 13 deletions

diff --git a/rdt/org.eclipse.ptp.rdt.sync.cdt.ui/src/org/eclipse/ptp/internal/rdt/sync/cdt/ui/wizards/NewRemoteSyncProjectWizardOperation.java b/rdt/org.eclipse.ptp.rdt.sync.cdt.ui/src/org/eclipse/ptp/internal/rdt/sync/cdt/ui/wizards/NewRemoteSyncProjectWizardOperation.java
index 5ffab44a9..049ad1322 100644
--- a/rdt/org.eclipse.ptp.rdt.sync.cdt.ui/src/org/eclipse/ptp/internal/rdt/sync/cdt/ui/wizards/NewRemoteSyncProjectWizardOperation.java
+++ b/rdt/org.eclipse.ptp.rdt.sync.cdt.ui/src/org/eclipse/ptp/internal/rdt/sync/cdt/ui/wizards/NewRemoteSyncProjectWizardOperation.java
@@ -38,7 +38,7 @@ import org.eclipse.ptp.rdt.sync.ui.ISynchronizeParticipant;
 /**
  * Static class that houses the function ("run") for initializing a new synchronized project.
  */
-public class NewRemoteSyncProjectWizardOperation implements Runnable {
+public class NewRemoteSyncProjectWizardOperation {
 	private static final String DEFAULT_BUILD_CONFIG_ID = "default-build-config-id"; //$NON-NLS-1$
 	private static final String ConfigMapKey = "config-map"; //$NON-NLS-1$
 
@@ -167,12 +167,6 @@ public class NewRemoteSyncProjectWizardOperation implements Runnable {
 		return RemoteSyncNature.hasNature(project);
 	}
 
-	@Override
-	public void run() {
-		// TODO Auto-generated method stub
-		
-	}
-	
 	private static IConfiguration findBuildConfigByName(String name, IConfiguration[] configArray) {
 		for (IConfiguration config : configArray) {
 			if (config.getName().equals(name)) {
diff --git a/rdt/org.eclipse.ptp.rdt.sync.core/src/org/eclipse/ptp/rdt/sync/core/resources/RemoteSyncNature.java b/rdt/org.eclipse.ptp.rdt.sync.core/src/org/eclipse/ptp/rdt/sync/core/resources/RemoteSyncNature.java
index 389844586..87a52fcb6 100644
--- a/rdt/org.eclipse.ptp.rdt.sync.core/src/org/eclipse/ptp/rdt/sync/core/resources/RemoteSyncNature.java
+++ b/rdt/org.eclipse.ptp.rdt.sync.core/src/org/eclipse/ptp/rdt/sync/core/resources/RemoteSyncNature.java
@@ -123,8 +123,7 @@ public class RemoteSyncNature implements IProjectNature {
 	 */
 	@Override
 	public void configure() throws CoreException {
-		// TODO Auto-generated method stub
-
+		// nothing to do
 	}
 
 	/*
@@ -134,8 +133,7 @@ public class RemoteSyncNature implements IProjectNature {
 	 */
 	@Override
 	public void deconfigure() throws CoreException {
-		// TODO Auto-generated method stub
-
+		// nothing to do
 	}
 
 	/*
diff --git a/rdt/org.eclipse.ptp.rdt.sync.ui/src/org/eclipse/ptp/internal/rdt/sync/ui/SyncMergeEditor.java b/rdt/org.eclipse.ptp.rdt.sync.ui/src/org/eclipse/ptp/internal/rdt/sync/ui/SyncMergeEditor.java
index 29a7fa63c..ae14d325e 100644
--- a/rdt/org.eclipse.ptp.rdt.sync.ui/src/org/eclipse/ptp/internal/rdt/sync/ui/SyncMergeEditor.java
+++ b/rdt/org.eclipse.ptp.rdt.sync.ui/src/org/eclipse/ptp/internal/rdt/sync/ui/SyncMergeEditor.java
@@ -146,6 +146,8 @@ public class SyncMergeEditor {
 		}
 	}
 
+	// IStreamContentAccessor interface needed for passing file contents.
+	// Not sure, though, why IModificationDate was implemented, but we keep it just in case.
 	private static class SyncMergeItem implements IStreamContentAccessor, ITypedElement, IModificationDate {
 		private final String content;
 
@@ -155,13 +157,12 @@ public class SyncMergeEditor {
 
 		@Override
 		public long getModificationDate() {
-			// TODO Auto-generated method stub
 			return 0;
 		}
 
 		@Override
 		public String getName() {
-			return "Compare Editor Test"; //$NON-NLS-1$
+			return "Compare Editor"; //$NON-NLS-1$
 		}
 
 		@Override