Let PacketSource::NextPacket() return an std::unique_ptr

webrtc/modules/audio_coding/neteq/tools/constant_pcm_packet_source.h

Index: webrtc/modules/audio_coding/neteq/tools/constant_pcm_packet_source.h
diff --git a/webrtc/modules/audio_coding/neteq/tools/constant_pcm_packet_source.h b/webrtc/modules/audio_coding/neteq/tools/constant_pcm_packet_source.h
index 69723035410c2b03d02bfd9d6f4b13395f708395..d1f88771b984e4ad0a83b255ce27f6e38687e65f 100644
--- a/webrtc/modules/audio_coding/neteq/tools/constant_pcm_packet_source.h
+++ b/webrtc/modules/audio_coding/neteq/tools/constant_pcm_packet_source.h
@@ -31,9 +31,7 @@ class ConstantPcmPacketSource : public PacketSource {
                           int sample_rate_hz,
                           int payload_type);
 
-  // Returns a pointer to the next packet. Will never return NULL. That is,
-  // the source is infinite.

[kwiberg-webrtc, 2016/05/24 09:04:56]

Is this comment no longer necessary? To me it looks useful...

[hlundin-webrtc, 2016/05/24 10:59:15]

I try to only document interface methods in the interface declaration. This is
an override, and having the same comment copied here will eventually lead to
discrepancies.

[kwiberg-webrtc, 2016/05/24 11:26:47]

Acknowledged.

-  Packet* NextPacket() override;
+  std::unique_ptr<Packet> NextPacket() override;
 
  private:
   void WriteHeader(uint8_t* packet_memory);