some minor clean up

diff --git a/lib/gvc/gvdevice.c b/lib/gvc/gvdevice.c
index b8e524b7048fd92ff4d8df095a4a19a0feebd46f..e9a9d3d949f3c0fc0111227f3f1c484928e4ea66 100644 (file)
--- a/lib/gvc/gvdevice.c
+++ b/lib/gvc/gvdevice.c
@@ -49,7 +49,7 @@
 #include "gvcproc.h"
 
 
-size_t gvdevice_write (GVJ_t * job, char *s, unsigned int len)
+size_t gvdevice_write (GVJ_t * job, const unsigned char *s, unsigned int len)
 {
     if (job->flags & GVDEVICE_COMPRESSED_FORMAT) {
 #ifdef HAVE_LIBZ
@@ -60,11 +60,9 @@ size_t gvdevice_write (GVJ_t * job, char *s, unsigned int len)
        return fwrite(s, sizeof(char), len, job->output_file);
 }
 
-/* gvdevice selection is done in gvrender_select() in gvrender.c */
-
 void gvdevice_fputs(GVJ_t * job, char *s)
 {
-    gvdevice_write (job, s, strlen(s));
+    gvdevice_write (job, (unsigned char*)s, strlen(s));
 }
 
 /* gvdevice_printf:
@@ -76,15 +74,15 @@ void gvdevice_fputs(GVJ_t * job, char *s)
  */
 void gvdevice_printf(GVJ_t * job, const char *format, ...)
 {
-    char buf[BUFSIZ];
+    unsigned char buf[BUFSIZ];
     unsigned int len;
     va_list argp;
 
     va_start(argp, format);
 #ifdef HAVE_VSNPRINTF
-    len = vsnprintf(buf, sizeof(buf), format, argp);
+    len = vsnprintf((char *)buf, sizeof(buf), format, argp);
 #else
-    len = vsprintf(buf, format, argp);
+    len = vsprintf((char *)buf, format, argp);
 #endif
     va_end(argp);
 

diff --git a/plugin/gd/gvdevice_gd.c b/plugin/gd/gvdevice_gd.c
index 02b7f83ebcd219161e7f97f65fdb50d0b808fc99..43fbd8a39dd2b749f21feb351789da78a659e170 100644 (file)
--- a/plugin/gd/gvdevice_gd.c
+++ b/plugin/gd/gvdevice_gd.c
@@ -34,17 +34,17 @@ typedef enum {
        FORMAT_XBM,
 } format_type;
 
-extern size_t gvdevice_write(GVJ_t * job, char * s, unsigned int len);
+extern size_t gvdevice_write(GVJ_t * job, const unsigned char * s, unsigned int len);
 
 int gvdevice_gd_putBuf (gdIOCtx *context, const void *buffer, int len)
 {
-    return gvdevice_write((GVJ_t *)(context->tell), (char *)buffer, (unsigned int)len);
+    return gvdevice_write((GVJ_t *)(context->tell), buffer, (unsigned int)len);
 }
 
 /* used by gif output */
 void gvdevice_gd_putC (gdIOCtx *context, int C)
 {
-    char c = C;
+    unsigned char c = C;
 
     gvdevice_write((GVJ_t *)(context->tell), &c, 1);
 }

diff --git a/plugin/gd/gvrender_gd_vrml.c b/plugin/gd/gvrender_gd_vrml.c
index efb0b50be150607145cb2661bb6a8360125fa2ab..712915c221b7363a3352e91be9f5cf02b1995c5f 100644 (file)
--- a/plugin/gd/gvrender_gd_vrml.c
+++ b/plugin/gd/gvrender_gd_vrml.c
@@ -272,9 +272,9 @@ static void vrml_end_node(GVJ_t *job)
 {
     if (im) {
        gdImagePng(im, PNGfile);
+       fclose(PNGfile);
        gdImageDestroy(im);
        im = NULL;
-       fclose(PNGfile);
     }
 }
 

diff --git a/plugin/gdk_pixbuf/gvdevice_gdk_pixbuf.c b/plugin/gdk_pixbuf/gvdevice_gdk_pixbuf.c
index 2fd3d5ffbc9fe6fb787a486903aa5c342fcf6c1a..99d70e069cb2593ddf2fb92377656962cdcdbd8f 100644 (file)
--- a/plugin/gdk_pixbuf/gvdevice_gdk_pixbuf.c
+++ b/plugin/gdk_pixbuf/gvdevice_gdk_pixbuf.c
@@ -59,12 +59,12 @@ argb2rgba ( unsigned int width, unsigned int height, unsigned char *data)
     }
 }
 
-extern size_t gvdevice_write(GVJ_t * job, char *s, unsigned int len);
+extern size_t gvdevice_write(GVJ_t * job, const unsigned char *s, unsigned int len);
 
 static gboolean
 writer ( const gchar *buf, gsize count, GError **error, gpointer data)
 {
-    if (count == gvdevice_write((GVJ_t *)data, (char*)buf, count))
+    if (count == gvdevice_write((GVJ_t *)data, (const unsigned char*)buf, count))
         return TRUE;
     return FALSE;
 }

diff --git a/plugin/pango/gvrender_pango.c b/plugin/pango/gvrender_pango.c
index 44b813f62115f3da9f08efb12ea0aeab46c4f959..e8bee88e1e046049b883a24d62c3c0a3ad617f28 100644 (file)
--- a/plugin/pango/gvrender_pango.c
+++ b/plugin/pango/gvrender_pango.c
@@ -113,12 +113,12 @@ static void cairogen_set_color(cairo_t * cr, gvcolor_t * color)
                         color->u.RGBA[2], color->u.RGBA[3]);
 }
 
-extern size_t gvdevice_write(GVJ_t * job, char *s, unsigned int len);
+extern size_t gvdevice_write(GVJ_t * job, const unsigned char *s, unsigned int len);
 
 static cairo_status_t
 writer (void *closure, const unsigned char *data, unsigned int length)
 {
-    if (length == gvdevice_write((GVJ_t *)closure, (char*)data, length))
+    if (length == gvdevice_write((GVJ_t *)closure, data, length))
        return CAIRO_STATUS_SUCCESS;
     return CAIRO_STATUS_WRITE_ERROR;
 }