From ed8630fd3d43d9197404d58809bb2b08135d1cb7 Mon Sep 17 00:00:00 2001
From: Sabine Teufl <teufl@fortiss.org>
Date: Wed, 4 Jan 2012 16:25:10 +0000
Subject: [PATCH] Review refs 311
---
.../fortiss/tooling/kernel/internal/PersistencyService.java | 3 ---
.../fortiss/tooling/kernel/service/IPersistencyService.java | 1 -
.../kernel/service/listener/IPersistencyServiceListener.java | 3 ++-
3 files changed, 2 insertions(+), 5 deletions(-)
diff --git a/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/PersistencyService.java b/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/PersistencyService.java
index 93920e9b7..07a6e7540 100644
--- a/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/PersistencyService.java
+++ b/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;
}
diff --git a/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/service/IPersistencyService.java b/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/service/IPersistencyService.java
index 77ebd8b00..9b7db028a 100644
--- a/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/service/IPersistencyService.java
+++ b/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. */
diff --git a/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/service/listener/IPersistencyServiceListener.java b/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/service/listener/IPersistencyServiceListener.java
index 37bce5f2c..e646a4dd1 100644
--- a/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/service/listener/IPersistencyServiceListener.java
+++ b/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);
--
GitLab