[Home]

Summary:ASTERISK-16523: [patch] Cleanup: consolidate offhook (new call)
Reporter:Damien Wedhorn (wedhorn)Labels:
Date Opened:2010-08-08 04:14:15Date Closed:2010-08-18 16:34:26
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_skinny
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) cleanup.stateoffhook.diff
Description:Consolidates all offhook (new call with dialtone) to setsubstate_offhook. This should be roughly equivalent to existing code, although a couple of calls now run through the full offhook sequence rather than an abbreviated one.

Tested on 7960 and 30VIP

****** ADDITIONAL INFORMATION ******

Please test the following, then dial a number:

Simple offhook
Newcall softkey
Call in, then transfer
Callforward softkey
Press line button
Comments:By: Damien Wedhorn (wedhorn) 2010-08-18 16:31:10

Tested by Salecha (see bug 17876)

By: Digium Subversion (svnbot) 2010-08-18 16:34:26

Repository: asterisk
Revision: 282701

U   trunk/channels/chan_skinny.c

------------------------------------------------------------------------
r282701 | wedhorn | 2010-08-18 16:34:25 -0500 (Wed, 18 Aug 2010) | 12 lines

Cleanup: consolidate offhook (new call).

Consolidates all offhook (new call with dialtone) to setsubstate_offhook. This should be roughly equivalent to existing code, although a couple of calls now run through the full offhook sequence rather than an abbreviated one.

(closes issue ASTERISK-16523)
Reported by: wedhorn
Patches:
     cleanup.stateoffhook.diff uploaded by wedhorn (license 30)
Tested by: salecha, wedhorn

Review: NA

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

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