kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-06 12:36:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

when unregistering a translator, don't rebuild the translation matrix unless needed

Browse Source 
when filtering formats out of an offer, ensure we check for translation ability in both directions


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@46526 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Kevin P. Fleming
2006-10-30 22:19:55 +00:00
parent c874ff482b
commit d8817c0d3c
1 changed files with 17 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
main/translate.c
View File

@@ -729,18 +729,25 @@ int ast_unregister_translator(struct ast_translator *t)
{ {
char tmp[80]; char tmp[80];
struct ast_translator *u; struct ast_translator *u;
int found = 0;
AST_LIST_LOCK(&translators); AST_LIST_LOCK(&translators);
AST_LIST_TRAVERSE_SAFE_BEGIN(&translators, u, list) { AST_LIST_TRAVERSE_SAFE_BEGIN(&translators, u, list) {
if (u == t) { if (u == t) {
AST_LIST_REMOVE_CURRENT(&translators, list); AST_LIST_REMOVE_CURRENT(&translators, list);
if (option_verbose > 1) if (option_verbose > 1)
ast_verbose(VERBOSE_PREFIX_2 "Unregistered translator '%s' from format %s to %s\n", term_color(tmp, t->name, COLOR_MAGENTA, COLOR_BLACK, sizeof(tmp)), ast_getformatname(1 << t->srcfmt), ast_getformatname(1 << t->dstfmt)); ast_verbose(VERBOSE_PREFIX_2 "Unregistered translator '%s' from format %s to %s\n", term_color(tmp, t->name, COLOR_MAGENTA, COLOR_BLACK, sizeof(tmp)), ast_getformatname(1 << t->srcfmt), ast_getformatname(1 << t->dstfmt));
found = 1;
break; break;
} }
} }
AST_LIST_TRAVERSE_SAFE_END; AST_LIST_TRAVERSE_SAFE_END;
rebuild_matrix(0);
if (found)
rebuild_matrix(0);
AST_LIST_UNLOCK(&translators); AST_LIST_UNLOCK(&translators);
return (u ? 0 : -1); return (u ? 0 : -1);
} }
@@ -850,6 +857,11 @@ unsigned int ast_translate_available_formats(unsigned int dest, unsigned int src
to this format, remove it from the result */ to this format, remove it from the result */
if (!tr_matrix[src_audio][powerof(x)].step) if (!tr_matrix[src_audio][powerof(x)].step)
res &= ~x; res &= ~x;
/* now check the opposite direction */
if (!tr_matrix[powerof(x)][src_audio].step)
res &= ~x;
} }
/* For a given source video format, traverse the list of /* For a given source video format, traverse the list of
@@ -870,6 +882,10 @@ unsigned int ast_translate_available_formats(unsigned int dest, unsigned int src
to this format, remove it from the result */ to this format, remove it from the result */
if (!tr_matrix[src_video][powerof(x)].step) if (!tr_matrix[src_video][powerof(x)].step)
res &= ~x; res &= ~x;
/* now check the opposite direction */
if (!tr_matrix[powerof(x)][src_video].step)
res &= ~x;
} }
AST_LIST_UNLOCK(&translators); AST_LIST_UNLOCK(&translators);