From 03a711d82dee5277954624708640da02f2c300c4 Mon Sep 17 00:00:00 2001
From: knsv <knut@sveido.com>
Date: Mon, 5 Jan 2015 14:12:49 +0100
Subject: [PATCH] Cleanup of code

---
 .../sequenceDiagram/sequenceRenderer.js       |  6 ++--
 src/diagrams/sequenceDiagram/svgDraw.js       | 28 +++++++++----------
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/src/diagrams/sequenceDiagram/sequenceRenderer.js b/src/diagrams/sequenceDiagram/sequenceRenderer.js
index 8f844cc87..f2a9f1e38 100644
--- a/src/diagrams/sequenceDiagram/sequenceRenderer.js
+++ b/src/diagrams/sequenceDiagram/sequenceRenderer.js
@@ -302,6 +302,9 @@ module.exports.draw = function (text, id) {
     exports.bounds.init();
     var diagram = d3.select('#'+id);
 
+    var startx;
+    var stopx;
+
     // Fetch data from the parsing
     var actors = sq.yy.getActors();
     var actorKeys = sq.yy.getActorKeys();
@@ -381,9 +384,6 @@ module.exports.draw = function (text, id) {
                 drawMessage(diagram, startx, stopx, exports.bounds.getVerticalPos(), msg);
 
         }
-
-        var startx;
-        var stopx;
     });
 
     var box = exports.bounds.getBounds();
diff --git a/src/diagrams/sequenceDiagram/svgDraw.js b/src/diagrams/sequenceDiagram/svgDraw.js
index f4cf5cc2d..7c2043f4f 100644
--- a/src/diagrams/sequenceDiagram/svgDraw.js
+++ b/src/diagrams/sequenceDiagram/svgDraw.js
@@ -51,7 +51,7 @@ exports.drawLabel = function(elem , txtObject){
     rectData.class = 'labelBox';
     //rectData.color = 'white';
 
-    var label = exports.drawRect(elem, rectData);
+    exports.drawRect(elem, rectData);
 
     txtObject.y = txtObject.y + txtObject.labelMargin;
     txtObject.x = txtObject.x + 0.5*txtObject.labelMargin;
@@ -79,16 +79,16 @@ exports.drawActor = function(elem, left,description,conf){
         .attr("stroke-width", '0.5px')
         .attr("stroke", '#999');
 
-    g.append("rect")
-        .attr("x", left)
-        .attr("y", 0)
-        .attr("fill", '#eaeaea')
-        .attr("stroke", '#666')
-        .attr("width", conf.width)
-        .attr("height", conf.height)
-        .attr("class", 'actor')
-        .attr("rx", 3)
-        .attr("ry", 3);
+    var rect = exports.getNoteRect();
+    rect.x = left;
+    rect.fill = '#eaeaea';
+    rect.width = conf.width;
+    rect.height = conf.height;
+    rect.class = 'actor';
+    rect.rx = 3;
+    rect.ry = 3;
+    exports.drawRect(g, rect);
+
     g.append("text")      // text label for the x axis
         .attr("x", center)
         .attr("y", (conf.height/2)+5)
@@ -103,7 +103,7 @@ exports.drawActor = function(elem, left,description,conf){
 exports.insertArrowHead = function(elem){
     elem.append("defs").append("marker")
         .attr("id", "arrowhead")
-        .attr("refX", 5) /*must be smarter way to calculate shift*/
+        .attr("refX", 5)
         .attr("refY", 2)
         .attr("markerWidth", 6)
         .attr("markerHeight", 4)
@@ -117,11 +117,11 @@ exports.insertArrowHead = function(elem){
 exports.insertArrowCrossHead = function(elem){
     elem.append("defs").append("marker")
         .attr("id", "crosshead")
-        .attr("refX", 15) /*must be smarter way to calculate shift*/
-        .attr("refY", 4)
         .attr("markerWidth", 8)
         .attr("markerHeight", 8)
         .attr("orient", "auto")
+        .attr("refX", 15)
+        .attr("refY", 4)
         .append("path")
         .attr("fill",'none')
         .attr("stroke",'#000000')