Changeset 5270:e25346c13638 in roaraudio for libroaresd/esdctl.c

Timestamp:

11/17/11 18:20:12 (12 years ago)

Author:

phi

Branch:

default

Phase:

public

Message:

fixed some gcc -Wextra warnings

File:

• libroaresd/esdctl.c (modified) (8 diffs)

libroaresd/esdctl.c

@@ -45 +45 @@
 // so always return -1
 int esd_lock( int esd ) {
+ (void)esd;
+ errno = ENOSYS;
  return -1;
 }
 int esd_unlock( int esd ) {
+ (void)esd;
+ errno = ENOSYS;
  return -1;
 }
@@ -66 +70 @@
 
  return roar_set_standby(&con, ROAR_STANDBY_INACTIVE);
+}
+
+static void _format2str(char * buf, size_t len, esd_format_t format) {
+
+ if ( (format & ESD_BITS16) == ESD_BITS16 ) {
+  roar_mm_strlcat(buf, "16 bit ", len);
+ } else {
+  roar_mm_strlcat(buf, "8 bit ", len);
+ }
+
+ if ( (format & ESD_STEREO) == ESD_STEREO ) {
+  roar_mm_strlcat(buf, "stereo ", len);
+ } else {
+  roar_mm_strlcat(buf, "mono ", len);
+ }
 }
 
@@ -72 +91 @@
  char buf[80] = "";
 
- if ( server_info->format & ESD_BITS16 )
-  strcat(buf, "16 bit ");
- else
-  strcat(buf, "8 bit ");
-
- if ( server_info->format & ESD_STEREO )
-  strcat(buf, "stereo ");
- else
-  strcat(buf, "mono ");
+ _format2str(buf, sizeof(buf), server_info->format);
 
  printf("server version   = %i\n",        server_info->version);
@@ -87 +98 @@
 }
 
-
 void esd_print_player_info( esd_player_info_t *player_info ) {
  char buf[80] = "";
 
- if ( (player_info->format & ESD_BITS16) == ESD_BITS16 )
-  strcat(buf, "16 bit ");
- else
-  strcat(buf, "8 bit ");
-
- if ( (player_info->format & ESD_STEREO) == ESD_STEREO )
-  strcat(buf, "stereo ");
- else
-  strcat(buf, "mono ");
+ _format2str(buf, sizeof(buf), player_info->format);
 
  printf("player %i name    = %s\n",        player_info->source_id, player_info->name);
@@ -109 +111 @@
 
 void esd_print_sample_info( esd_sample_info_t *sample_info ) {
+ char buf[80] = "";
+
+ _format2str(buf, sizeof(buf), sample_info->format);
+
+ printf("sample %i name    = %s\n",        sample_info->sample_id, sample_info->name);
+ printf("sample %i format  = 0x%08x %s\n", sample_info->sample_id, sample_info->format, buf);
+ printf("sample %i rate    = %i\n",        sample_info->sample_id, sample_info->rate);
+ printf("sample %i left    = %i\n",        sample_info->sample_id, sample_info->left_vol_scale);
+ printf("sample %i right   = %i\n",        sample_info->sample_id, sample_info->right_vol_scale);
+ printf("sample %i length  = %i\n",        sample_info->sample_id, sample_info->length);
 }
 
@@ -282 +294 @@
 /* retrieve all information from server, and update until unsubsribed or closed */
 esd_info_t *esd_subscribe_all_info( int esd ) {
+ (void)esd;
+ errno = ENOSYS;
  return NULL; // Not yet implemented in upstream esd
 }
@@ -288 +302 @@
 esd_info_t *esd_update_info( int esd, esd_info_t *info,
                              esd_update_info_callbacks_t *callbacks ) {
+ (void)esd, (void)info, (void)callbacks;
+ errno = ENOSYS;
  return NULL; // Not yet implemented in upstream esd
 }
 
 esd_info_t *esd_unsubscribe_info( int esd ) {
+ (void)esd;
+ errno = ENOSYS;
  return NULL; // Not yet implemented in upstream esd
 }
@@ -362 +380 @@
 int esd_set_default_sample_pan( int esd, int sample_id,
                                 int left_scale, int right_scale ) {
+ // this is not yet implemented by the RA protocol.
+ (void)esd, (void)sample_id, (void)left_scale, (void)right_scale;
+ errno = ENOSYS;
  return -1;
 }