Changeset 5651:b0e163908378 in roaraudio for roard

Timestamp:

09/12/12 16:22:42 (12 years ago)

Author:

phi

Branch:

default

Phase:

public

Message:

Added compiler checks for printf() like format strings.

Location:

roard

Files:

• codecfilter_flac.c (modified) (1 diff)
• codecfilter_vorbis.c (modified) (3 diffs)
• lib.c (modified) (1 diff)
• midi.c (modified) (2 diffs)
• roard.c (modified) (3 diffs)

roard/codecfilter_flac.c

@@ -153 +153 @@
    if ( stream_meta_add(ROAR_STREAM(self->ss)->id, type, "", value) == -1 ) {
     ROAR_WARN("cf_flac_cb_metadata(..., client_data=%p{ROAR_STREAM(.ss)->id=%i}): Can not add meta data: %s",
-              ROAR_STREAM(self->ss)->id, roar_error2str(roar_error));
+              self->ss, ROAR_STREAM(self->ss)->id, roar_error2str(roar_error));
    }
 

roard/codecfilter_vorbis.c

@@ -48 +48 @@
  r = stream_vio_s_read(ROAR_STREAM_SERVER(datasource), ptr, size*nmemb);
 
- ROAR_DBG("cf_vorbis_vfvio_read(ptr=%p, size=%lu, nmemb=%lu, datasource=%p): r=%i", ptr, size, nmemb, datasource, r);
+ ROAR_DBG("cf_vorbis_vfvio_read(ptr=%p, size=%lu, nmemb=%lu, datasource=%p): r=%i", ptr, (unsigned long int)size, (unsigned long int)nmemb, datasource, r);
 
  if ( r == -1 ) {
-  ROAR_DBG("cf_vorbis_vfvio_read(ptr=%p, size=%lu, nmemb=%lu, datasource=%p) = 0 // roar_error=%i(%s)", ptr, size, nmemb, datasource, roar_error, roar_error2str(roar_error));
+  ROAR_DBG("cf_vorbis_vfvio_read(ptr=%p, size=%lu, nmemb=%lu, datasource=%p) = 0 // roar_error=%i(%s)", ptr, (unsigned long int)size, (unsigned long int)nmemb, datasource, roar_error, roar_error2str(roar_error));
   return 0;
  }
@@ -61 +61 @@
  r /= size;
  
- ROAR_DBG("cf_vorbis_vfvio_read(ptr=%p, size=%lu, nmemb=%lu, datasource=%p) = %i", ptr, size, nmemb, datasource, r);
+ ROAR_DBG("cf_vorbis_vfvio_read(ptr=%p, size=%lu, nmemb=%lu, datasource=%p) = %i", ptr, (unsigned long int)size, (unsigned long int)nmemb, datasource, r);
  return r;
 }
@@ -289 +289 @@
  }
 
- ROAR_DBG("ov_read(*) = %i", done);
+ ROAR_DBG("ov_read(*) = %li", (long int)done);
 
  if ( done == 0 ) {

roard/lib.c

@@ -32 +32 @@
  size_t i;
 
- ROAR_WARN("lib_run_bg(cmd='%s', ...): This function should never be called. Contact devels.");
+ ROAR_WARN("lib_run_bg(cmd='%s', ...): This function should never be called. Contact devels.", cmd);
 
  child = roar_fork(NULL);

roard/midi.c

@@ -823 +823 @@
 
   if ( streams_get_flag(g_midi_clock.stream, ROAR_FLAG_SYNC) ) {
-   ROAR_DBG("midi_clock_tick(void): TICK! (nt=%lu)", g_midi_clock.nt);
+   ROAR_DBG("midi_clock_tick(void): TICK! (nt=%lu)", (unsigned long int)g_midi_clock.nt);
    if ( midi_new_bufmes(&buf, &mes) == -1 ) {
     ROAR_ERR("midi_clock_tick(void): Can not create Clock-Tick-Message");
@@ -839 +839 @@
    ss->state = ROAR_STREAMSTATE_OLD;
   } else {
-   ROAR_DBG("midi_clock_tick(void): silent tick. (nt=%lu)", g_midi_clock.nt);
+   ROAR_DBG("midi_clock_tick(void): silent tick. (nt=%lu)", (unsigned long int)g_midi_clock.nt);
   }
  }

roard/roard.c

@@ -977 +977 @@
    protoname = roar_proto2str(g_listen[i].proto);
    if ( protoname == NULL ) {
-    ROAR_ERR("check_listen(void): protocol %i is unknown (protocol ID not assigned?).");
+    ROAR_ERR("check_listen(void): protocol %i is unknown (protocol ID not assigned?).", g_listen[i].proto);
    } else {
     ROAR_DBG("check_listen(void): Unknown protocol %s(%i) is used.", protoname, g_listen[i].proto);
@@ -1226 +1226 @@
   if ( roar_x11_delete_prop(x11con, "ROAR_SERVER") == -1 ) {
    ret = -1;
-   ROAR_ERR("Error while unregistereing from X11", ROAR_DBG_INFO_INFO);
+   ROAR_ERR("Error while unregistereing from X11");
   } else {
    ROAR_INFO("Successfully unregistered from X11", ROAR_DBG_INFO_INFO);
@@ -1233 +1233 @@
   if ( roar_x11_set_prop(x11con, "ROAR_SERVER", sockname) == -1 ) {
    ret = -1;
-   ROAR_ERR("Error while registereing to X11", ROAR_DBG_INFO_INFO);
+   ROAR_ERR("Error while registereing to X11");
   } else {
    ROAR_INFO("Successfully registered to X11", ROAR_DBG_INFO_INFO);