This change moves the default feature digit timeout to 1000 ms from the previous...

As per bug 14515, a dev discussion arrived at a "mediated concensus"
of a default feature digit timeout of 1.0 sec. Some voted for 1300;
ctooley thought 1500 for distracted phone users in phone booths;
kpfleming put his foot down at 1.0 sec.

Users who found the previous default max delay of 250 msec perfect,
are welcome to override the new default. Notice that I said that
250 msec was the default; wait a minute, you might say, the config
file said it was 500 msec!; well, because of the bug fix for 14515,
we found that 500 msec was actually enforcing a max of 250. The bug
fix would restore 500 msec, but we felt even that was a bit tight
for most users... 2000 msec was pushed earlier by mmichelson, so
that reduces to 1000 msec after the bug fix. Enjoy!

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@178956 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/configs/features.conf.sample b/configs/features.conf.sample
index c5949a97f80dd8369f4b3127530e2debd537a1f4..2dda81f0425e7de871de5186faef34ddc92fd24d 100644 (file)
--- a/configs/features.conf.sample
+++ b/configs/features.conf.sample
@@ -35,9 +35,9 @@ context => parkedcalls                ; Which context parked calls are in
 ;xfersound = beep              ; to indicate an attended transfer is complete
 ;xferfailsound = beeperr       ; to indicate a failed transfer
 ;pickupexten = *8              ; Configure the pickup extension. (default is *8)
-;featuredigittimeout = 500     ; Max time (ms) between digits for 
-                               ; feature activation  (default is 500 ms)
-;atxfernoanswertimeout = 15    ; Timeout for answer on attended transfer default is 15 seconds.
+;featuredigittimeout = 1000 ; Max time (ms) between digits for 
+                            ; feature activation  (default is 1000 ms)
+;atxfernoanswertimeout = 15 ; Timeout for answer on attended transfer default is 15 seconds.
 
 ; Note that the DTMF features listed below only work when two channels have answered and are bridged together.
 ; They can not be used while the remote party is ringing or in progress. If you require this feature you can use

diff --git a/res/res_features.c b/res/res_features.c
index 98983270ac772bcf9045b7739e0d111d36fc8a4d..819481e8bcf71fe66967431ed1edbeaaa4536e92 100644 (file)
--- a/res/res_features.c
+++ b/res/res_features.c
@@ -66,7 +66,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 
 #define DEFAULT_PARK_TIME 45000
 #define DEFAULT_TRANSFER_DIGIT_TIMEOUT 3000
-#define DEFAULT_FEATURE_DIGIT_TIMEOUT 500
+#define DEFAULT_FEATURE_DIGIT_TIMEOUT 1000
 #define DEFAULT_NOANSWER_TIMEOUT_ATTENDED_TRANSFER 15000
 
 #define AST_MAX_WATCHERS 256