Review

refs 311

org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/PersistencyService.java

@@ -54,7 +54,6 @@ import org.osgi.framework.Bundle;
  * @version $Rev$
  * @ConQAT.Rating YELLOW Hash: 4F70931B343453F4C70065EF91708C65
  */
-// TODO: add information for which option (XML file) this class is implemented
 public class PersistencyService implements IPersistencyService {
 
 	/** The prototype provider extension point ID. */
@@ -106,8 +105,6 @@ public class PersistencyService implements IPersistencyService {
 	@Override
 	public synchronized boolean isDirty() {
 		for (ITopLevelElement context : contextCache) {
-			// TODO: warum kann man das nicht unsaved nennen, sondern nennt es
-			// dirty, wenn unsaved gemeint ist?
 			if (context.isDirty()) {
 				return true;
 			}

org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/service/IPersistencyService.java

@@ -69,7 +69,6 @@ public interface IPersistencyService {
 	ITopLevelElement getTopLevelElementFor(EObject modelElement);
 
 	/** Returns whether the some storage provider has unsaved changes. */
-	// TODO: warum nennt man unsaved dirty?
 	boolean isDirty();
 
 	/** Saves any dirty top-level elements to disk. */

org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/service/listener/IPersistencyServiceListener.java

@@ -39,7 +39,8 @@ public interface IPersistencyServiceListener {
 	void topLevelElementRemoved(ITopLevelElement element);
 
 	/** Notifies the listener that the underlying persistent model has changed. */
-	// Would it be useful to have the information (here: persistent model) in
+	// TODO: Would it be useful to have the information (here: persistent model)
+	// in
 	// the name?
 	void topLevelElementChanged(ITopLevelElement element);