From: Wim Taymans <wim.taymans@collabora.co.uk>
Date: Mon, 13 Sep 2010 13:44:52 +0000 (+0200)
Subject: rtpsession: Small cleanups
X-Git-Tag: RELEASE-0.10.26~135
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d541f5e24db9c88abeb29be9c134636934c0f131;p=platform%2Fupstream%2Fgst-plugins-good.git

rtpsession: Small cleanups

Make the property description prettier.
Actually multiple the bandwidth with the fraction.
---

diff --git a/gst/rtpmanager/gstrtpsession.c b/gst/rtpmanager/gstrtpsession.c
index 63123cdcb..a8a11df8d 100644
--- a/gst/rtpmanager/gstrtpsession.c
+++ b/gst/rtpmanager/gstrtpsession.c
@@ -542,7 +542,8 @@ gst_rtp_session_class_init (GstRtpSessionClass * klass)
 
   g_object_class_install_property (gobject_class, PROP_RTCP_FRACTION,
       g_param_spec_double ("rtcp-fraction", "RTCP Fraction",
-          "The RTCP bandwidth of the session in bytes per second (or as a real fraction of the RTP bandwidth if < 1)",
+          "The RTCP bandwidth of the session in bytes per second "
+          "(or as a real fraction of the RTP bandwidth if < 1.0)",
           0.0, G_MAXDOUBLE, DEFAULT_RTCP_FRACTION, G_PARAM_READWRITE));
 
   g_object_class_install_property (gobject_class, PROP_RTCP_RR_BANDWIDTH,
diff --git a/gst/rtpmanager/rtpstats.c b/gst/rtpmanager/rtpstats.c
index 8d1f5cfa1..a7066b5a4 100644
--- a/gst/rtpmanager/rtpstats.c
+++ b/gst/rtpmanager/rtpstats.c
@@ -58,22 +58,22 @@ rtp_stats_set_bandwidths (RTPSessionStats * stats, guint rtp_bw,
     rtcp_bw = rs + rr;
 
   /* If rtcp_bw is between 0 and 1, it is a fraction of rtp_bw */
-  if (rtcp_bw > 0 && rtcp_bw < 1) {
-    if (rtp_bw > 0)
+  if (rtcp_bw > 0.0 && rtcp_bw < 1.0) {
+    if (rtp_bw > 0.0)
       rtcp_bw = rtp_bw * rtcp_bw;
     else
-      rtcp_bw = -1;
+      rtcp_bw = -1.0;
   }
 
   /* RTCP is 5% of the RTP bandwidth */
-  if (rtp_bw == -1 && rtcp_bw > 0)
+  if (rtp_bw == -1 && rtcp_bw > 1.0)
     rtp_bw = rtcp_bw * 20;
-  else if (rtp_bw != -1 && rtcp_bw < 0)
+  else if (rtp_bw != -1 && rtcp_bw < 0.0)
     rtcp_bw = rtp_bw / 20;
-  else if (rtp_bw == -1 && rtcp_bw < 0) {
+  else if (rtp_bw == -1 && rtcp_bw < 0.0) {
     /* nothing given, take defaults */
     rtp_bw = RTP_STATS_BANDWIDTH;
-    rtcp_bw = rtp_bw = RTP_STATS_RTCP_FRACTION;
+    rtcp_bw = rtp_bw * RTP_STATS_RTCP_FRACTION;
   }
 
   stats->bandwidth = rtp_bw;