[asterisk-commits] kharwell: branch kharwell/media_formats_translation_core r413540 - in /team/k...
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Thu May 8 15:31:28 CDT 2014
Author: kharwell
Date: Thu May 8 15:31:20 2014
New Revision: 413540
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=413540
Log:
another round of review finding changes
Modified:
team/kharwell/media_formats_translation_core/include/asterisk/codec.h
team/kharwell/media_formats_translation_core/include/asterisk/format.h
team/kharwell/media_formats_translation_core/main/codec.c
team/kharwell/media_formats_translation_core/main/format.c
team/kharwell/media_formats_translation_core/main/translate.c
Modified: team/kharwell/media_formats_translation_core/include/asterisk/codec.h
URL: http://svnview.digium.com/svn/asterisk/team/kharwell/media_formats_translation_core/include/asterisk/codec.h?view=diff&rev=413540&r1=413539&r2=413540
==============================================================================
--- team/kharwell/media_formats_translation_core/include/asterisk/codec.h (original)
+++ team/kharwell/media_formats_translation_core/include/asterisk/codec.h Thu May 8 15:31:20 2014
@@ -177,15 +177,4 @@
*/
unsigned int ast_codec_determine_length(const struct ast_codec *codec, unsigned int samples);
-/*!
- * \brief Get the names of all codecs, separated by a '|', matching the given
- * codec's original id.
- *
- * \param codec The codec to match against
- * \param buf the string buffer to write names to
- *
- * \retval the given string buffer.
- */
-struct ast_str **ast_codec_get_names(const struct ast_codec *codec, struct ast_str **buf);
-
#endif /* _AST_CODEC_H */
Modified: team/kharwell/media_formats_translation_core/include/asterisk/format.h
URL: http://svnview.digium.com/svn/asterisk/team/kharwell/media_formats_translation_core/include/asterisk/format.h?view=diff&rev=413540&r1=413539&r2=413540
==============================================================================
--- team/kharwell/media_formats_translation_core/include/asterisk/format.h (original)
+++ team/kharwell/media_formats_translation_core/include/asterisk/format.h Thu May 8 15:31:20 2014
@@ -138,14 +138,6 @@
enum ast_format_cmp_res ast_format_cmp(const struct ast_format *format1, const struct ast_format *format2);
/*!
- * \brief Compare the format's codec to the given codec.
- *
- * \retval ast_format_cmp_res representing the result of comparing format->codec and codec
- */
-enum ast_format_cmp_res ast_format_cmp_codec(
- const struct ast_format *format, const struct ast_codec *codec);
-
-/*1
* \brief Get a common joint capability between two formats
*
* \retval non-NULL if joint capability exists
Modified: team/kharwell/media_formats_translation_core/main/codec.c
URL: http://svnview.digium.com/svn/asterisk/team/kharwell/media_formats_translation_core/main/codec.c?view=diff&rev=413540&r1=413539&r2=413540
==============================================================================
--- team/kharwell/media_formats_translation_core/main/codec.c (original)
+++ team/kharwell/media_formats_translation_core/main/codec.c Thu May 8 15:31:20 2014
@@ -352,30 +352,3 @@
return codec->get_length(samples);
}
-static int codec_append_name(void *obj, void *arg, int flags)
-{
- const struct ast_codec *left = obj;
- const struct ast_codec *right = arg;
-
- if (left == right) {
- ast_str_append(buf, 0, "%s|", left->name);
- }
- return 0;
-}
-
-struct ast_str **ast_codec_get_names(const struct ast_codec *codec,
- struct ast_str **buf)
-{
- int size = ast_str_strlen(*buf) + 1;
-
- ast_str_append(buf, 0, "(");
- ao2_callback(codecs, OBJ_NODATA, codec_append_name, codec);
-
- if (size == ast_str_strlen(*buf)) {
- ast_str_append(buf, 0, "nothing)");
- } else {
- ast_str_truncate(buf, -1);
- ast_str_append(buf, 0, ")");
- }
- return buf;
-}
Modified: team/kharwell/media_formats_translation_core/main/format.c
URL: http://svnview.digium.com/svn/asterisk/team/kharwell/media_formats_translation_core/main/format.c?view=diff&rev=413540&r1=413539&r2=413540
==============================================================================
--- team/kharwell/media_formats_translation_core/main/format.c (original)
+++ team/kharwell/media_formats_translation_core/main/format.c Thu May 8 15:31:20 2014
@@ -205,11 +205,6 @@
return AST_FORMAT_CMP_EQUAL;
}
-enum ast_format_cmp_res ast_format_cmp_codec(const struct ast_format *format1, const struct ast_codec *codec)
-{
- return format->codec == codec ? AST_FORMAT_CMP_EQUAL : AST_FORMAT_CMP_NOT_EQUAL;
-}
-
struct ast_format *ast_format_joint(const struct ast_format *format1, const struct ast_format *format2)
{
if (format1->codec != format2->codec) {
Modified: team/kharwell/media_formats_translation_core/main/translate.c
URL: http://svnview.digium.com/svn/asterisk/team/kharwell/media_formats_translation_core/main/translate.c?view=diff&rev=413540&r1=413539&r2=413540
==============================================================================
--- team/kharwell/media_formats_translation_core/main/translate.c (original)
+++ team/kharwell/media_formats_translation_core/main/translate.c Thu May 8 15:31:20 2014
@@ -696,8 +696,8 @@
return 0;
}
- src_ll = ast_format_cmp_codec(ast_format_slin, src) == AST_FORMAT_CMP_EQUAL;
- dst_ll = ast_format_cmp_codec(ast_format_slin, dst) == AST_FORMAT_CMP_EQUAL;
+ src_ll = !strcmp(src->name, "slin");
+ dst_ll = !strcmp(dst->name, "slin");
if (src_ll) {
if (dst_ll && (src_rate == dst_rate)) {
return AST_TRANS_COST_LL_LL_ORIGSAMP;
@@ -827,16 +827,25 @@
}
}
+static void codec_append_name(const struct ast_codec *codec, struct ast_str **buf)
+{
+ if (codec) {
+ ast_str_append(buf, 0, "(%s@%d)", codec->name, codec->sample_rate);
+ } else {
+ ast_str_append(buf, 0, "(nothing)");
+ }
+}
+
const char *ast_translate_path_to_str(struct ast_trans_pvt *p, struct ast_str **str)
{
if (!p || !p->t) {
return "";
}
- ast_codec_get_names(&p->t->src_codec, str);
+ codec_append_name(&p->t->src_codec, str);
while (p) {
ast_str_append(str, 0, "->");
- ast_codec_get_names(&p->t->dst_codec, str);
+ codec_append_name(&p->t->dst_codec, str);
p = p->next;
}
@@ -998,7 +1007,7 @@
ast_str_reset(str);
if (step) {
- ast_codec_get_names(&step->src_codec, &str);
+ codec_append_name(&step->src_codec, &str);
while (src != dst) {
step = matrix_get(src, dst)->step;
if (!step) {
@@ -1006,7 +1015,7 @@
break;
}
ast_str_append(&str, 0, "->");
- ast_codec_get_names(&step->src_codec, &str);
+ codec_append_name(&step->src_codec, &str);
src = step->dst_fmt_index;
}
}
More information about the asterisk-commits
mailing list