gst-plugins-ugly: x264enc: remove useless and semantically ( though not practically) wrong code

Vincent Penquerc'h vincent at kemper.freedesktop.org
Fri Jan 13 05:48:41 PST 2012


Module: gst-plugins-ugly
Branch: master
Commit: 69541031da9302b8fe6bfbb3e52a213400fa287b
URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-ugly/commit/?id=69541031da9302b8fe6bfbb3e52a213400fa287b

Author: Vincent Penquerc'h <vincent.penquerch at collabora.co.uk>
Date:   Fri Jan 13 13:46:50 2012 +0000

x264enc: remove useless and semantically (though not practically) wrong code

The object will be freed, so it's pointless to set the pointers
to NULL anyway.

---

 ext/x264/gstx264enc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/ext/x264/gstx264enc.c b/ext/x264/gstx264enc.c
index 2be3dd3..ea3cf0b 100644
--- a/ext/x264/gstx264enc.c
+++ b/ext/x264/gstx264enc.c
@@ -962,7 +962,7 @@ gst_x264_enc_finalize (GObject * object)
 
 #define FREE_STRING(ptr) \
   if (ptr) \
-    ptr = (GString *)g_string_free (ptr, TRUE);
+    g_string_free (ptr, TRUE);
 
   FREE_STRING (encoder->tunings);
   FREE_STRING (encoder->option_string);



More information about the gstreamer-commits mailing list