<p> Attention is currently required from: N A. </p>
<p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/17716">View Change</a></p><p>6 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17716/comment/4da678cc_f7cbcd1a">Patch Set #3, Line 7:</a> <code style="font-family:monospace,monospace">func_frameintercept: Add FRAME_INTERCEPT function</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is not descriptive enough. What does intercept mean? What does it do? Why does this exist alongside func_framedrop?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File configs/samples/stasis.conf.sample:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17716/comment/d7ae97d0_52b1cd76">Patch Set #3, Line 57:</a> <code style="font-family:monospace,monospace">; decline=ast_channel_wink_type</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This should be in a separate change.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File doc/CHANGES-staging/func_frameintercept.txt:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17716/comment/6423f1e1_e6ff7a33">Patch Set #3, Line 3:</a> <code style="font-family:monospace,monospace">Adds the FRAME_INTERCEPT function, which can be used</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same as commit message, this needs to be more descriptive.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File funcs/func_frameintercept.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17716/comment/876a4282_89d00976">Patch Set #3, Line 32:</a> <code style="font-family:monospace,monospace">/*! \todo deprecate/remove func_holdintercept in 21, since this module is a superset of its functionality */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">The func_holdintercept is currently core supported, unless Sangoma takes this on as core supported then both would continue to exist.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File include/asterisk/stasis_channels.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17716/comment/80a5b9b0_5057aeda">Patch Set #3, Line 536:</a> <code style="font-family:monospace,monospace">struct stasis_message_type *ast_channel_wink_type(void);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This should be a separate change.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File main/manager_channels.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17716/comment/f2d05075_c0197728">Patch Set #3, Line 989:</a> <code style="font-family:monospace,monospace">static void channel_wink_cb(void *data, struct stasis_subscription *sub,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This should be a separate change.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17716">change 17716</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/17716"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I5ad83b95a4dce3b64b408a1497b0cd63051eb274 </div>
<div style="display:none"> Gerrit-Change-Number: 17716 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 03 Jan 2022 16:56:49 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>