Fixed lint warnings in webrtc/modules/remote_bitrate_estimator

webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
-#ifndef WEBRTC_MODULES_RTP_RTCP_SOURCE_BWE_DEFINES_H_
-#define WEBRTC_MODULES_RTP_RTCP_SOURCE_BWE_DEFINES_H_
+#ifndef WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_INCLUDE_BWE_DEFINES_H_
+#define WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_INCLUDE_BWE_DEFINES_H_
 
 #include "webrtc/typedefs.h"
 
-#define BWE_MAX(a,b) ((a)>(b)?(a):(b))
-#define BWE_MIN(a,b) ((a)<(b)?(a):(b))
+#define BWE_MAX(a, b) ((a) > (b) ? (a) : (b))
+#define BWE_MIN(a, b) ((a) < (b) ? (a) : (b))
 
 namespace webrtc {
 
 static const int64_t kBitrateWindowMs = 1000;
 
-enum BandwidthUsage
-{
-    kBwNormal = 0,
-    kBwUnderusing = 1,
-    kBwOverusing = 2,
+enum BandwidthUsage {
+  kBwNormal = 0,
+  kBwUnderusing = 1,
+  kBwOverusing = 2,
 };
 
-enum RateControlState
-{
-    kRcHold,
-    kRcIncrease,
-    kRcDecrease
-};
+enum RateControlState { kRcHold, kRcIncrease, kRcDecrease };
 
-enum RateControlRegion
-{
-    kRcNearMax,
-    kRcAboveMax,
-    kRcMaxUnknown
-};
+enum RateControlRegion { kRcNearMax, kRcAboveMax, kRcMaxUnknown };
 
-class RateControlInput
-{
-public:
-    RateControlInput(BandwidthUsage bwState,
-                     uint32_t incomingBitRate,
-                     double noiseVar)
-        : _bwState(bwState),
-          _incomingBitRate(incomingBitRate),
-          _noiseVar(noiseVar) {}
+class RateControlInput {
+ public:
+  RateControlInput(BandwidthUsage bw_state,
+                   uint32_t incoming_bitrate,
+                   double noise_var)
+      : bw_state_(bw_state),
+        incoming_bitrate_(incoming_bitrate),
+        noise_var_(noise_var) {}
 
-    BandwidthUsage  _bwState;
-    uint32_t      _incomingBitRate;
-    double              _noiseVar;
+  BandwidthUsage bw_state_;
+  uint32_t incoming_bitrate_;
+  double noise_var_;

[stefan-webrtc, 2015/12/11 13:07:55]

I'd actually prefer to switch to noise_var etc and skip the trailing _ since
these are public. Also switch to a struct instead of a class.

Sorry for not realizing this before...

 };
 }  // namespace webrtc
 
-#endif // WEBRTC_MODULES_RTP_RTCP_SOURCE_BWE_DEFINES_H_
+#endif  // WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_INCLUDE_BWE_DEFINES_H_