From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Sat, 14 Jan 2023 05:26:25 +0000 (-0800)
Subject: smyrna DrawEllipse: use doubles for local temporaries
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c239a13258d4a4a4170b4e7db51469df842ea618;p=graphviz

smyrna DrawEllipse: use doubles for local temporaries

Values being stored into these were doubles and later use of these promoted them
to double. So it is not clear why they were not double to begin with. Squashes 2
-Wfloat-conversion warnings.
---

diff --git a/cmd/smyrna/draw.c b/cmd/smyrna/draw.c
index 800e2d34f..162175eb4 100644
--- a/cmd/smyrna/draw.c
+++ b/cmd/smyrna/draw.c
@@ -140,17 +140,15 @@ static void DrawBeziers(sdot_op* o, int param)
 //void DrawEllipse(xdot_point* xpoint,GLfloat xradius, GLfloat yradius,int filled)
 static void DrawEllipse(sdot_op*  o, int param)
 {
-    //to draw a circle set xradius and yradius same values
-    GLfloat x, y, xradius, yradius;
     int i = 0;
     int filled;
     xdot_op * op=&o->op;
     view->Topview->global_z += o->layer * LAYER_DIFF;
     set_options(param);
-    x = op->u.ellipse.x - dx;
-    y = op->u.ellipse.y - dy;
-    xradius = (GLfloat) op->u.ellipse.w;
-    yradius = (GLfloat) op->u.ellipse.h;
+    double x = op->u.ellipse.x - dx;
+    double y = op->u.ellipse.y - dy;
+    double xradius = op->u.ellipse.w;
+    double yradius = op->u.ellipse.h;
     if (op->kind == xd_filled_ellipse) {
 	if (param == 0)
 	    glColor4f(view->fillColor.R, view->fillColor.G,