From d2f93e3afc7939c18de92b0fce7b3857a39697a0 Mon Sep 17 00:00:00 2001 From: Wim Taymans Date: Wed, 5 Mar 2014 14:28:26 +0100 Subject: [PATCH] session: small cleanups It's nicer to explicitly check for NULL on pointer types to make it clear that it's a pointer and not a boolean. --- gst/rtpmanager/rtpsession.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gst/rtpmanager/rtpsession.c b/gst/rtpmanager/rtpsession.c index d77abc3..6df9c02 100644 --- a/gst/rtpmanager/rtpsession.c +++ b/gst/rtpmanager/rtpsession.c @@ -1628,7 +1628,7 @@ rtp_session_get_source_by_ssrc (RTPSession * sess, guint32 ssrc) RTP_SESSION_LOCK (sess); result = find_source (sess, ssrc); - if (result) + if (result != NULL) g_object_ref (result); RTP_SESSION_UNLOCK (sess); @@ -2293,7 +2293,7 @@ rtp_session_process_pli (RTPSession * sess, guint32 sender_ssrc, return; src = find_source (sess, sender_ssrc); - if (!src) + if (src == NULL) return; rtp_session_request_local_key_unit (sess, src, FALSE, current_time); @@ -3811,7 +3811,7 @@ rtp_session_request_key_unit (RTPSession * sess, guint32 ssrc, RTP_SESSION_LOCK (sess); src = find_source (sess, ssrc); - if (!src) + if (src == NULL) goto no_source; if (fir) { -- 2.7.4