[asterisk-bugs] [Asterisk 0012674]: [branch] revert ast_queue_hangup and create ast_queue_hangup_with_cause

[noreply at bugs.digium.com]

The following issue is now READY FOR TESTING. 
====================================================================== 
http://bugs.digium.com/view.php?id=12674 
====================================================================== 
Reported By:                mvanbaak
Assigned To:                mvanbaak
====================================================================== 
Project:                    Asterisk
Issue ID:                   12674
Category:                   General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 117135 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             05-17-2008 08:05 CDT
Last Modified:              05-19-2008 17:48 CDT
====================================================================== 
Summary:                    [branch] revert ast_queue_hangup and create
ast_queue_hangup_with_cause
Description: 
Recently, ast_queue_hangup got an extra parameter to set the hangupcause.
Some people told me that was not ok and there should be an
ast_queue_hangup_with_cause.
The second thing in this commit that was not ok, was the use of
ast_frame.seqno for the hangupcause.
I addressed that in this patch as well.
====================================================================== 

---------------------------------------------------------------------- 
 mvanbaak - 05-19-08 17:48  
---------------------------------------------------------------------- 
svn asterisk/team/mvanbaak/ast_frame_dataunion has the new code.
Please give me feedback. I think it's ready for trunk.

thanks to kevin and russell for all the feedback and hints on this one. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
05-19-08 17:48  mvanbaak       SVN Revision (number only!)116882 => 117135    
05-19-08 17:48  mvanbaak       Note Added: 0087047                          
05-19-08 17:48  mvanbaak       Status                   assigned => ready for
testing
======================================================================