]> granicus.if.org Git - graphviz/commitdiff
gvrender_beziercurve: realign 'filled' parameter type
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Tue, 9 Nov 2021 02:33:49 +0000 (18:33 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 14 Nov 2021 21:59:18 +0000 (13:59 -0800)
This appears to have been a mistake, in that all the other `gvrender_*`
functions take an `int` for the `filled` parameter. This does not appear to have
caused any visible effects, as the values passed to this function always fit in
a `boolean`. But the compiler complained that it thought unintended value
truncation was occurring.

Squashes 3 -Wconversion compiler warnings.

lib/gvc/gvcproc.h
lib/gvc/gvrender.c

index d3ce7feab17f98cee4f315c09e7b35fa26e9d3ec..f967438d5e17ae84076b6e80859f5fe7099655dc 100644 (file)
     void gvrender_polygon(GVJ_t* job, pointf* af, int n, int filled);
     void gvrender_box(GVJ_t * job, boxf BF, int filled);
     void gvrender_beziercurve(GVJ_t * job, pointf * AF, int n,
-                       int arrow_at_start, int arrow_at_end, boolean filled);
+                       int arrow_at_start, int arrow_at_end, int filled);
     void gvrender_polyline(GVJ_t * job, pointf * AF, int n);
     void gvrender_comment(GVJ_t * job, char *str);
     void gvrender_usershape(GVJ_t * job, char *name, pointf * AF, int n, boolean filled, char *imagescale, char *imagepos);
index d8974eecca93886e0f7275cd22e2a8f2c788d183..0752974cc3cb908a098781bd6d65ee7d17877a06 100644 (file)
@@ -606,8 +606,7 @@ void gvrender_box(GVJ_t * job, boxf B, int filled)
 }
 
 void gvrender_beziercurve(GVJ_t * job, pointf * af, int n,
-                         int arrow_at_start, int arrow_at_end,
-                         boolean filled)
+                         int arrow_at_start, int arrow_at_end, int filled)
 {
     gvrender_engine_t *gvre = job->render.engine;