Cast: Refactoring RtpReceiver to Clang format

media/cast/rtp_receiver/receiver_stats.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/cast/rtp_receiver/receiver_stats.h"
 
 #include "base/logging.h"
 #include "media/cast/rtp_receiver/rtp_receiver_defines.h"
 
 namespace media {
@@ skipping 19 matching lines @@
                                   uint32* jitter) {
   // Compute losses.
   if (interval_number_packets_ == 0) {
     *fraction_lost = 0;
   } else {
     int diff = 0;
     if (interval_wrap_count_ == 0) {
       diff = max_sequence_number_ - interval_min_sequence_number_ + 1;
     } else {
       diff = kMaxSequenceNumber * (interval_wrap_count_ - 1) +
-        (max_sequence_number_ - interval_min_sequence_number_ +
-            kMaxSequenceNumber + 1);
+             (max_sequence_number_ - interval_min_sequence_number_ +
+              kMaxSequenceNumber + 1);
     }
 
     if (diff < 1) {
       *fraction_lost = 0;
     } else {
-      *fraction_lost =  static_cast<uint8>((256 * (1 -
-          static_cast<float>(interval_number_packets_) / abs(diff))));
+      *fraction_lost = static_cast<uint8>(
+          (256 *

[hubbe, 2014/02/03 21:49:36]

This is pretty ugly.
Perhaps use fabs() and void the static cast?
Or use a temporary variable?

[mikhal1, 2014/02/04 00:16:27]

Done.

+           (1 - static_cast<float>(interval_number_packets_) / abs(diff))));
     }
   }
 
   int expected_packets_num = max_sequence_number_ - min_sequence_number_ + 1;
   if (total_number_packets_ == 0) {
     *cumulative_lost = 0;
   } else if (sequence_number_cycles_ == 0) {
     *cumulative_lost = expected_packets_num - total_number_packets_;
   } else {
-    *cumulative_lost = kMaxSequenceNumber * (sequence_number_cycles_ - 1) +
+    *cumulative_lost =
+        kMaxSequenceNumber * (sequence_number_cycles_ - 1) +
         (expected_packets_num - total_number_packets_ + kMaxSequenceNumber);
   }
 
   // Extended high sequence number consists of the highest seq number and the
   // number of cycles (wrap).
-  *extended_high_sequence_number = (sequence_number_cycles_ << 16) +
-      max_sequence_number_;
+  *extended_high_sequence_number =
+      (sequence_number_cycles_ << 16) + max_sequence_number_;
 
   *jitter = static_cast<uint32>(abs(jitter_.InMillisecondsRoundedUp()));
 
   // Reset interval values.
   interval_min_sequence_number_ = 0;
   interval_number_packets_ = 0;
   interval_wrap_count_ = 0;
 }
 
 void ReceiverStats::UpdateStatistics(const RtpCastHeader& header) {
@@ skipping 17 matching lines @@
     }
     max_sequence_number_ = new_seq_num;
   }
 
   // Compute Jitter.
   base::TimeTicks now = clock_->NowTicks();
   base::TimeDelta delta_new_timestamp =
       base::TimeDelta::FromMilliseconds(header.webrtc.header.timestamp);
   if (total_number_packets_ > 0) {
     // Update jitter.
-    base::TimeDelta delta = (now - last_received_packet_time_) -
+    base::TimeDelta delta =
+        (now - last_received_packet_time_) -
         ((delta_new_timestamp - last_received_timestamp_) / 90);
     jitter_ += (delta - jitter_) / 16;
   }
   last_received_timestamp_ = delta_new_timestamp;
   last_received_packet_time_ = now;
 
   // Increment counters.
   ++total_number_packets_;
   ++interval_number_packets_;
 }
 
 }  // namespace cast
 }  // namespace media