[Home]

Summary:ASTERISK-12185: use ast_getformatname_multiple instead of ast_getformatname for log message
Reporter:klaus3000 (klaus3000)Labels:
Date Opened:2008-06-13 02:41:55Date Closed:2008-06-19 11:57:44
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/Channels
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:Hi!

When debugging codec problems (non matching codecs) I often see the log message:

WARNING[19062]: channel.c:2757 set_format: Unable to find a codec translation path from unknown to unknown

This log message is not very useful as it does not display the codecs as a channel my support multiple codecs. Using ast_getformatname_multiple instead of ast_getformatname would solve this issue

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

The problem is in main/channel.c in the function
 static int set_format()

(just grep for "Unable to find a codec translation path")

The bug is present in 1.4.21 and 1.6
Comments:By: Digium Subversion (svnbot) 2008-06-19 11:51:19

Repository: asterisk
Revision: 123930

U   branches/1.4/main/channel.c

------------------------------------------------------------------------
r123930 | tilghman | 2008-06-19 11:51:17 -0500 (Thu, 19 Jun 2008) | 5 lines

Change informative messages to use the _multiple variant when multiple formats
are possible.
(Closes issue ASTERISK-12185)
Reported by klaus3000

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

http://svn.digium.com/view/asterisk?view=rev&revision=123930
By: Digium Subversion (svnbot) 2008-06-19 11:55:53

Repository: asterisk
Revision: 123931

_U  trunk/
U   trunk/main/channel.c

------------------------------------------------------------------------
r123931 | tilghman | 2008-06-19 11:55:52 -0500 (Thu, 19 Jun 2008) | 13 lines

Merged revisions 123930 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r123930 | tilghman | 2008-06-19 11:58:19 -0500 (Thu, 19 Jun 2008) | 5 lines

Change informative messages to use the _multiple variant when multiple formats
are possible.
(Closes issue ASTERISK-12185)
Reported by klaus3000

........

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

http://svn.digium.com/view/asterisk?view=rev&revision=123931
By: Digium Subversion (svnbot) 2008-06-19 11:57:44

Repository: asterisk
Revision: 123932

_U  branches/1.6.0/
U   branches/1.6.0/main/channel.c

------------------------------------------------------------------------
r123932 | tilghman | 2008-06-19 11:57:43 -0500 (Thu, 19 Jun 2008) | 21 lines

Merged revisions 123931 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
r123931 | tilghman | 2008-06-19 12:02:54 -0500 (Thu, 19 Jun 2008) | 13 lines

Merged revisions 123930 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r123930 | tilghman | 2008-06-19 11:58:19 -0500 (Thu, 19 Jun 2008) | 5 lines

Change informative messages to use the _multiple variant when multiple formats
are possible.
(Closes issue ASTERISK-12185)
Reported by klaus3000

........

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

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

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