From daa1113fec7aecec275f4fa291185017fd07a086 Mon Sep 17 00:00:00 2001
From: Kjell Morgenstern <kjell@gmx.de>
Date: Sun, 22 Nov 2020 07:35:45 +0100
Subject: [PATCH] Fix minor rounding error. Remove unused variable

---
 src/items/ruler.cpp | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/src/items/ruler.cpp b/src/items/ruler.cpp
index 152a78063..4bbe0ba12 100644
--- a/src/items/ruler.cpp
+++ b/src/items/ruler.cpp
@@ -117,12 +117,9 @@ QString Ruler::makeSvg(double inches) {
 
 	// get units
 	int units = m_modelPart->localProp("width").toString().contains("cm") ? IndexCm : IndexIn;
-
 	QString svg = TextUtils::makeSVGHeader(GraphicsUtils::SVGDPI, GraphicsUtils::StandardFritzingDPI, (inches + offset + offset) * GraphicsUtils::SVGDPI, GraphicsUtils::SVGDPI/2);
-	int counter;
 
 	if (units == IndexCm) {
-		counter = 0;
 		svg += "<g font-family='Droid Sans' text-anchor='middle' font-size='100' stroke-width='1' style='stroke: rgb(100,100,100)'>";
 
 		// 1/100 centimeter spacing
@@ -191,9 +188,7 @@ QString Ruler::makeSvg(double inches) {
 			double x = (offset + (i / 16.0)) * GraphicsUtils::StandardFritzingDPI;
 			if (i % 16 == 0) {
 
-				// I wonder if QT compiler would see that this is just a constant, and not perform the runtime math?
-				// If not calculating out the value could save some cycles:)
-				h = 0.196875;//.125 + (1.15 / 16);
+				h = .125 + (1.15 / 16);
 
 				double y = (GraphicsUtils::StandardFritzingDPI/4)+35;//(h + .101) * GraphicsUtils::StandardFritzingDPI;
 				svg += QString("<text x='%1' y='%2'>%3</text>")
@@ -205,10 +200,10 @@ QString Ruler::makeSvg(double inches) {
 				}
 			}
 			else if (i % 8 == 0) {
-				h = 0.17812;//.125 +  (.85 / 16);
+				h = .125 +  (.85 / 16);
 			}
 			else if (i % 4 == 0) {
-				h = 0.14062;//.125 +  (.25 / 16);
+				h = .125 +  (.25 / 16);
 			}
 
 			svg += QString("<line x1='%1' y1='0' x2='%1' y2='%2' />\n")