[asterisk-bugs] [Asterisk 0015932]: [patch] Make internal_timing on by default

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15932 
====================================================================== 
Reported By:                dimas
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   15932
Category:                   Core/General
Reproducibility:            have not tried
Severity:                   tweak
Priority:                   normal
Status:                     closed
Target Version:             1.6.2.9
Asterisk Version:           SVN 
JIRA:                       SWP-725 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-09-22 09:45 CDT
Last Modified:              2010-05-19 12:49 CDT
====================================================================== 
Summary:                    [patch] Make internal_timing on by default
Description: 
Are there any reason to not have internal_timing on by default?

As I already wrote in https://issues.asterisk.org/view.php?id=15296, 
"I never understood why internal_timing is not YES by default. Everybody I
know turn it on as the first thing after Asterisk installation. And I see
lots of "i have bad audio" topics on forums where people eventually get
advice "turn internal timing on". From the other hand, I have not seen any
problem which is solved by turning that timing off.. "

Since Leif suggested me to fill an issue about the subject so here we
are...
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0005374 [patch] Asynchronous generation of outg...
parent of           0016285 Review of internal_timing code
====================================================================== 

---------------------------------------------------------------------- 
 (0122177) svnbot (reporter) - 2010-05-19 12:49
 https://issues.asterisk.org/view.php?id=15932#c122177 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 264250

_U  branches/1.6.2/
U   branches/1.6.2/configure
U   branches/1.6.2/configure.ac
U   branches/1.6.2/include/asterisk/autoconfig.h.in
U   branches/1.6.2/include/asterisk/options.h

------------------------------------------------------------------------
r264250 | tilghman | 2010-05-19 12:49:44 -0500 (Wed, 19 May 2010) | 31
lines

Merged revisions 264249 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
  r264249 | tilghman | 2010-05-19 12:48:31 -0500 (Wed, 19 May 2010) | 24
lines
  
  Merged revisions 264248 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r264248 | tilghman | 2010-05-19 12:41:29 -0500 (Wed, 19 May 2010) | 17
lines
    
    Internal timing is now on by default, if you're using DAHDI 2.3 or
above.
    
    The reason for ensuring DAHDI 2.3 or above is that this version
ensures that
    a timer is always available, whereas in previous versions, it was
possible
    for DAHDI to be loaded, but have no drivers to actually generate
timing.  If
    internal_timing was turned on in this circumstance, a complete lack of
audio
    would result.  This is the reason why internal_timing was not on by
default.
    However, now that DAHDI ensures the availability of a timer, there is
no
    reason for this setting to be off (and in fact, it solves a great many
initial
    user problems).
    
    (closes issue https://issues.asterisk.org/view.php?id=15932)
     Reported by: dimas
     Patches: 
           20100519__issue15932.diff.txt uploaded by tilghman (license 14)
     Tested by: tilghman
  ........
................

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-05-19 12:49 svnbot         Checkin                                      
2010-05-19 12:49 svnbot         Note Added: 0122177                          
======================================================================