# HG changeset patch # User Peter Hofer # Date 1320249921 -3600 # Node ID abb55d24ba56dffb5752d38ada4b60b281dd3381 # Parent e3e8c6bc9c14ef286b5b66f983f2569972943a15 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. diff -r e3e8c6bc9c14 -r abb55d24ba56 src/share/tools/IdealGraphVisualizer/GraphTextEditor/src/com/sun/hotspot/igv/graphtexteditor/TextTopComponent.java --- 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 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);