Added an operator[] to Buffer, to make reading data easier.

webrtc/base/buffer.h

 /*
  *  Copyright 2004 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.
  */
 
@@ skipping 106 matching lines @@
     return *this;
   }
 
   bool operator==(const Buffer& buf) const {
     assert(IsConsistent());
     return size_ == buf.size() && memcmp(data_.get(), buf.data(), size_) == 0;
   }
 
   bool operator!=(const Buffer& buf) const { return !(*this == buf); }
 
+  uint8_t& operator[](size_t index)       { return data()[index]; }
+  uint8_t  operator[](size_t index) const { return data()[index]; }

[kwiberg-webrtc, 2016/02/29 12:40:46]

It's probably a good idea to DCHECK that the index is good; aside from brevity,
that's what makes operator[] worth having.

[ossu, 2016/02/29 12:48:53]

Acknowledged.

+
   // The SetData functions replace the contents of the buffer. They accept the
   // same input types as the constructors.
   template <typename T, typename internal::ByteType<T>::t = 0>
   void SetData(const T* data, size_t size) {
     assert(IsConsistent());
     size_ = 0;
     AppendData(data, size);
   }
 
   template <typename T, size_t N, typename internal::ByteType<T>::t = 0>
@@ skipping 132 matching lines @@
   }
 
   size_t size_;
   size_t capacity_;
   std::unique_ptr<uint8_t[]> data_;
 };
 
 }  // namespace rtc
 
 #endif  // WEBRTC_BASE_BUFFER_H_