[Asterisk-code-review] spelling: resync (asterisk[master])
Josh Soref
asteriskteam at digium.com
Sun Nov 7 00:16:19 CDT 2021
Josh Soref has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/17166 )
Change subject: spelling: resync
......................................................................
spelling: resync
Change-Id: Iaf8f32b154246b0a5c4630481471bd8421c77ff7
---
M include/asterisk/abstract_jb.h
M main/fixedjitterbuf.c
M tests/test_jitterbuf.c
3 files changed, 12 insertions(+), 12 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/66/17166/1
diff --git a/include/asterisk/abstract_jb.h b/include/asterisk/abstract_jb.h
index eab571b..7ceb9f5 100644
--- a/include/asterisk/abstract_jb.h
+++ b/include/asterisk/abstract_jb.h
@@ -107,7 +107,7 @@
typedef long (*jb_next_impl)(void *jb);
/*! \brief Remove first frame */
typedef int (*jb_remove_impl)(void *jb, struct ast_frame **fout);
-/*! \brief Force resynch */
+/*! \brief Force resync */
typedef void (*jb_force_resynch_impl)(void *jb);
/*! \brief Empty and reset jb */
typedef void (*jb_empty_and_reset_impl)(void *jb);
diff --git a/main/fixedjitterbuf.c b/main/fixedjitterbuf.c
index 6c92329..3601e26 100644
--- a/main/fixedjitterbuf.c
+++ b/main/fixedjitterbuf.c
@@ -156,11 +156,11 @@
offset */
offset = diff - jb->tail->ms;
- /* Do we really need to resynch, or this is just a frame for dropping? */
+ /* Do we really need to resync, or this is just a frame for dropping? */
if (!jb->force_resynch && (offset < jb->conf.resync_threshold && offset > -jb->conf.resync_threshold))
return FIXED_JB_DROP;
- /* Reset the force resynch flag */
+ /* Reset the force resync flag */
jb->force_resynch = 0;
/* apply the offset to the jb state */
@@ -211,7 +211,7 @@
/* check if the new frame is not too late */
if (delivery < jb->next_delivery) {
/* should drop the frame, but let first resynch_jb() check if this is not a jump in ts, or
- the force resynch flag was not set. */
+ the force resync flag was not set. */
return resynch_jb(jb, data, ms, ts, now);
}
@@ -219,7 +219,7 @@
However, allow more resync_threshold ms in advance */
if (delivery > jb->next_delivery + jb->delay + jb->conf.resync_threshold) {
/* should drop the frame, but let first resynch_jb() check if this is not a jump in ts, or
- the force resynch flag was not set. */
+ the force resync flag was not set. */
return resynch_jb(jb, data, ms, ts, now);
}
@@ -234,15 +234,15 @@
delivery < frame->delivery + frame->ms ||
(frame->next && delivery + ms > frame->next->delivery)))
{
- /* TODO: Should we check for resynch here? Be careful to do not allow threshold smaller than
+ /* TODO: Should we check for resync here? Be careful to do not allow threshold smaller than
the size of the jb */
/* should drop the frame, but let first resynch_jb() check if this is not a jump in ts, or
- the force resynch flag was not set. */
+ the force resync flag was not set. */
return resynch_jb(jb, data, ms, ts, now);
}
- /* Reset the force resynch flag */
+ /* Reset the force resync flag */
jb->force_resynch = 0;
/* Get a new frame */
diff --git a/tests/test_jitterbuf.c b/tests/test_jitterbuf.c
index 76fb3c4..dd7334d 100644
--- a/tests/test_jitterbuf.c
+++ b/tests/test_jitterbuf.c
@@ -1067,7 +1067,7 @@
/*!
* \internal
- * \brief Insert frames into the jitter buffer for the resynch tests
+ * \brief Insert frames into the jitter buffer for the resync tests
*/
static void test_jb_resynch_frame_insertion(struct jitterbuf *jb, enum jb_frame_type frame_type)
{
@@ -1096,7 +1096,7 @@
case TEST_INIT:
info->name = "jitterbuffer_resynch_control";
info->category = "/main/jitterbuf/";
- info->summary = "Tests sending control frames that force a resynch";
+ info->summary = "Tests sending control frames that force a resync";
info->description = "Control frames are sent to a jitter buffer. After some "
"number of frames, the source timestamps jump, forcing a resync of "
"the jitter buffer. Since the frames are control, the resync happens "
@@ -1171,7 +1171,7 @@
case TEST_INIT:
info->name = "jitterbuffer_resynch_voice";
info->category = "/main/jitterbuf/";
- info->summary = "Tests sending voice frames that force a resynch";
+ info->summary = "Tests sending voice frames that force a resync";
info->description = "Voice frames are sent to a jitter buffer. After some "
"number of frames, the source timestamps jump, forcing a resync of "
"the jitter buffer. Since the frames are voice, the resync happens "
@@ -1272,7 +1272,7 @@
AST_TEST_REGISTER(jitterbuffer_overflow_voice);
AST_TEST_REGISTER(jitterbuffer_overflow_control);
- /* Buffer resynch */
+ /* Buffer resync */
AST_TEST_REGISTER(jitterbuffer_resynch_voice);
AST_TEST_REGISTER(jitterbuffer_resynch_control);
--
To view, visit https://gerrit.asterisk.org/c/asterisk/+/17166
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Iaf8f32b154246b0a5c4630481471bd8421c77ff7
Gerrit-Change-Number: 17166
Gerrit-PatchSet: 1
Gerrit-Owner: Josh Soref <jsoref at gmail.com>
Gerrit-CC: Friendly Automation
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211107/7974566a/attachment-0001.html>
More information about the asterisk-code-review
mailing list