commit 913cfa76ece2ebd91179fb149af857ee7df1614c
author Artem Titov <titovartem@webrtc.org> Wed Jul 28 23:57:33 2021 +0200
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Aug 09 15:51:03 2021 +0000
tree 7f432edabdf2757c63bf87534042c941e396c74b
parent 7f9ab1aa1b65dbaca64d613614f405e4972366b6

Use backticks not vertical bars to denote variables in comments for /modules/rtp_rtcp

Bug: webrtc:12338
Change-Id: I52eb3b6675c4705e22f51b70799ed6139a3b46bc
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/227164
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#34686}

modules/rtp_rtcp/source/rtp_rtcp_impl2.cc

diff --git a/modules/rtp_rtcp/source/rtp_rtcp_impl2.cc b/modules/rtp_rtcp/source/rtp_rtcp_impl2.cc
index f20fe87..136c11c 100644
--- a/modules/rtp_rtcp/source/rtp_rtcp_impl2.cc
+++ b/modules/rtp_rtcp/source/rtp_rtcp_impl2.cc
@@ -53,7 +53,7 @@
 
 int DelayMillisForDuration(TimeDelta duration) {
   // TimeDelta::ms() rounds downwards sometimes which leads to too little time
-  // slept. Account for this, unless |duration| is exactly representable in
+  // slept. Account for this, unless `duration` is exactly representable in
   // millisecs.
   return (duration.us() + rtc::kNumMillisecsPerSec - 1) /
          rtc::kNumMicrosecsPerMillisec;
@@ -528,9 +528,9 @@
              : -1;
 }
 
-// TODO(tommi): Check if |avg_rtt_ms|, |min_rtt_ms|, |max_rtt_ms| params are
+// TODO(tommi): Check if `avg_rtt_ms`, `min_rtt_ms`, `max_rtt_ms` params are
 // actually used in practice (some callers ask for it but don't use it). It
-// could be that only |rtt| is needed and if so, then the fast path could be to
+// could be that only `rtt` is needed and if so, then the fast path could be to
 // just call rtt_ms() and rely on the calculation being done periodically.
 int32_t ModuleRtpRtcpImpl2::RTT(const uint32_t remote_ssrc,
                                 int64_t* rtt,
@@ -550,7 +550,7 @@
   if (expected_retransmission_time_ms > 0) {
     return expected_retransmission_time_ms;
   }
-  // No rtt available (|kRttUpdateInterval| not yet passed?), so try to
+  // No rtt available (`kRttUpdateInterval` not yet passed?), so try to
   // poll avg_rtt_ms directly from rtcp receiver.
   if (rtcp_receiver_.RTT(rtcp_receiver_.RemoteSSRC(), nullptr,
                          &expected_retransmission_time_ms, nullptr,
@@ -686,7 +686,7 @@
     wait_time = kStartUpRttMs;
   }
 
-  // Send a full NACK list once within every |wait_time|.
+  // Send a full NACK list once within every `wait_time`.
   return now - nack_last_time_sent_full_ms_ > wait_time;
 }
 
@@ -865,7 +865,7 @@
     TimeDelta duration) {
   // We end up here under various sequences including the worker queue, and
   // the RTCPSender lock is held.
-  // See note in ScheduleRtcpSendEvaluation about why |worker_queue_| can be
+  // See note in ScheduleRtcpSendEvaluation about why `worker_queue_` can be
   // accessed.
   worker_queue_->PostDelayedTask(
       ToQueuedTask(task_safety_,