[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