<div>how to link the data base</div>
<div><br><br> </div>
<div class="gmail_quote">On Wed, Sep 21, 2011 at 4:02 AM, <span dir="ltr"><<a href="mailto:asterisk-dev-request@lists.digium.com">asterisk-dev-request@lists.digium.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">Send asterisk-dev mailing list submissions to<br> <a href="mailto:asterisk-dev@lists.digium.com">asterisk-dev@lists.digium.com</a><br>
<br>To subscribe or unsubscribe via the World Wide Web, visit<br> <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br>or, via email, send a message with subject or body 'help' to<br>
<a href="mailto:asterisk-dev-request@lists.digium.com">asterisk-dev-request@lists.digium.com</a><br><br>You can reach the person managing the list at<br> <a href="mailto:asterisk-dev-owner@lists.digium.com">asterisk-dev-owner@lists.digium.com</a><br>
<br>When replying, please edit your Subject line so it is more specific<br>than "Re: Contents of asterisk-dev digest..."<br><br><br>Today's Topics:<br><br> 1. Re: [Code Review] Incorrect Voicemail duration reported in<br>
messages containing silence (take 2) (mjordan)<br> 2. Re: [Code Review] Incorrect Voicemail duration reported in<br> messages containing silence (take 2) (mjordan)<br> 3. Re: [Code Review] Incorrect Voicemail duration reported in<br>
messages containing silence (take 2) (rmudgett)<br><br><br>----------------------------------------------------------------------<br><br>Message: 1<br>Date: Tue, 20 Sep 2011 21:54:01 -0000<br>From: "mjordan" <<a href="mailto:reviewboard@asterisk.org">reviewboard@asterisk.org</a>><br>
Subject: Re: [asterisk-dev] [Code Review] Incorrect Voicemail duration<br> reported in messages containing silence (take 2)<br>To: <a href="mailto:rmudgett@digium.com">rmudgett@digium.com</a><br>Cc: , mjordan <<a href="mailto:reviewboard@asterisk.org">reviewboard@asterisk.org</a>>, Asterisk Developers<br>
<<a href="mailto:asterisk-dev@lists.digium.com">asterisk-dev@lists.digium.com</a>><br>Message-ID: <<a href="mailto:20110920215401.24422.50383@hotblack.digium.com">20110920215401.24422.50383@hotblack.digium.com</a>><br>
Content-Type: text/plain; charset="utf-8"<br><br><br>-----------------------------------------------------------<br>This is an automatically generated e-mail. To reply, visit:<br><a href="https://reviewboard.asterisk.org/r/1443/" target="_blank">https://reviewboard.asterisk.org/r/1443/</a><br>
-----------------------------------------------------------<br><br>(Updated Sept. 20, 2011, 4:54 p.m.)<br><br><br>Review request for Asterisk Developers and rmudgett.<br><br><br>Changes<br>-------<br><br>Removed assignment of sound_duration pointer to duration in play_record_review. Fixed spacing issue.<br>
<br>Reran through Asterisk test suite, no failures.<br><br><br>Summary<br>-------<br><br>This is a different take on a solution that was previously proposed here: <a href="https://reviewboard.asterisk.org/r/1403/" target="_blank">https://reviewboard.asterisk.org/r/1403/</a> - which I've now closed out.<br>
<br>The problem is still the same: The patch for ASTERISK-2234 is viewed as a regression in ASTERISK-16981. ASTERISK-2234 changed the way voicemail duration is calculated in the presence of silence. Instead of the duration being the length of the voicemail minus whatever silence accumulates at the end of a call (when that call ends in silence), the duration calculation was changed to include only the time in the audio stream that was not considered to be silence. Consider a voicemail system set up to ensure a message has a minimum duration of 4 seconds, and a silence cutoff of 6 seconds. This patch solved the following situation:<br>
1. A person begins recording with 4 seconds of silence<br>2. Person says "hello?", taking 1 second<br>3. Person waits for 6 seconds and is hung up on<br><br>Prior to this patch, the voicemail's duration would be calculated at 5 seconds and would be allowed through; post-patch the duration would be calculated at 1 second and would be discarded.<br>
<br>The previous patch simply rolled out ASTERISK-2234. After some discussion, this patch takes a different approach.<br><br>This patch keeps the previous approach of calculating the silence throughout the duration of the voicemail, but also records the duration of the actual file. The functions in app.c responsible for this mechanism report both durations back up to the caller. A caller of the method must supply a pointer for the duration; it does not have to supply a pointer for the sound-only duration. Those applications (app_voicemail, app_minivm) that make calculations as to whether or not they will keep a message can use either duration (with this patch, they will use the sound-only duration), but will report externally to users the duration of the entire message.<br>
<br>This fixes the bug with the e-mail notifications, while keeping the behavior intended by ASTERISK-2234.<br><br><br>This addresses bugs ASTERISK-16981 and ASTERISK-2234.<br> <a href="https://issues.asterisk.org/jira/browse/ASTERISK-16981" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-16981</a><br>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-2234" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-2234</a><br><br><br>Diffs (updated)<br>-----<br><br> /branches/1.8/apps/app_dial.c 337061<br>
/branches/1.8/apps/app_followme.c 337061<br> /branches/1.8/apps/app_meetme.c 337061<br> /branches/1.8/apps/app_minivm.c 337061<br> /branches/1.8/apps/app_voicemail.c 337061<br> /branches/1.8/include/asterisk/app.h 337061<br>
/branches/1.8/main/app.c 337061<br><br>Diff: <a href="https://reviewboard.asterisk.org/r/1443/diff" target="_blank">https://reviewboard.asterisk.org/r/1443/diff</a><br><br><br>Testing<br>-------<br><br>Tested using 1.8 and the following:<br>
<br>minsecs = 4<br>silencethreshold = 128<br>maxsilence = 10<br><br>* Leaving voicemails with silence in the beginning, punctuated by audio, followed by 10 seconds of silence (message kept with correct duration)<br>* Leaving voicmeails that consisted of all silence (message dropped)<br>
* Leaving voicemails of silence ended by '#' (message dropped)<br>* Leaving voicemail with silence, audio, silence, audio, silence, followed by '#' (message kept with correct duration)<br><br>Also tested with the Asterisk TestSuite's leave voicemail tests.<br>
<br><br>Thanks,<br><br>mjordan<br><br>-------------- next part --------------<br>An HTML attachment was scrubbed...<br>URL: <<a href="http://lists.digium.com/pipermail/asterisk-dev/attachments/20110920/e40814ec/attachment-0001.htm" target="_blank">http://lists.digium.com/pipermail/asterisk-dev/attachments/20110920/e40814ec/attachment-0001.htm</a>><br>
<br>------------------------------<br><br>Message: 2<br>Date: Tue, 20 Sep 2011 22:13:48 -0000<br>From: "mjordan" <<a href="mailto:reviewboard@asterisk.org">reviewboard@asterisk.org</a>><br>Subject: Re: [asterisk-dev] [Code Review] Incorrect Voicemail duration<br>
reported in messages containing silence (take 2)<br>To: <a href="mailto:rmudgett@digium.com">rmudgett@digium.com</a><br>Cc: , mjordan <<a href="mailto:reviewboard@asterisk.org">reviewboard@asterisk.org</a>>, Asterisk Developers<br>
<<a href="mailto:asterisk-dev@lists.digium.com">asterisk-dev@lists.digium.com</a>><br>Message-ID: <<a href="mailto:20110920221348.25928.57021@hotblack.digium.com">20110920221348.25928.57021@hotblack.digium.com</a>><br>
Content-Type: text/plain; charset="utf-8"<br><br><br>-----------------------------------------------------------<br>This is an automatically generated e-mail. To reply, visit:<br><a href="https://reviewboard.asterisk.org/r/1443/" target="_blank">https://reviewboard.asterisk.org/r/1443/</a><br>
-----------------------------------------------------------<br><br>(Updated Sept. 20, 2011, 5:13 p.m.)<br><br><br>Review request for Asterisk Developers and rmudgett.<br><br><br>Summary<br>-------<br><br>This is a different take on a solution that was previously proposed here: <a href="https://reviewboard.asterisk.org/r/1403/" target="_blank">https://reviewboard.asterisk.org/r/1403/</a> - which I've now closed out.<br>
<br>The problem is still the same: The patch for ASTERISK-2234 is viewed as a regression in ASTERISK-16981. ASTERISK-2234 changed the way voicemail duration is calculated in the presence of silence. Instead of the duration being the length of the voicemail minus whatever silence accumulates at the end of a call (when that call ends in silence), the duration calculation was changed to include only the time in the audio stream that was not considered to be silence. Consider a voicemail system set up to ensure a message has a minimum duration of 4 seconds, and a silence cutoff of 6 seconds. This patch solved the following situation:<br>
1. A person begins recording with 4 seconds of silence<br>2. Person says "hello?", taking 1 second<br>3. Person waits for 6 seconds and is hung up on<br><br>Prior to this patch, the voicemail's duration would be calculated at 5 seconds and would be allowed through; post-patch the duration would be calculated at 1 second and would be discarded.<br>
<br>The previous patch simply rolled out ASTERISK-2234. After some discussion, this patch takes a different approach.<br><br>This patch keeps the previous approach of calculating the silence throughout the duration of the voicemail, but also records the duration of the actual file. The functions in app.c responsible for this mechanism report both durations back up to the caller. A caller of the method must supply a pointer for the duration; it does not have to supply a pointer for the sound-only duration. Those applications (app_voicemail, app_minivm) that make calculations as to whether or not they will keep a message can use either duration (with this patch, they will use the sound-only duration), but will report externally to users the duration of the entire message.<br>
<br>This fixes the bug with the e-mail notifications, while keeping the behavior intended by ASTERISK-2234.<br><br><br>This addresses bugs ASTERISK-16981 and ASTERISK-2234.<br> <a href="https://issues.asterisk.org/jira/browse/ASTERISK-16981" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-16981</a><br>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-2234" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-2234</a><br><br><br>Diffs (updated)<br>-----<br><br> /branches/1.8/apps/app_dial.c 337061<br>
/branches/1.8/apps/app_followme.c 337061<br> /branches/1.8/apps/app_meetme.c 337061<br> /branches/1.8/apps/app_minivm.c 337061<br> /branches/1.8/apps/app_voicemail.c 337061<br> /branches/1.8/include/asterisk/app.h 337061<br>
/branches/1.8/main/app.c 337061<br><br>Diff: <a href="https://reviewboard.asterisk.org/r/1443/diff" target="_blank">https://reviewboard.asterisk.org/r/1443/diff</a><br><br><br>Testing<br>-------<br><br>Tested using 1.8 and the following:<br>
<br>minsecs = 4<br>silencethreshold = 128<br>maxsilence = 10<br><br>* Leaving voicemails with silence in the beginning, punctuated by audio, followed by 10 seconds of silence (message kept with correct duration)<br>* Leaving voicmeails that consisted of all silence (message dropped)<br>
* Leaving voicemails of silence ended by '#' (message dropped)<br>* Leaving voicemail with silence, audio, silence, audio, silence, followed by '#' (message kept with correct duration)<br><br>Also tested with the Asterisk TestSuite's leave voicemail tests.<br>
<br><br>Thanks,<br><br>mjordan<br><br>-------------- next part --------------<br>An HTML attachment was scrubbed...<br>URL: <<a href="http://lists.digium.com/pipermail/asterisk-dev/attachments/20110920/53dcbee9/attachment-0001.htm" target="_blank">http://lists.digium.com/pipermail/asterisk-dev/attachments/20110920/53dcbee9/attachment-0001.htm</a>><br>
<br>------------------------------<br><br>Message: 3<br>Date: Tue, 20 Sep 2011 22:32:06 -0000<br>From: "rmudgett" <<a href="mailto:reviewboard@asterisk.org">reviewboard@asterisk.org</a>><br>Subject: Re: [asterisk-dev] [Code Review] Incorrect Voicemail duration<br>
reported in messages containing silence (take 2)<br>To: <a href="mailto:rmudgett@digium.com">rmudgett@digium.com</a><br>Cc: , rmudgett <<a href="mailto:reviewboard@asterisk.org">reviewboard@asterisk.org</a>>, Asterisk Developers<br>
<<a href="mailto:asterisk-dev@lists.digium.com">asterisk-dev@lists.digium.com</a>><br>Message-ID: <<a href="mailto:20110920223206.27553.1854@hotblack.digium.com">20110920223206.27553.1854@hotblack.digium.com</a>><br>
Content-Type: text/plain; charset="utf-8"<br><br><br>-----------------------------------------------------------<br>This is an automatically generated e-mail. To reply, visit:<br><a href="https://reviewboard.asterisk.org/r/1443/#review4416" target="_blank">https://reviewboard.asterisk.org/r/1443/#review4416</a><br>
-----------------------------------------------------------<br><br>Ship it!<br><br><br>Looks fine to me. It would be nice if those long lines you touched were wrapped to 90 like the guidelines request.<br><br>- rmudgett<br>
<br><br>On Sept. 20, 2011, 5:13 p.m., mjordan wrote:<br>><br>> -----------------------------------------------------------<br>> This is an automatically generated e-mail. To reply, visit:<br>> <a href="https://reviewboard.asterisk.org/r/1443/" target="_blank">https://reviewboard.asterisk.org/r/1443/</a><br>
> -----------------------------------------------------------<br>><br>> (Updated Sept. 20, 2011, 5:13 p.m.)<br>><br>><br>> Review request for Asterisk Developers and rmudgett.<br>><br>><br>> Summary<br>
> -------<br>><br>> This is a different take on a solution that was previously proposed here: <a href="https://reviewboard.asterisk.org/r/1403/" target="_blank">https://reviewboard.asterisk.org/r/1403/</a> - which I've now closed out.<br>
><br>> The problem is still the same: The patch for ASTERISK-2234 is viewed as a regression in ASTERISK-16981. ASTERISK-2234 changed the way voicemail duration is calculated in the presence of silence. Instead of the duration being the length of the voicemail minus whatever silence accumulates at the end of a call (when that call ends in silence), the duration calculation was changed to include only the time in the audio stream that was not considered to be silence. Consider a voicemail system set up to ensure a message has a minimum duration of 4 seconds, and a silence cutoff of 6 seconds. This patch solved the following situation:<br>
> 1. A person begins recording with 4 seconds of silence<br>> 2. Person says "hello?", taking 1 second<br>> 3. Person waits for 6 seconds and is hung up on<br>><br>> Prior to this patch, the voicemail's duration would be calculated at 5 seconds and would be allowed through; post-patch the duration would be calculated at 1 second and would be discarded.<br>
><br>> The previous patch simply rolled out ASTERISK-2234. After some discussion, this patch takes a different approach.<br>><br>> This patch keeps the previous approach of calculating the silence throughout the duration of the voicemail, but also records the duration of the actual file. The functions in app.c responsible for this mechanism report both durations back up to the caller. A caller of the method must supply a pointer for the duration; it does not have to supply a pointer for the sound-only duration. Those applications (app_voicemail, app_minivm) that make calculations as to whether or not they will keep a message can use either duration (with this patch, they will use the sound-only duration), but will report externally to users the duration of the entire message.<br>
><br>> This fixes the bug with the e-mail notifications, while keeping the behavior intended by ASTERISK-2234.<br>><br>><br>> This addresses bugs ASTERISK-16981 and ASTERISK-2234.<br>> <a href="https://issues.asterisk.org/jira/browse/ASTERISK-16981" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-16981</a><br>
> <a href="https://issues.asterisk.org/jira/browse/ASTERISK-2234" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-2234</a><br>><br>><br>> Diffs<br>> -----<br>><br>> /branches/1.8/apps/app_dial.c 337061<br>
> /branches/1.8/apps/app_followme.c 337061<br>> /branches/1.8/apps/app_meetme.c 337061<br>> /branches/1.8/apps/app_minivm.c 337061<br>> /branches/1.8/apps/app_voicemail.c 337061<br>> /branches/1.8/include/asterisk/app.h 337061<br>
> /branches/1.8/main/app.c 337061<br>><br>> Diff: <a href="https://reviewboard.asterisk.org/r/1443/diff" target="_blank">https://reviewboard.asterisk.org/r/1443/diff</a><br>><br>><br>> Testing<br>> -------<br>
><br>> Tested using 1.8 and the following:<br>><br>> minsecs = 4<br>> silencethreshold = 128<br>> maxsilence = 10<br>><br>> * Leaving voicemails with silence in the beginning, punctuated by audio, followed by 10 seconds of silence (message kept with correct duration)<br>
> * Leaving voicmeails that consisted of all silence (message dropped)<br>> * Leaving voicemails of silence ended by '#' (message dropped)<br>> * Leaving voicemail with silence, audio, silence, audio, silence, followed by '#' (message kept with correct duration)<br>
><br>> Also tested with the Asterisk TestSuite's leave voicemail tests.<br>><br>><br>> Thanks,<br>><br>> mjordan<br>><br>><br><br>-------------- next part --------------<br>An HTML attachment was scrubbed...<br>
URL: <<a href="http://lists.digium.com/pipermail/asterisk-dev/attachments/20110920/076c27e4/attachment.htm" target="_blank">http://lists.digium.com/pipermail/asterisk-dev/attachments/20110920/076c27e4/attachment.htm</a>><br>
<br>------------------------------<br><br>_______________________________________________<br>--Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--/" target="_blank">http://www.api-digital.com--</a><br>
<br>AstriCon 2010 - October 26-28 Washington, DC<br>Put in your talk proposal: <a href="http://www.bit.ly/speak-astricon2010" target="_blank">http://www.bit.ly/speak-astricon2010</a><br><br>asterisk-dev mailing list<br>To UNSUBSCRIBE or update options visit:<br>
<a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br><br>End of asterisk-dev Digest, Vol 86, Issue 126<br>*********************************************<br>
</blockquote></div><br><br clear="all"><br>-- <br>
<div><span style="LINE-HEIGHT: normal; TEXT-TRANSFORM: none; FONT-VARIANT: normal; FONT-STYLE: normal; TEXT-INDENT: 0px; BORDER-COLLAPSE: separate; FONT-FAMILY: 'Times New Roman'; WHITE-SPACE: normal; LETTER-SPACING: normal; COLOR: rgb(0,0,0); FONT-WEIGHT: normal; WORD-SPACING: 0px"><span style="FONT-FAMILY: arial">
<div><font color="#666666"><font face="tahoma, sans-serif"><span></span><span></span></font></font><span></span><span></span><span></span><span></span> </div>
<div><font color="#666666"><font face="tahoma, sans-serif">B. Idroo<span> </span></font></font><b><font color="#666666"><font face="tahoma, sans-serif">Careem</font></font></b></div>
<div><font color="#666666"><font face="tahoma, sans-serif">Mobile: 94 777 56 56 33</font></font></div>
<div>
<div>
<div><font color="#666666"><font face="tahoma, sans-serif">Email:<span> </span></font></font><a href="mailto:careem@aandrholdings.com" target="_blank"><span style="TEXT-DECORATION: none"><font color="#666666"><font face="tahoma, sans-serif">careem@aandrholdings.com</font></font></span></a><font color="#666666"><font face="tahoma, sans-serif"><span> </span></font></font><font color="#666666"><font face="tahoma, sans-serif"><br>
</font></font><font color="#666666"><font face="tahoma, sans-serif">Web: </font></font><a href="http://www.aandrholdings.com/" target="_blank"><font color="#666666"><font face="tahoma, sans-serif">www.aandrholdings.com</font></font></a></div>
</div></div></span></span></div><br>