[retro-gtk/runner-cleanup: 1/4] pa-player: Use g_clear_pointer()



commit 7d149cb9d9ca2ab2210046e2e254fc6400b336bc
Author: Adrien Plazas <kekun plazas laposte net>
Date:   Sat May 23 11:21:36 2020 +0200

    pa-player: Use g_clear_pointer()
    
    This makes the code more concise.

 retro-runner/retro-pa-player.c | 17 +++--------------
 1 file changed, 3 insertions(+), 14 deletions(-)
---
diff --git a/retro-runner/retro-pa-player.c b/retro-runner/retro-pa-player.c
index 6f553c3..175975b 100644
--- a/retro-runner/retro-pa-player.c
+++ b/retro-runner/retro-pa-player.c
@@ -31,12 +31,7 @@ retro_pa_player_finalize (GObject *object)
   RetroPaPlayer *self = (RetroPaPlayer *)object;
 
   g_clear_object (&self->core);
-
-  if (self->simple != NULL) {
-    pa_simple_free (self->simple);
-    self->simple = NULL;
-  }
-
+  g_clear_pointer (&self->simple, pa_simple_free);
   g_clear_pointer (&self->src, src_delete);
   g_array_unref (self->buffer);
 
@@ -77,9 +72,7 @@ prepare_for_sample_rate (RetroPaPlayer *self,
   sample_spec.rate = (guint32) sample_rate;
   sample_spec.channels = 2;
 
-  if (self->simple != NULL)
-    pa_simple_free (self->simple);
-
+  g_clear_pointer (&self->simple, pa_simple_free);
   self->simple = pa_simple_new (NULL, NULL, PA_STREAM_PLAYBACK, NULL, "",
                                 &sample_spec, NULL, NULL, &error);
   if (!self->simple) {
@@ -231,11 +224,7 @@ retro_pa_player_set_core (RetroPaPlayer *self,
                                0);
   }
 
-  if (self->simple != NULL) {
-    pa_simple_free (self->simple);
-    self->simple = NULL;
-  }
-
+  g_clear_pointer (&self->simple, pa_simple_free);
   src_reset (self->src);
 }
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]