[Home]

Summary:ASTERISK-00088: Show codec names instead of numbers
Reporter:John Laur (gork)Labels:
Date Opened:2003-08-12 14:54:15Date Closed:2008-01-15 14:32:06.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) show_codec_names-08122003gork.diff
Description:Instead of printing format numbers for audio codecs, image formats, and video frame types, print format names in translation cost matrix and elsewhere in debug messages.

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

I have included a patch below that I believe to be reasonably complete. If I have missed something, it's a very elusive error message buried somewhere. This patch is current as of Aug. 12, 2003 CVS. John Todd and others, I know, would be very appreciative if this could go into CVS :)
Comments:By: x martinp (martinp) 2003-08-12 21:25:54

Did you fill and send the disclaimer ?
By: John Laur (gork) 2003-08-12 21:41:42

Yes I had sent a scan via email to mark quite a long time ago.. a month or more, anyway. Please let me know if you do not have it for whatever reason and I'll fax it again tomorrow.

Please contact me via email at johnl@blurbco.com

edited on: 08-12-03 21:26
By: John Laur (gork) 2003-08-15 11:19:34

Have you gotten the disclaimer yet? This patch is very difficult to maintain recently as it touches so many files :)
By: x martinp (martinp) 2003-08-16 01:02:41

Fix in CVS. Although I think that the function ast_getformatname should include printing sets of codecs like capability, etc... since asterisk codecs are the power of 2 it's easy to separate it from the capabilities .... and print like that: G729A/GSM/ALAW/ULAW ...
By: Digium Subversion (svnbot) 2008-01-15 14:32:06.000-0600

Repository: asterisk
Revision: 1348

U   trunk/apps/app_agi.c
U   trunk/apps/app_voicemail2.c
U   trunk/channel.c
U   trunk/channels/chan_agent.c
U   trunk/channels/chan_iax.c
U   trunk/channels/chan_iax2.c
U   trunk/channels/chan_modem.c
U   trunk/channels/chan_phone.c
U   trunk/channels/chan_sip.c
U   trunk/channels/chan_zap.c
U   trunk/channels/chan_zap_old.c
U   trunk/file.c
U   trunk/frame.c
U   trunk/image.c
U   trunk/include/asterisk/frame.h
U   trunk/res/res_musiconhold.c
U   trunk/rtp.c
U   trunk/translate.c

------------------------------------------------------------------------
r1348 | martinp | 2008-01-15 14:32:05 -0600 (Tue, 15 Jan 2008) | 2 lines

Show the names of the codecs instead of the numbers (bug ASTERISK-88)

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

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