Changeset 5111:090c5c2831a7 in roaraudio

Timestamp:

07/31/11 20:43:58 (13 years ago)

Author:

phi

Branch:

default

Phase:

public

Message:

fixed some compiler warnings

Files:

• include/roaraudio/compilerhacks.h (modified) (1 diff)
• libroar/buffer.c (modified) (2 diffs)
• libroar/debug.c (modified) (2 diffs)
• libroar/enumdev.c (modified) (1 diff)
• libroar/vio_cmd.c (modified) (1 diff)
• libroar/vio_rtp.c (modified) (1 diff)
• libroar/vs.c (modified) (1 diff)

include/roaraudio/compilerhacks.h

@@ -58 +58 @@
 #endif
 
+#ifndef _LIBROAR_IGNORE_RET
+#define _LIBROAR_IGNORE_RET(x) ((void)((x)+1))
+#endif
+
 #ifndef _LIBROAR_NOATTR_TO_STATIC
 #define _LIBROAR_ATTR_TO_STATIC _LIBROAR_ATTR_DEPRECATED

libroar/buffer.c

@@ -542 +542 @@
  *copy = NULL;
 
+ // TODO:  This function does not handle ring buffers.
+ // FIXME: This function does not error out in case of wrong buffer type in buf.
+ // TODO:  This function does not handle buffer meta data.
+
  while (cur != NULL) {
   if ( roar_buffer_new(&new, cur->user_len) == -1 ) {
@@ -553 +557 @@
    *copy = new;
 
-  roar_buffer_get_data(cur, &od);
-  roar_buffer_get_data(new, &nd);
+  _LIBROAR_IGNORE_RET(roar_buffer_get_data(cur, &od));
+  _LIBROAR_IGNORE_RET(roar_buffer_get_data(new, &nd));
   memcpy(nd, od, cur->user_len);
 
-  roar_buffer_add(*copy, new);
+  _LIBROAR_IGNORE_RET(roar_buffer_add(*copy, new));
 
   cur = cur->next;

libroar/debug.c

@@ -135 +135 @@
 
  if ( vio != NULL ) {
-  roar_vio_write(vio, buf, ret);
+  _LIBROAR_IGNORE_RET(roar_vio_write(vio, buf, ret));
   roar_vio_sync(vio);
  } else {
@@ -207 +207 @@
      return;
     }
-    roar_vio_write(vio, buf, len);
-    roar_vio_write(vio, "\n", 1);
+    _LIBROAR_IGNORE_RET(roar_vio_write(vio, buf, len));
+    _LIBROAR_IGNORE_RET(roar_vio_write(vio, "\n", 1));
     roar_vio_sync(vio);
    break;

libroar/enumdev.c

@@ -280 +280 @@
  int is_uniq;
 
- // load config:
- roar_libroar_get_config();
-
  if ( flags & ROAR_ENUM_FLAG_HARDNONBLOCK )
   flags |= ROAR_ENUM_FLAG_NONBLOCK;

libroar/vio_cmd.c

@@ -522 +522 @@
   self->options -= ROAR_VIO_CMD_OPTS_NONBLOCK;
 
- roar_vio_nonblock(self->next, state); // this should help, but may not nessessery.
+ _LIBROAR_IGNORE_RET(roar_vio_nonblock(self->next, state)); // this should help, but may not nessessery.
 
  return 0;

libroar/vio_rtp.c

@@ -394 +394 @@
   self->bpf                 = info->channels * info->bits / 8;
 
-  roar_vio_ctl(self->vio, cmd, data);
+  _LIBROAR_IGNORE_RET(roar_vio_ctl(self->vio, cmd, data));
   return 0;
  }

libroar/vs.c

@@ -1541 +1541 @@
  if ( vss->async.level == ROAR_VS_ASYNCLEVEL_AUTO && !vss->async.lock &&
       vss->async.read_latency                     && !vss->async.qlen ) {
-  roar_vs_latency2(vss, vss->async.last_backend < 0 ? ROAR_VS_BACKEND_NONE : vss->async.last_backend, ROAR_VS_ASYNC, NULL);
+  _LIBROAR_IGNORE_RET(roar_vs_latency2(vss,
+                                       vss->async.last_backend < 0 ? ROAR_VS_BACKEND_NONE : vss->async.last_backend,
+                                       ROAR_VS_ASYNC, NULL));
  }