Mercurial > hg > truffle

changeset 3611:abb55d24ba56

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
IdealGraphVisualizer: the side-by-side view of source graphs for diff graphs doesn't work properly because nodes that exist only in graph B (the 'new' graph) are in most cases assigned different ids. Show a standard one-graph view instead.
author Peter Hofer <peter.hofer@jku.at>
date Wed, 02 Nov 2011 17:05:21 +0100
parents e3e8c6bc9c14
children ed3ac862d22d
files src/share/tools/IdealGraphVisualizer/GraphTextEditor/src/com/sun/hotspot/igv/graphtexteditor/TextTopComponent.java
diffstat 1 files changed, 7 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/tools/IdealGraphVisualizer/GraphTextEditor/src/com/sun/hotspot/igv/graphtexteditor/TextTopComponent.java	Wed Nov 02 15:56:28 2011 +0100
+++ b/src/share/tools/IdealGraphVisualizer/GraphTextEditor/src/com/sun/hotspot/igv/graphtexteditor/TextTopComponent.java	Wed Nov 02 17:05:21 2011 +0100
@@ -280,7 +280,11 @@
     private void displayDiagram(Diagram diagram) {
         if (diagram == null) {
             showCard(NO_GRAPH);
-        } else if (diagram.getGraph().getSourceGraphs() != null) {
+        } /* This side-by-side view of the source graphs for diff graphs doesn't
+           * work properly because nodes that exist only in graph B (the 'new'
+           * graph) are in most cases assigned different ids.
+
+            else if (diagram.getGraph().getSourceGraphs() != null) {
             showCard(TWO_GRAPHS);
             Pair<InputGraph, InputGraph> graphs = diagram.getGraph().getSourceGraphs();
             leftEditor.setStructuredText(convert(graphs.getLeft(), diagram));
@@ -288,7 +292,8 @@
 
             // TODO: Hack to update view - remove
             SelectionCoordinator.getInstance().getHighlightedChangedEvent().fire();
-        } else {
+        } */
+        else {
             showCard(ONE_GRAPH);
             StructuredText text = convert(diagram.getGraph(), diagram);
             singleEditor.setStructuredText(text);