Changeset 3641:6c894ae04329 in roaraudio

Timestamp:

03/11/10 23:59:11 (14 years ago)

Author:

phi

Branch:

default

Phase:

public

Message:

use -Wextra

Location:

roarclients

Files:

• Makefile (modified) (1 diff)
• polytest.c (modified) (1 diff)
• roarctl.c (modified) (1 diff)
• roarmonhttp.c (modified) (1 diff)
• roarvorbis.c (modified) (1 diff)
• roarvumeter.c (modified) (1 diff)

roarclients/Makefile

@@ -16 +16 @@
 #DEFINES        = -DDEBUG
 INCLUDE    = -I../include -I./include
-CFLAGS    += -g -Wall -O2 $(DEFINES) $(INCLUDE) $(INCPATH)
+CFLAGS    += -g -Wall -Wextra -O2 $(DEFINES) $(INCLUDE) $(INCPATH)
 LDFLAGS    = -g -L../lib/ $(LDPATH)
 LIBS       = $(LIBROAR)

roarclients/polytest.c

@@ -34 +34 @@
  size_t il = 10;
  size_t ol = 20;
- int i;
+ size_t i;
  int tmp;
 

roarclients/roarctl.c

@@ -327 +327 @@
    printf("Stream name           : %s\n", name);
 
-  if ( s.pos_rel_id == -1 ) {
+  if ( (int)s.pos_rel_id == -1 ) {
    printf("Relativ position id   : none (stream not synchronized)\n");
-  } else if ( s.pos_rel_id == id[i] ) {
+  } else if ( (int)s.pos_rel_id == id[i] ) {
    printf("Relativ position id   : %i (self synchronized)\n", s.pos_rel_id);
   } else {

roarclients/roarmonhttp.c

@@ -123 +123 @@
     }
 
-    if ( todo == len ) { // we wrote all of the pkg
+    if ( (ssize_t)todo == len ) { // we wrote all of the pkg
      if ( roar_buffer_next(&ring) == -1 )
       return -1;

roarclients/roarvorbis.c

@@ -86 +86 @@
  if ( *out == -1 ) {
   need_new_stream = 1;
- } else if ( info->rate != vi->rate || info->channels != vi->channels ) {
+ } else if ( info->rate != (uint16_t)vi->rate || info->channels != (uint16_t)vi->channels ) {
   need_new_stream = 1;
  }

roarclients/roarvumeter.c

@@ -72 +72 @@
  char * beat[2]     = {"     ", "Beat!"};
  char * dbeat       = beat[0];
- int have_beat;
+ int     have_beat  = 0;
  int16_t beat_val, beat_old;