<p> Attention is currently required from: N A. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/16481">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</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/+/16481?tab=comments">Patch Set #4:</a> </p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I need to trunk to some systems which are SF-controlled only and do not accept MF. The other side of this is that the SF code that exists today in DAHDI is broken and basically useless. The SF requirements are the same as MF so this works perfectly and adds a tech agnostic way to do that.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I did some tested and it works perfectly, e.g. SendSF -> ReceiveSF.</p><p style="white-space: pre-wrap; word-wrap: break-word;">There is also interest/need for this from other members of the community. The SF builds on the MF quite naturally so that's why it's there.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">There's a big difference between MF and SF.  I can see MF as just a different tone matrix than DTMF.  For SF you have to accumulate and count 2600Hz pulses so while it may be convenient for you to lump them together, they're not architecturally related other than using the DSP.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;"><br>What should I do with this review, exactly, delete/abandon it and then do what you'd mentioned?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">1.  Abandon this review.<br>2.  In your local branch that has your other review, make your changes to support adding MF to Dial, leaving out all SF stuff, and save it as a new commit.  Don't amend the original commit.  Then do a 'git review' to upload.  This should complete your MF work.<br>3.  In that same branch, create a third commit with your SF changes AS A NEW APP (app_sf).  Don't lump it in with MF.  Do a 'git review' again.  I'm not sold on the idea of SF support yet but at least this will make it easier to deal with.</p><p style="white-space: pre-wrap; word-wrap: break-word;">At the end, the SF review will have the second MF review as its parent, and the second MF review will have the first one as its parent.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/16481">change 16481</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/+/16481"/><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: I05ba61ce49092d3f131ade9dc4d97dcc9887fa1d </div>
<div style="display:none"> Gerrit-Change-Number: 16481 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </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: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 13 Sep 2021 16:07:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Comment-In-Reply-To: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>