Changeset 3634:4302b0003d93 in roaraudio for libroardsp

Timestamp:

03/11/10 23:30:09 (14 years ago)

Author:

phi

Branch:

default

Phase:

public

Message:

bit more in direction of -Wextra

Location:

libroardsp

Files:

• Makefile (modified) (1 diff)
• alaw.c (modified) (1 diff)
• mulaw.c (modified) (1 diff)
• transcode.c (modified) (4 diffs)

libroardsp/Makefile

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

libroardsp/alaw.c

@@ -808 +808 @@
 int roardsp_conv_pcm162alaw (char * out, int16_t * in, size_t len) {
 #ifdef ROAR_SUPPORT_ALAW_RW
- int i;
+ unsigned int i;
 
  for (i = 0; i < len; i++) {

libroardsp/mulaw.c

@@ -1491 +1491 @@
 int roardsp_conv_pcm162mulaw (char * out, int16_t * in, size_t len) {
 #ifdef ROAR_SUPPORT_MULAW_RW
- int i;
+ unsigned int i;
 
  for (i = 0; i < len; i++) {

libroardsp/transcode.c

@@ -57 +57 @@
 
  for (i = 0; g_xcoders[i].codec != -1; i++) {
-  if ( g_xcoders[i].codec == info->codec ) {
+  if ( g_xcoders[i].codec == (int)info->codec ) {
    state->entry = &(g_xcoders[i]);
    break;
@@ -141 +141 @@
   return -1;
 
- if ( state->packet_len > 0 && state->packet_len != len )
+ if ( state->packet_len > 0 && state->packet_len != (ssize_t)len )
   return -1;
 
@@ -183 +183 @@
 
  if ( state->iobuffer == NULL ) {
-  while ( len >= state->packet_len ) {
+  while ( (ssize_t)len >= state->packet_len ) {
    if ( roar_xcoder_proc_packet(state, buf, state->packet_len) == -1 ) {
     ROAR_DBG("roar_xcoder_proc(state=%p, buf=%p, len=%lu) = -1 // roar_xcoder_proc_packet() error", state, buf, (unsigned long)len);
@@ -271 +271 @@
     }
 
-    if ( curlen < state->packet_len ) { // this should not happen...
+    if ( (ssize_t)curlen < state->packet_len ) { // this should not happen...
      roar_buffer_free(bufbuf);
      return -1;