[Home]

Summary:ASTERISK-03249: Bug in dsp.c with BUSYDETECT_TONEONLY
Reporter:Julian J. M. (julianjm)Labels:
Date Opened:2005-01-10 22:03:50.000-0600Date Closed:2008-01-15 15:21:30.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) dsp.c.diff
Description:BUSYDETECT_TONEONLY doesn't work with variable silence length, because there's a bug (typo) in dsp.c... It's so trivial I think i don't need the disclaimer...
Comments:By: Andrey S Pankov (casper) 2005-01-11 07:01:51.000-0600

Even oneliners should be disclaimed.

And please add .txt extention to files you upload next time.
By: Mark Spencer (markster) 2005-01-11 12:07:16.000-0600

Fixed in CVS head.  I just made the change myself to be on the safe side since it's so trivial.  It took me a while to understand this but I think the patch is right and that it should fix busy detect in general not just in this case with tone only.
By: Russell Bryant (russell) 2005-01-12 23:02:59.000-0600

fixed in 1.0
By: Digium Subversion (svnbot) 2008-01-15 15:21:13.000-0600

Repository: asterisk
Revision: 4759

U   trunk/dsp.c

------------------------------------------------------------------------
r4759 | markster | 2008-01-15 15:21:12 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix typo in tone detect (bug ASTERISK-3249)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=4759
By: Digium Subversion (svnbot) 2008-01-15 15:21:30.000-0600

Repository: asterisk
Revision: 4779

U   branches/v1-0/dsp.c

------------------------------------------------------------------------
r4779 | russell | 2008-01-15 15:21:30 -0600 (Tue, 15 Jan 2008) | 2 lines

fix typo in tone detect (bug ASTERISK-3249)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=4779