commit c9d8d895c577b0ecdf920eaa6f37b588a656384b
author David Benjamin <davidben@google.com> Wed Sep 21 12:04:55 2016 -0400
committer David Benjamin <davidben@google.com> Wed Sep 21 17:44:55 2016 -0400
tree c61f52c0068a9ed8cc2bb4b7458315f09f54029d
parent 909f884d5926ccee810ec0c8825469fd6fd20e6a

Fix compiler warnings.

This should fix everything but the platform_get_snd_device_name_extn
warnings which will be in a separate CL.

Change-Id: Iace30af1cb7754e0bcfd25de8dc55c2c059e30c6
Test: Still compiles.

visualizer/offload_visualizer.c

diff --git a/visualizer/offload_visualizer.c b/visualizer/offload_visualizer.c
index 69cea57..8b1c469 100644
--- a/visualizer/offload_visualizer.c
+++ b/visualizer/offload_visualizer.c
@@ -81,11 +81,11 @@
     effect_ops_t ops;
 };
 
-typedef struct output_context_s {
+struct output_context_s {
     struct listnode outputs_list_node;  /* node in active_outputs_list */
     audio_io_handle_t handle; /* io handle */
     struct listnode effects_list; /* list of effects attached to this output */
-} output_context_t;
+};
 
 
 /* maximum time since last capture buffer update before resetting capture buffer. This means
@@ -427,7 +427,7 @@
 
 __attribute__ ((visibility ("default")))
 int visualizer_hal_start_output(audio_io_handle_t output, int pcm_id) {
-    int ret;
+    int ret = 0;
     struct listnode *node;
 
     ALOGV("%s output %d pcm_id %d", __func__, output, pcm_id);
@@ -473,7 +473,7 @@
 
 __attribute__ ((visibility ("default")))
 int visualizer_hal_stop_output(audio_io_handle_t output, int pcm_id) {
-    int ret;
+    int ret = 0;
     struct listnode *node;
     struct listnode *fx_node;
     output_context_t *out_ctxt;
@@ -869,8 +869,8 @@
     case VISUALIZER_CMD_MEASURE: {
         if (pReplyData == NULL || replySize == NULL ||
                 *replySize < (sizeof(int32_t) * MEASUREMENT_COUNT)) {
-            ALOGV("%s VISUALIZER_CMD_MEASURE error *replySize %d <"
-                    "(sizeof(int32_t) * MEASUREMENT_COUNT) %d",
+            ALOGV("%s VISUALIZER_CMD_MEASURE error *replySize %u <"
+                    "(sizeof(int32_t) * MEASUREMENT_COUNT) %zu",
                     __func__, *replySize, sizeof(int32_t) * MEASUREMENT_COUNT);
             android_errorWriteLog(0x534e4554, "30229821");
             return -EINVAL;