Summary: | ASTERISK-09896: undefined codecs in AST_FORMAT_LIST | ||
Reporter: | klaus3000 (klaus3000) | Labels: | |
Date Opened: | 2007-07-18 03:56:12 | Date Closed: | 2007-08-07 10:23:12 |
Priority: | Trivial | Regression? | No |
Status: | Closed/Complete | Components: | PBX/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ||
Description: | Hi! In main/frame.c the AST_FORMAT_LIST contains several entries like: { 0, 0, "nothing", "undefined" }, IMHO this entries just cause confusion because the layout of AST_FORMAT_LIST is independent from the codec list in frame.h. Thus, all the "undefined" lines could be removed. | ||
Comments: | By: Jason Parker (jparker) 2007-07-20 18:09:04 I looked through frame.c, and I believe that you're correct.. It may have mattered at one point, but they have the bit field, which links it to the codec in frame.h It's also interesting to note that it's currently out of sync with the codec list. By: Digium Subversion (svnbot) 2007-08-07 10:23:10 Repository: asterisk Revision: 78338 ------------------------------------------------------------------------ r78338 | file | 2007-08-07 10:23:08 -0500 (Tue, 07 Aug 2007) | 4 lines (closes issue ASTERISK-9896) Reported by: klaus3000 Clean up AST_FORMAT_LIST list. It may have mattered in the old days to have undefined entries but these days it does not. ------------------------------------------------------------------------ |