From 71659c8525d428ed6ddcc1792d4f112190e773af Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=B6ren=20Henning?= <stu114708@informatik.uni-kiel.de>
Date: Tue, 29 Mar 2016 12:14:44 +0200
Subject: [PATCH] updated TODO comments
---
.../kieker/analysis/graph/export/dot/GraphTransformer.java | 2 +-
src/main/java/kieker/analysis/graph/impl/GraphImpl.java | 6 +++++-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/main/java/kieker/analysis/graph/export/dot/GraphTransformer.java b/src/main/java/kieker/analysis/graph/export/dot/GraphTransformer.java
index bb54c471..15aaff81 100644
--- a/src/main/java/kieker/analysis/graph/export/dot/GraphTransformer.java
+++ b/src/main/java/kieker/analysis/graph/export/dot/GraphTransformer.java
@@ -28,10 +28,10 @@ public class GraphTransformer extends AbstractTransformer<String> {
GraphTransformer graphTransformer = new GraphTransformer(childGraph, false);
String childGraphString = graphTransformer.transform();
dotBuilder.addSubgraph(childGraphString);
+ // TODO Add style, label, etc
} else {
dotBuilder.addNode(vertex.getId().toString());
// TODO Add style, label, etc
- // TODO Handle child graph
}
}
diff --git a/src/main/java/kieker/analysis/graph/impl/GraphImpl.java b/src/main/java/kieker/analysis/graph/impl/GraphImpl.java
index 2dc03d57..ce3ef5ef 100644
--- a/src/main/java/kieker/analysis/graph/impl/GraphImpl.java
+++ b/src/main/java/kieker/analysis/graph/impl/GraphImpl.java
@@ -83,7 +83,11 @@ public class GraphImpl implements Graph {
}
}
- // TODO Maybe: Check whether both vertices are in the same graph
+ // TODO Check whether both vertices have the same root graph
+ // TODO If not, don't add the edge and throw exception
+
+ // TODO Check whether both vertices are in the same graph
+ // TODO If not, add edge to the "higher" one
final Edge edge = new EdgeImpl(idString, outVertex, inVertex, this);
this.edges.put(edge.getId().toString(), edge);
--
GitLab