From c463a784c340874c75032da7602ec1fde43f75ed Mon Sep 17 00:00:00 2001 From: Henrik Grunell Date: Tue, 22 Dec 2020 14:47:07 +0100 Subject: [PATCH] Clarification of RtpPacket constructor in comment. See also b/175210069 for more context. Bug: None Change-Id: I06e9848028c0f11362db373af54b42cbc67aee77 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/198780 Reviewed-by: Danil Chapovalov Commit-Queue: Henrik Grunell Cr-Commit-Position: refs/heads/master@{#32874} --- modules/rtp_rtcp/source/rtp_packet.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/modules/rtp_rtcp/source/rtp_packet.h b/modules/rtp_rtcp/source/rtp_packet.h index b3e67be7cf..67516e97e7 100644 --- a/modules/rtp_rtcp/source/rtp_packet.h +++ b/modules/rtp_rtcp/source/rtp_packet.h @@ -30,6 +30,8 @@ class RtpPacket { // packet creating and used if available in Parse function. // Adding and getting extensions will fail until |extensions| is // provided via constructor or IdentifyExtensions function. + // |*extensions| is only accessed during construction; the pointer is not + // stored. RtpPacket(); explicit RtpPacket(const ExtensionManager* extensions); RtpPacket(const RtpPacket&);