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

noreply at bugs.digium.com noreply at bugs.digium.com
Thu May 22 12:07:58 CDT 2008


A NOTE has been added to this issue. 
====================================================================== 
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:                     closed
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:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             05-17-2008 08:05 CDT
Last Modified:              05-22-2008 12:07 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.
====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 05-22-08 12:07  
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 117831

_U  team/ctooley/excel-sip-changes/
U   team/ctooley/excel-sip-changes/apps/app_alarmreceiver.c
U   team/ctooley/excel-sip-changes/apps/app_chanspy.c
U   team/ctooley/excel-sip-changes/apps/app_dial.c
U   team/ctooley/excel-sip-changes/apps/app_disa.c
U   team/ctooley/excel-sip-changes/apps/app_externalivr.c
U   team/ctooley/excel-sip-changes/apps/app_festival.c
U   team/ctooley/excel-sip-changes/apps/app_followme.c
U   team/ctooley/excel-sip-changes/apps/app_ices.c
U   team/ctooley/excel-sip-changes/apps/app_meetme.c
U   team/ctooley/excel-sip-changes/apps/app_milliwatt.c
U   team/ctooley/excel-sip-changes/apps/app_mp3.c
U   team/ctooley/excel-sip-changes/apps/app_nbscat.c
U   team/ctooley/excel-sip-changes/apps/app_queue.c
U   team/ctooley/excel-sip-changes/apps/app_sms.c
U   team/ctooley/excel-sip-changes/apps/app_speech_utils.c
U   team/ctooley/excel-sip-changes/apps/app_test.c
U   team/ctooley/excel-sip-changes/apps/app_zapbarge.c
U   team/ctooley/excel-sip-changes/apps/app_zapscan.c
U   team/ctooley/excel-sip-changes/channels/chan_alsa.c
U   team/ctooley/excel-sip-changes/channels/chan_console.c
U   team/ctooley/excel-sip-changes/channels/chan_gtalk.c
U   team/ctooley/excel-sip-changes/channels/chan_h323.c
U   team/ctooley/excel-sip-changes/channels/chan_iax2.c
U   team/ctooley/excel-sip-changes/channels/chan_jingle.c
U   team/ctooley/excel-sip-changes/channels/chan_local.c
U   team/ctooley/excel-sip-changes/channels/chan_mgcp.c
U   team/ctooley/excel-sip-changes/channels/chan_misdn.c
U   team/ctooley/excel-sip-changes/channels/chan_oss.c
U   team/ctooley/excel-sip-changes/channels/chan_phone.c
U   team/ctooley/excel-sip-changes/channels/chan_sip.c
U   team/ctooley/excel-sip-changes/channels/chan_skinny.c
U   team/ctooley/excel-sip-changes/channels/chan_unistim.c
U   team/ctooley/excel-sip-changes/channels/chan_zap.c
U   team/ctooley/excel-sip-changes/channels/iax2-parser.c
U   team/ctooley/excel-sip-changes/codecs/codec_a_mu.c
U   team/ctooley/excel-sip-changes/codecs/codec_adpcm.c
U   team/ctooley/excel-sip-changes/codecs/codec_alaw.c
U   team/ctooley/excel-sip-changes/codecs/codec_g722.c
U   team/ctooley/excel-sip-changes/codecs/codec_g726.c
U   team/ctooley/excel-sip-changes/codecs/codec_gsm.c
U   team/ctooley/excel-sip-changes/codecs/codec_lpc10.c
U   team/ctooley/excel-sip-changes/codecs/codec_resample.c
U   team/ctooley/excel-sip-changes/codecs/codec_ulaw.c
U   team/ctooley/excel-sip-changes/codecs/codec_zap.c
U   team/ctooley/excel-sip-changes/configs/jabber.conf.sample
U   team/ctooley/excel-sip-changes/formats/format_g723.c
U   team/ctooley/excel-sip-changes/formats/format_g726.c
U   team/ctooley/excel-sip-changes/formats/format_g729.c
U   team/ctooley/excel-sip-changes/formats/format_gsm.c
U   team/ctooley/excel-sip-changes/formats/format_h263.c
U   team/ctooley/excel-sip-changes/formats/format_h264.c
U   team/ctooley/excel-sip-changes/formats/format_ilbc.c
U   team/ctooley/excel-sip-changes/formats/format_jpeg.c
U   team/ctooley/excel-sip-changes/formats/format_pcm.c
U   team/ctooley/excel-sip-changes/formats/format_sln.c
U   team/ctooley/excel-sip-changes/formats/format_sln16.c
U   team/ctooley/excel-sip-changes/formats/format_vox.c
U   team/ctooley/excel-sip-changes/formats/format_wav.c
U   team/ctooley/excel-sip-changes/formats/format_wav_gsm.c
U   team/ctooley/excel-sip-changes/include/asterisk/channel.h
U   team/ctooley/excel-sip-changes/include/asterisk/frame.h
U   team/ctooley/excel-sip-changes/main/abstract_jb.c
U   team/ctooley/excel-sip-changes/main/app.c
U   team/ctooley/excel-sip-changes/main/audiohook.c
U   team/ctooley/excel-sip-changes/main/channel.c
U   team/ctooley/excel-sip-changes/main/dsp.c
U   team/ctooley/excel-sip-changes/main/features.c
U   team/ctooley/excel-sip-changes/main/file.c
U   team/ctooley/excel-sip-changes/main/frame.c
U   team/ctooley/excel-sip-changes/main/indications.c
U   team/ctooley/excel-sip-changes/main/rtp.c
U   team/ctooley/excel-sip-changes/main/slinfactory.c
U   team/ctooley/excel-sip-changes/main/translate.c
U   team/ctooley/excel-sip-changes/main/udptl.c
U   team/ctooley/excel-sip-changes/res/res_adsi.c
U   team/ctooley/excel-sip-changes/res/res_agi.c
U   team/ctooley/excel-sip-changes/res/res_musiconhold.c

------------------------------------------------------------------------
r117831 | ctooley | 2008-05-22 12:07:53 -0500 (Thu, 22 May 2008) | 60
lines

Merged revisions 117792,117794,117802,117812,117822 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
r117792 | seanbright | 2008-05-22 10:49:17 -0500 (Thu, 22 May 2008) | 1
line

Minor text fix.  roster -> resource.
................
r117794 | mmichelson | 2008-05-22 11:05:18 -0500 (Thu, 22 May 2008) | 3
lines

Committing a fix pointed out by Atis Lezdins on the asterisk-dev list.
Thanks!


................
r117802 | mvanbaak | 2008-05-22 11:29:54 -0500 (Thu, 22 May 2008) | 21
lines

- revert change to ast_queue_hangup and create ast_queue_hangup_with_cause
- make data member of the ast_frame struct a named union instead of a void

Recently the ast_queue_hangup function got a new parameter, the
hangupcause
Feedback came in that this is no good and that instead a new function
should be created.
This I did.

The hangupcause was stored in the seqno member of the ast_frame struct.
This is not very
elegant, and since there's already a data member that one should be used.
Problem is, this member was a void *.
Now it's a named union so it can hold a pointer, an uint32 and there's a
padding in case someone
wants to store another type in there in the future.

This commit is so massive, because all ast_frame.data uses have to be
altered to ast_frame.data.data

Thanks russellb and kpfleming for the feedback.

(closes issue http://bugs.digium.com/view.php?id=12674)
Reported by: mvanbaak

................
r117812 | tilghman | 2008-05-22 11:50:32 -0500 (Thu, 22 May 2008) | 13
lines

Blocked revisions 117809 via svnmerge

........
r117809 | tilghman | 2008-05-22 11:47:03 -0500 (Thu, 22 May 2008) | 6
lines

Take into account the length of delimiters when calculating result string
length.
(closes issue http://bugs.digium.com/view.php?id=12696)
 Reported by: adomjan
 Patches: 
       func_realtime.c-longdelimiter.patch uploaded by adomjan (license
487)

........

................
r117822 | mvanbaak | 2008-05-22 12:06:00 -0500 (Thu, 22 May 2008) | 2
lines

forgot chan_misdn

................

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

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

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
05-22-08 12:07  svnbot         Note Added: 0087215                          
======================================================================




More information about the asterisk-bugs mailing list