Correct signature of internal function that does not require return value 15/178415/1
authorSangchul Lee <sc11.lee@samsung.com>
Thu, 10 May 2018 03:21:22 +0000 (12:21 +0900)
committerSangchul Lee <sc11.lee@samsung.com>
Thu, 10 May 2018 03:24:08 +0000 (12:24 +0900)
svace info : Redundant comparison 'ret' == '0' (0 == 0) is always true at
mm_sound_plugin_codec_wave.c:604.

[Version] 0.12.17
[Issue Type] SVACE (WGID 34314)

Change-Id: I9e89b3fca2b98f45a5e97ae30fb6a122ed2689d5
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
packaging/libmm-sound.spec
server/plugin/wav/mm_sound_plugin_codec_wave.c

index 3b5668f28d25cfe4658d4eee9409649fa8c3972c..af47ece0312496b13283495b17781a852a7f3fef 100644 (file)
@@ -1,6 +1,6 @@
 Name:       libmm-sound
 Summary:    MMSound Package contains client lib and sound_server binary
-Version:    0.12.16
+Version:    0.12.17
 Release:    0
 Group:      System/Libraries
 License:    Apache-2.0
index 563f3b13381c42136df647b400400c49d7a78f1b..2b9ccf6aa88a6d434922e2f57f6bf02dc8b3df7b 100644 (file)
@@ -451,7 +451,7 @@ static void _pa_stream_uncork(wave_info_t *h)
        pa_threaded_mainloop_unlock(h->m);
 }
 
-static int _pa_stream_stop_disconnect(wave_info_t *h)
+static void _pa_stream_stop_disconnect(wave_info_t *h)
 {
        assert(h);
        assert(h->m);
@@ -471,8 +471,6 @@ static int _pa_stream_stop_disconnect(wave_info_t *h)
 
        pa_threaded_mainloop_free(h->m);
        h->m = NULL;
-
-       return 0;
 }
 
 static int* _mm_sound_plug_codec_wave_get_supported_types(void)
@@ -581,7 +579,6 @@ static int _mm_sound_plug_codec_wave_play(MMHandleType handle)
 
 static int _mm_sound_plug_codec_wave_stop(MMHandleType handle)
 {
-       int ret = 0;
        wave_info_t *p = (wave_info_t*) handle;
 
        if (!p) {
@@ -591,12 +588,12 @@ static int _mm_sound_plug_codec_wave_stop(MMHandleType handle)
 
        debug_msg("Handle %p stop requested", p);
 
-       ret = _pa_stream_stop_disconnect(p);
+       _pa_stream_stop_disconnect(p);
 
        if (p->stop_cb)
                p->stop_cb(p->cb_param);
 
-       return (ret == 0) ? MM_ERROR_NONE : MM_ERROR_SOUND_INTERNAL;
+       return MM_ERROR_NONE;
 }
 
 static int _mm_sound_plug_codec_wave_destroy(MMHandleType handle)