# HG changeset patch # User Andreas Woess # Date 1329085314 -3600 # Node ID 24d9f3310ed63dbf3a8b37e0f5feadafc0b0c9e8 # Parent 76841bdd5f3e346f64173f462effbb8f8db7d0e1 Visualizer: Fix edge tooltip showing wrong node ids. GraalEdgeColorFilter: Don't change style of dashed lines. diff -r 76841bdd5f3e -r 24d9f3310ed6 src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java --- a/src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java Sat Feb 11 23:35:44 2012 +0100 +++ b/src/share/tools/IdealGraphVisualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java Sun Feb 12 23:21:54 2012 +0100 @@ -35,7 +35,7 @@ /** * Filter that colors usage and successor edges differently. - * + * * @author Peter Hofer */ public class GraalEdgeColorFilter extends AbstractFilter { @@ -77,7 +77,9 @@ for (Connection c : is.getConnections()) { if (c.getLabel() == null || !c.getLabel().endsWith("#NDF")) { c.setColor(color); - c.setStyle(style); + if (c.getStyle() != ConnectionStyle.DASHED) { + c.setStyle(style); + } } else if ("EndNode".equals(c.getOutputSlot().getFigure().getProperties().get("class")) || "EndNode".equals(c.getOutputSlot().getProperties().get("class"))) { c.setColor(successorColor); diff -r 76841bdd5f3e -r 24d9f3310ed6 src/share/tools/IdealGraphVisualizer/Graph/src/com/sun/hotspot/igv/graph/Connection.java --- a/src/share/tools/IdealGraphVisualizer/Graph/src/com/sun/hotspot/igv/graph/Connection.java Sat Feb 11 23:35:44 2012 +0100 +++ b/src/share/tools/IdealGraphVisualizer/Graph/src/com/sun/hotspot/igv/graph/Connection.java Sun Feb 12 23:21:54 2012 +0100 @@ -32,7 +32,7 @@ import java.util.List; /** - * + * * @author Thomas Wuerthinger */ public class Connection implements Source.Provider, Link { @@ -112,7 +112,7 @@ outputSlot.getFigure().removeSuccessor(inputSlot.getFigure()); outputSlot.connections.remove(this); } - + public String getToolTipText() { StringBuilder builder = new StringBuilder(); if (label != null) { @@ -120,9 +120,9 @@ } else { builder.append("From "); } - builder.append(getOutputSlot().getFigure()); + builder.append(getOutputSlot().getFigure().getSource().getSourceNodes().get(0).getId()); builder.append(" to "); - builder.append(getInputSlot().getFigure()); + builder.append(getInputSlot().getFigure().getSource().getSourceNodes().get(0).getId()); return builder.toString(); } diff -r 76841bdd5f3e -r 24d9f3310ed6 visualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java --- a/visualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java Sat Feb 11 23:35:44 2012 +0100 +++ b/visualizer/Graal/src/com/sun/hotspot/igv/graal/filters/GraalEdgeColorFilter.java Sun Feb 12 23:21:54 2012 +0100 @@ -35,7 +35,7 @@ /** * Filter that colors usage and successor edges differently. - * + * * @author Peter Hofer */ public class GraalEdgeColorFilter extends AbstractFilter { @@ -77,7 +77,9 @@ for (Connection c : is.getConnections()) { if (c.getLabel() == null || !c.getLabel().endsWith("#NDF")) { c.setColor(color); - c.setStyle(style); + if (c.getStyle() != ConnectionStyle.DASHED) { + c.setStyle(style); + } } else if ("EndNode".equals(c.getOutputSlot().getFigure().getProperties().get("class")) || "EndNode".equals(c.getOutputSlot().getProperties().get("class"))) { c.setColor(successorColor); diff -r 76841bdd5f3e -r 24d9f3310ed6 visualizer/Graph/src/com/sun/hotspot/igv/graph/Connection.java --- a/visualizer/Graph/src/com/sun/hotspot/igv/graph/Connection.java Sat Feb 11 23:35:44 2012 +0100 +++ b/visualizer/Graph/src/com/sun/hotspot/igv/graph/Connection.java Sun Feb 12 23:21:54 2012 +0100 @@ -32,7 +32,7 @@ import java.util.List; /** - * + * * @author Thomas Wuerthinger */ public class Connection implements Source.Provider, Link { @@ -112,7 +112,7 @@ outputSlot.getFigure().removeSuccessor(inputSlot.getFigure()); outputSlot.connections.remove(this); } - + public String getToolTipText() { StringBuilder builder = new StringBuilder(); if (label != null) { @@ -120,9 +120,9 @@ } else { builder.append("From "); } - builder.append(getOutputSlot().getFigure()); + builder.append(getOutputSlot().getFigure().getSource().getSourceNodes().get(0).getId()); builder.append(" to "); - builder.append(getInputSlot().getFigure()); + builder.append(getInputSlot().getFigure().getSource().getSourceNodes().get(0).getId()); return builder.toString(); }