]> granicus.if.org Git - graphviz/commitdiff
smyrna DrawEllipse: use doubles for local temporaries
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 14 Jan 2023 05:26:25 +0000 (21:26 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 15 Jan 2023 16:59:31 +0000 (08:59 -0800)
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.

cmd/smyrna/draw.c

index 800e2d34fca1029416e2c9a35f3744e3aeb56956..162175eb4b503b82fa504dc6f4bf7d066cdd9013 100644 (file)
@@ -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,