Updated structures and functions for setting the max bitrate limit to take rtc::Optional<int>

webrtc/base/optional.h

 /*
  *  Copyright 2015 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_BASE_OPTIONAL_H_
 #define WEBRTC_BASE_OPTIONAL_H_
 
 #include <algorithm>
 #include <utility>
-
+#include <ostream>
 #include "webrtc/base/checks.h"
 
 namespace rtc {
 
 // Simple std::experimental::optional-wannabe. It either contains a T or not.
 // In order to keep the implementation simple and portable, this implementation
 // actually contains a (default-constructed) T even when it supposedly doesn't
 // contain a value; use e.g. rtc::scoped_ptr<T> instead if that's too
 // expensive.
 //
@@ skipping 100 matching lines @@
                                           : m1.has_value_ != m2.has_value_;
   }
 
  private:
   // Invariant: Unless *this has been moved from, value_ is default-initialized
   // (or copied or moved from a default-initialized T) if !has_value_.
   T value_;
   bool has_value_;
 };
 
+// Defines the operator to write Optional<T> to an output stream. Allows for
+// simplified logging of optional parameters.
+template <typename T>
+std::ostream& operator<<(std::ostream& stream, rtc::Optional<T> value) {

[kwiberg-webrtc, 2016/03/29 11:30:45]

Second argument should be const ref to avoid copying.

[skvlad, 2016/03/30 19:40:44]

Done.

+  if (value) {
+    stream << *value;
+  } else {
+    stream << "<not set>";
+  }
+  return stream;
+}
+
 }  // namespace rtc
 
 #endif  // WEBRTC_BASE_OPTIONAL_H_