RED

refs 2490

org.fortiss.tooling.base/trunk/src/org/fortiss/tooling/base/utils/AnnotationUtils.java

@@ -80,6 +80,7 @@ public class AnnotationUtils {
 	 * @param element
 	 *            {@link IModelElement} for which annotations should be instantiated recursively.
 	 */
+	// TODO(VA) This method is used nowhere, is it really useful?
 	public static void instantiateAnnotationsRecursive(IModelElement element) {
 		IAnnotationValueService.getInstance().instantiateAnnotations(element);
 		for(IModelElement child : getChildrenWithType(element, IModelElement.class)) {

org.fortiss.tooling.base/trunk/src/org/fortiss/tooling/base/utils/BaseModelElementUtils.java

@@ -133,6 +133,8 @@ public class BaseModelElementUtils {
 	 *         {@link IConnector} contains exactly one {@link IConnection} that points to a
 	 *         successor {@link IConnector} of the desired type.
 	 */
+	// TODO(VA) This method always returns null (!)
+	// It is however not used anywhere so I think it can be removed.
 	@SuppressWarnings("unchecked")
 	public static <T extends IConnector> T getSuccessorConnector(IConnector connector,
 			Class<T> successorType) {

org.fortiss.tooling.base/trunk/src/org/fortiss/tooling/base/utils/EllipseLayoutUtils.java

@@ -108,6 +108,7 @@ public final class EllipseLayoutUtils {
 	}
 
 	/** Returns the layouted connector's angle. */
+	// TODO(VA) This method is never used -> remove?
 	public static Angle getConnectorAngle(ILayoutedModelElement layouted) {
 		Angle angle = getAngle(layouted, CONNECTOR_ANGLE);
 		if(angle == null) {

org.fortiss.tooling.base/trunk/src/org/fortiss/tooling/base/utils/MigrationUtils.java

@@ -35,6 +35,8 @@ import org.fortiss.tooling.kernel.model.INamedCommentedElement;
 /**
  * Utility methods for implementation of drop-in replacement migrations of (sub-types of) types
  * defined in {@code org.fortiss.tooling.base}.
+ *
+ * TODO(VA) This class is not used anywhere (anymore?) -> remove?
  * 
  * @author barner
  * @author $Author$
@@ -79,6 +81,7 @@ public class MigrationUtils {
 	 *            set, and it is replaces {@code oldIConnector} in the connector list of the
 	 *            corresponding owner model element.
 	 */
+	// TODO(VA) This method is not used (anymore?) -> remove
 	public static void migrateIConnector(IConnector oldIConnector, IConnector newIConnector) {
 		EList<IConnection> connections = new BasicEList<IConnection>(oldIConnector.getIncoming());
 		for(IConnection connection : connections) {

org.fortiss.tooling.base/trunk/src/org/fortiss/tooling/base/utils/VisualizationModelElementFactory.java

@@ -38,6 +38,7 @@ import org.fortiss.tooling.base.model.visualization.VisualizationPackage;
  */
 public class VisualizationModelElementFactory {
 	/** Creates a visualization {@link Dimension}. */
+	// TODO(VA) This method is not used anywhere -> remove?
 	public static <T> Dimension<T> createDimension(String name) {
 		Dimension<T> dimension = VisualizationFactory.eINSTANCE.createDimension();
 		dimension.setName(name);
@@ -62,6 +63,7 @@ public class VisualizationModelElementFactory {
 	}
 
 	/** Creates a visualization {@link DataSet}. */
+	// TODO(VA) This method is not used anywhere -> remove?
 	public static DataSet createDataSet(List<DataPoint<?>> dataPoints) {
 		DataSet dataSet = VisualizationFactory.eINSTANCE.createDataSet();
 		dataSet.getPoints().addAll(dataPoints);
@@ -69,6 +71,7 @@ public class VisualizationModelElementFactory {
 	}
 
 	/** Creates a visualization {@link DataSetCollection}. */
+	// TODO(VA) This method is not used anywhere -> remove?
 	@SuppressWarnings("unchecked")
 	public static DataSetCollection createDataSetCollection(List<DataSet> dataSets,
 			List<Axis<?>> axis, List<Dimension<?>> dims) {