[Asterisk-code-review] codec opus: Replace res format attr opus with the one from c... (asterisk[master])
Kevin Harwell
asteriskteam at digium.com
Tue Sep 27 13:42:15 CDT 2016
Kevin Harwell has uploaded a new change for review.
https://gerrit.asterisk.org/3988
Change subject: codec_opus: Replace res_format_attr_opus with the one from codec_opus
......................................................................
codec_opus: Replace res_format_attr_opus with the one from codec_opus
Preparation
ASTERISK-26409
Change-Id: I9f20e7cce00c32464d9a180e81283d49d199d0a3
(cherry picked from commit 59f7662a93bf9c07204fb50e1020a0f5bfbbd5c9)
---
M include/asterisk/opus.h
M res/res_format_attr_opus.c
2 files changed, 227 insertions(+), 178 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/88/3988/1
diff --git a/include/asterisk/opus.h b/include/asterisk/opus.h
index 0fbcfa1..6d2cdcf 100644
--- a/include/asterisk/opus.h
+++ b/include/asterisk/opus.h
@@ -18,24 +18,47 @@
/*!
* \file
- * \brief Opus Format Attributes (http://tools.ietf.org/html/draft-ietf-payload-rtp-opus)
+ * \brief Codec opus externals and format attributes
*
- * \author Lorenzo Miniero <lorenzo at meetecho.com>
+ * RFC - https://tools.ietf.org/rfc/rfc7587.txt
*/
#ifndef _AST_FORMAT_OPUS_H_
#define _AST_FORMAT_OPUS_H_
-/*! Opus format attribute key value pairs, all are accessible through ast_format_get_value()*/
-enum opus_attr_keys {
- OPUS_ATTR_KEY_MAX_BITRATE, /*! value is an int (6000-510000 in spec). */
- OPUS_ATTR_KEY_MAX_PLAYRATE, /*! value is an int (8000-48000), maximum output rate the receiver can render. */
- OPUS_ATTR_KEY_MINPTIME, /*! value is an int (3-120 in spec, 10-60 in format.c), decoder's minimum length of time in milliseconds. */
- OPUS_ATTR_KEY_STEREO, /*! value is an int, 1 prefer receiving stereo, 0 prefer mono. */
- OPUS_ATTR_KEY_CBR, /*! value is an int, 1 use constant bitrate, 0 use variable bitrate. */
- OPUS_ATTR_KEY_FEC, /*! value is an int, 1 encode with FEC, 0 do not use FEC. */
- OPUS_ATTR_KEY_DTX, /*! value is an int, 1 dtx is enabled, 0 dtx not enabled. */
- OPUS_ATTR_KEY_SPROP_CAPTURE_RATE, /*! value is an int (8000-48000), likely input rate we're going to produce. */
- OPUS_ATTR_KEY_SPROP_STEREO, /*! value is an int, 1 likely to send stereo, 0 likely to send mono. */
-};
+/*! \brief Maximum sampling rate an endpoint is capable of receiving */
+#define CODEC_OPUS_ATTR_MAX_PLAYBACK_RATE "maxplaybackrate"
+/*! \brief An alias for maxplaybackrate (used in older versions) */
+#define CODEC_OPUS_ATTR_MAX_CODED_AUDIO_BANDWIDTH "maxcodedaudiobandwidth"
+/*! \brief Maximum sampling rate an endpoint is capable of sending */
+#define CODEC_OPUS_ATTR_SPROP_MAX_CAPTURE_RATE "sprop-maxcapturerate"
+/*! \brief Maximum duration of packet (in milliseconds) */
+#define CODEC_OPUS_ATTR_MAX_PTIME "maxptime"
+/*! \brief Duration of packet (in milliseconds) */
+#define CODEC_OPUS_ATTR_PTIME "ptime"
+/*! \brief Maximum average received bit rate (in bits per second) */
+#define CODEC_OPUS_ATTR_MAX_AVERAGE_BITRATE "maxaveragebitrate"
+/*! \brief Decode stereo (1) vs mono (0) */
+#define CODEC_OPUS_ATTR_STEREO "stereo"
+/*! \brief Likeliness of sender producing stereo (1) vs mono (0) */
+#define CODEC_OPUS_ATTR_SPROP_STEREO "sprop-stereo"
+/*! \brief Decoder prefers a constant (1) vs variable (0) bitrate */
+#define CODEC_OPUS_ATTR_CBR "cbr"
+/*! \brief Use forward error correction (1) or not (0) */
+#define CODEC_OPUS_ATTR_FEC "useinbandfec"
+/*! \brief Use discontinuous transmission (1) or not (0) */
+#define CODEC_OPUS_ATTR_DTX "usedtx"
+/*! \brief Custom data object */
+#define CODEC_OPUS_ATTR_DATA "data"
+
+/*! \brief Default attribute values */
+#define CODEC_OPUS_DEFAULT_SAMPLE_RATE 48000
+#define CODEC_OPUS_DEFAULT_MAX_PLAYBACK_RATE 48000
+#define CODEC_OPUS_DEFAULT_MAX_PTIME 120
+#define CODEC_OPUS_DEFAULT_PTIME 20
+#define CODEC_OPUS_DEFAULT_BITRATE -1000 /* OPUS_AUTO */
+#define CODEC_OPUS_DEFAULT_CBR 0
+#define CODEC_OPUS_DEFAULT_FEC 0
+#define CODEC_OPUS_DEFAULT_DTX 0
+#define CODEC_OPUS_DEFAULT_STEREO 0
#endif /* _AST_FORMAT_OPUS_H */
diff --git a/res/res_format_attr_opus.c b/res/res_format_attr_opus.c
index 4c09bef..dfdf9b6 100644
--- a/res/res_format_attr_opus.c
+++ b/res/res_format_attr_opus.c
@@ -33,9 +33,10 @@
#include "asterisk/module.h"
#include "asterisk/format.h"
-#include "asterisk/logger.h" /* for ast_log, LOG_WARNING */
-#include "asterisk/strings.h" /* for ast_str_append */
-#include "asterisk/utils.h" /* for MIN, ast_malloc, ast_free */
+#include "asterisk/logger.h"
+#include "asterisk/strings.h"
+#include "asterisk/utils.h"
+#include "asterisk/opus.h"
/*!
* \brief Opus attribute structure.
@@ -43,32 +44,42 @@
* \note http://tools.ietf.org/html/rfc7587#section-6
*/
struct opus_attr {
- unsigned int maxbitrate;
- unsigned int maxplayrate;
- unsigned int unused; /* was minptime, kept for binary compatibility */
- unsigned int stereo;
- unsigned int cbr;
- unsigned int fec;
- unsigned int dtx;
- unsigned int spropmaxcapturerate;
- unsigned int spropstereo;
+ int maxbitrate;
+ int maxplayrate;
+ int ptime;
+ int stereo;
+ int cbr;
+ int fec;
+ int dtx;
+ int spropmaxcapturerate;
+ int spropstereo;
+ int maxptime;
+ /* Note data is expected to be an ao2_object type */
+ void *data;
};
static struct opus_attr default_opus_attr = {
- .maxplayrate = 48000,
- .spropmaxcapturerate = 48000,
- .maxbitrate = 510000,
- .stereo = 0,
- .spropstereo = 0,
- .cbr = 0,
- .fec = 1,
- .dtx = 0,
+ .maxbitrate = CODEC_OPUS_DEFAULT_BITRATE,
+ .maxplayrate = CODEC_OPUS_DEFAULT_SAMPLE_RATE,
+ .ptime = CODEC_OPUS_DEFAULT_PTIME,
+ .stereo = CODEC_OPUS_DEFAULT_STEREO,
+ .cbr = CODEC_OPUS_DEFAULT_CBR,
+ .fec = CODEC_OPUS_DEFAULT_FEC,
+ .dtx = CODEC_OPUS_DEFAULT_DTX,
+ .spropmaxcapturerate = CODEC_OPUS_DEFAULT_SAMPLE_RATE,
+ .spropstereo = CODEC_OPUS_DEFAULT_STEREO,
+ .maxptime = CODEC_OPUS_DEFAULT_MAX_PTIME
};
static void opus_destroy(struct ast_format *format)
{
struct opus_attr *attr = ast_format_get_attribute_data(format);
+ if (!attr) {
+ return;
+ }
+
+ ao2_cleanup(attr->data);
ast_free(attr);
}
@@ -81,81 +92,65 @@
return -1;
}
- if (original) {
- *attr = *original;
- } else {
- *attr = default_opus_attr;
- }
+ *attr = original ? *original : default_opus_attr;
+ ao2_bump(attr->data);
ast_format_set_attribute_data(dst, attr);
return 0;
}
+static void sdp_fmtp_get(const char *attributes, const char *name, int *attr)
+{
+ const char *kvp = "";
+ int val;
+
+ if (attributes && !(kvp = strstr(attributes, name))) {
+ return;
+ }
+
+ /*
+ * If the named attribute is not at the start of the given attributes, and
+ * the preceding character is not a space or semicolon then it's not the
+ * attribute we are looking for. It's an attribute with the name embedded
+ * within it (e.g. ptime in maxptime, stereo in sprop-stereo).
+ */
+ if (kvp != attributes && *(kvp - 1) != ' ' && *(kvp - 1) != ';') {
+ /* Keep searching as it might still be in the attributes string */
+ sdp_fmtp_get(strchr(kvp, ';'), name, attr);
+ /*
+ * Otherwise it's a match, so retrieve the value and set the attribute.
+ */
+ } else if (sscanf(kvp, "%*[^=]=%30d", &val) == 1) {
+ *attr = val;
+ }
+}
+
static struct ast_format *opus_parse_sdp_fmtp(const struct ast_format *format, const char *attributes)
{
struct ast_format *cloned;
struct opus_attr *attr;
- const char *kvp;
- unsigned int val;
cloned = ast_format_clone(format);
if (!cloned) {
return NULL;
}
+
attr = ast_format_get_attribute_data(cloned);
- if ((kvp = strstr(attributes, "maxplaybackrate")) && sscanf(kvp, "maxplaybackrate=%30u", &val) == 1) {
- attr->maxplayrate = val;
- } else {
- attr->maxplayrate = 48000;
- }
-
- if ((kvp = strstr(attributes, "sprop-maxcapturerate")) && sscanf(kvp, "sprop-maxcapturerate=%30u", &val) == 1) {
- attr->spropmaxcapturerate = val;
- } else {
- attr->spropmaxcapturerate = 48000;
- }
-
- if ((kvp = strstr(attributes, "maxaveragebitrate")) && sscanf(kvp, "maxaveragebitrate=%30u", &val) == 1) {
- attr->maxbitrate = val;
- } else {
- attr->maxbitrate = 510000;
- }
-
- if (!strncmp(attributes, "stereo=1", 8)) {
- attr->stereo = 1;
- } else if (strstr(attributes, " stereo=1")) {
- attr->stereo = 1;
- } else if (strstr(attributes, ";stereo=1")) {
- attr->stereo = 1;
- } else {
- attr->stereo = 0;
- }
-
- if (strstr(attributes, "sprop-stereo=1")) {
- attr->spropstereo = 1;
- } else {
- attr->spropstereo = 0;
- }
-
- if (strstr(attributes, "cbr=1")) {
- attr->cbr = 1;
- } else {
- attr->cbr = 0;
- }
-
- if (strstr(attributes, "useinbandfec=1")) {
- attr->fec = 1;
- } else {
- attr->fec = 0;
- }
-
- if (strstr(attributes, "usedtx=1")) {
- attr->dtx = 1;
- } else {
- attr->dtx = 0;
- }
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_MAX_PLAYBACK_RATE, &attr->maxplayrate);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_MAX_CODED_AUDIO_BANDWIDTH,
+ &attr->maxplayrate);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_SPROP_MAX_CAPTURE_RATE,
+ &attr->spropmaxcapturerate);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_MAX_PTIME, &attr->maxptime);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_PTIME, &attr->ptime);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_MAX_AVERAGE_BITRATE, &attr->maxbitrate);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_STEREO, &attr->stereo);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_SPROP_STEREO, &attr->spropstereo);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_CBR, &attr->cbr);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_FEC, &attr->fec);
+ sdp_fmtp_get(attributes, CODEC_OPUS_ATTR_DTX, &attr->dtx);
return cloned;
}
@@ -163,7 +158,7 @@
static void opus_generate_sdp_fmtp(const struct ast_format *format, unsigned int payload, struct ast_str **str)
{
struct opus_attr *attr = ast_format_get_attribute_data(format);
- int added = 0;
+ int size;
if (!attr) {
/*
@@ -174,79 +169,52 @@
attr = &default_opus_attr;
}
- if (48000 != attr->maxplayrate) {
- if (added) {
- ast_str_append(str, 0, ";");
- } else if (0 < ast_str_append(str, 0, "a=fmtp:%u ", payload)) {
- added = 1;
- }
- ast_str_append(str, 0, "maxplaybackrate=%u", attr->maxplayrate);
+ size = ast_str_append(str, 0, "a=fmtp:%u ", payload);
+
+ if (CODEC_OPUS_DEFAULT_SAMPLE_RATE != attr->maxplayrate) {
+ ast_str_append(str, 0, "%s=%d;",
+ CODEC_OPUS_ATTR_MAX_PLAYBACK_RATE, attr->maxplayrate);
}
- if (48000 != attr->spropmaxcapturerate) {
- if (added) {
- ast_str_append(str, 0, ";");
- } else if (0 < ast_str_append(str, 0, "a=fmtp:%u ", payload)) {
- added = 1;
- }
- ast_str_append(str, 0, "sprop-maxcapturerate=%u", attr->spropmaxcapturerate);
+ if (CODEC_OPUS_DEFAULT_SAMPLE_RATE != attr->spropmaxcapturerate) {
+ ast_str_append(str, 0, "%s=%d;",
+ CODEC_OPUS_ATTR_SPROP_MAX_CAPTURE_RATE, attr->spropmaxcapturerate);
}
- if (510000 != attr->maxbitrate) {
- if (added) {
- ast_str_append(str, 0, ";");
- } else if (0 < ast_str_append(str, 0, "a=fmtp:%u ", payload)) {
- added = 1;
- }
- ast_str_append(str, 0, "maxaveragebitrate=%u", attr->maxbitrate);
+ if (CODEC_OPUS_DEFAULT_BITRATE != attr->maxbitrate || attr->maxbitrate > 0) {
+ ast_str_append(str, 0, "%s=%d;",
+ CODEC_OPUS_ATTR_MAX_AVERAGE_BITRATE, attr->maxbitrate);
}
- if (0 != attr->stereo) {
- if (added) {
- ast_str_append(str, 0, ";");
- } else if (0 < ast_str_append(str, 0, "a=fmtp:%u ", payload)) {
- added = 1;
- }
- ast_str_append(str, 0, "stereo=%u", attr->stereo);
+ if (CODEC_OPUS_DEFAULT_STEREO != attr->stereo) {
+ ast_str_append(str, 0, "%s=%d;",
+ CODEC_OPUS_ATTR_STEREO, attr->stereo);
}
- if (0 != attr->spropstereo) {
- if (added) {
- ast_str_append(str, 0, ";");
- } else if (0 < ast_str_append(str, 0, "a=fmtp:%u ", payload)) {
- added = 1;
- }
- ast_str_append(str, 0, "sprop-stereo=%u", attr->spropstereo);
+ if (CODEC_OPUS_DEFAULT_STEREO != attr->spropstereo) {
+ ast_str_append(str, 0, "%s=%d;",
+ CODEC_OPUS_ATTR_SPROP_STEREO, attr->spropstereo);
}
- if (0 != attr->cbr) {
- if (added) {
- ast_str_append(str, 0, ";");
- } else if (0 < ast_str_append(str, 0, "a=fmtp:%u ", payload)) {
- added = 1;
- }
- ast_str_append(str, 0, "cbr=%u", attr->cbr);
+ if (CODEC_OPUS_DEFAULT_CBR != attr->cbr) {
+ ast_str_append(str, 0, "%s=%d;",
+ CODEC_OPUS_ATTR_CBR, attr->cbr);
}
- if (0 != attr->fec) {
- if (added) {
- ast_str_append(str, 0, ";");
- } else if (0 < ast_str_append(str, 0, "a=fmtp:%u ", payload)) {
- added = 1;
- }
- ast_str_append(str, 0, "useinbandfec=%u", attr->fec);
+ if (CODEC_OPUS_DEFAULT_FEC!= attr->fec) {
+ ast_str_append(str, 0, "%s=%d;",
+ CODEC_OPUS_ATTR_FEC, attr->fec);
}
- if (0 != attr->dtx) {
- if (added) {
- ast_str_append(str, 0, ";");
- } else if (0 < ast_str_append(str, 0, "a=fmtp:%u ", payload)) {
- added = 1;
- }
- ast_str_append(str, 0, "usedtx=%u", attr->dtx);
+ if (CODEC_OPUS_DEFAULT_DTX != attr->dtx) {
+ ast_str_append(str, 0, "%s=%d;",
+ CODEC_OPUS_ATTR_DTX, attr->dtx);
}
- if (added) {
+ if (size == ast_str_strlen(*str)) {
+ ast_str_reset(*str);
+ } else {
+ ast_str_truncate(*str, -1);
ast_str_append(str, 0, "\r\n");
}
}
@@ -285,54 +253,111 @@
* to receive stereo signals, it may be a waste of bandwidth. */
attr_res->stereo = attr1->stereo && attr2->stereo ? 1 : 0;
- attr_res->maxbitrate = MIN(attr1->maxbitrate, attr2->maxbitrate);
+ if (attr1->maxbitrate < 0) {
+ attr_res->maxbitrate = attr2->maxbitrate;
+ } else if (attr2->maxbitrate < 0) {
+ attr_res->maxbitrate = attr1->maxbitrate;
+ } else {
+ attr_res->maxbitrate = MIN(attr1->maxbitrate, attr2->maxbitrate);
+ }
+
attr_res->spropmaxcapturerate = MIN(attr1->spropmaxcapturerate, attr2->spropmaxcapturerate);
attr_res->maxplayrate = MIN(attr1->maxplayrate, attr2->maxplayrate);
return jointformat;
}
-static struct ast_format *opus_set(const struct ast_format *format, const char *name, const char *value)
+static struct ast_format *opus_set(const struct ast_format *format,
+ const char *name, const char *value)
{
struct ast_format *cloned;
struct opus_attr *attr;
- unsigned int val;
+ int val;
- if (sscanf(value, "%30u", &val) != 1) {
- ast_log(LOG_WARNING, "Unknown value '%s' for attribute type '%s'\n",
- value, name);
+ if (!(cloned = ast_format_clone(format))) {
return NULL;
}
- cloned = ast_format_clone(format);
- if (!cloned) {
- return NULL;
- }
attr = ast_format_get_attribute_data(cloned);
- if (!strcasecmp(name, "max_bitrate")) {
- attr->maxbitrate = val;
- } else if (!strcasecmp(name, "max_playrate")) {
+ if (!strcmp(name, CODEC_OPUS_ATTR_DATA)) {
+ ao2_cleanup(attr->data);
+ attr->data = ao2_bump((void*)value);
+ return cloned;
+ }
+
+ if (sscanf(value, "%30d", &val) != 1) {
+ ast_log(LOG_WARNING, "Unknown value '%s' for attribute type '%s'\n",
+ value, name);
+ ao2_ref(cloned, -1);
+ return NULL;
+ }
+
+ if (!strcasecmp(name, CODEC_OPUS_ATTR_MAX_PLAYBACK_RATE)) {
attr->maxplayrate = val;
- } else if (!strcasecmp(name, "minptime")) {
- attr->unused = val;
- } else if (!strcasecmp(name, "stereo")) {
- attr->stereo = val;
- } else if (!strcasecmp(name, "cbr")) {
- attr->cbr = val;
- } else if (!strcasecmp(name, "fec")) {
- attr->fec = val;
- } else if (!strcasecmp(name, "dtx")) {
- attr->dtx = val;
- } else if (!strcasecmp(name, "sprop_capture_rate")) {
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_MAX_CODED_AUDIO_BANDWIDTH)) {
+ attr->maxplayrate = val;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_SPROP_MAX_CAPTURE_RATE)) {
attr->spropmaxcapturerate = val;
- } else if (!strcasecmp(name, "sprop_stereo")) {
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_MAX_PTIME)) {
+ attr->maxptime = val;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_PTIME)) {
+ attr->ptime = val;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_MAX_AVERAGE_BITRATE)) {
+ attr->maxbitrate = val;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_STEREO)) {
+ attr->stereo = val;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_SPROP_STEREO)) {
attr->spropstereo = val;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_CBR)) {
+ attr->cbr = val;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_FEC)) {
+ attr->fec = val;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_DTX)) {
+ attr->dtx = val;
} else {
ast_log(LOG_WARNING, "unknown attribute type %s\n", name);
}
return cloned;
+}
+
+static const void *opus_get(const struct ast_format *format, const char *name)
+{
+ struct opus_attr *attr = ast_format_get_attribute_data(format);
+ int *val = NULL;
+
+ if (!attr) {
+ return NULL;
+ }
+
+ if (!strcasecmp(name, CODEC_OPUS_ATTR_DATA)) {
+ return ao2_bump(attr->data);
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_MAX_PLAYBACK_RATE)) {
+ val = &attr->maxplayrate;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_SPROP_MAX_CAPTURE_RATE)) {
+ val = &attr->spropmaxcapturerate;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_MAX_PTIME)) {
+ val = &attr->maxptime;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_PTIME)) {
+ val = &attr->ptime;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_MAX_AVERAGE_BITRATE)) {
+ val = &attr->maxbitrate;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_STEREO)) {
+ val = &attr->stereo;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_SPROP_STEREO)) {
+ val = &attr->spropstereo;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_CBR)) {
+ val = &attr->cbr;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_FEC)) {
+ val = &attr->fec;
+ } else if (!strcasecmp(name, CODEC_OPUS_ATTR_DTX)) {
+ val = &attr->dtx;
+ } else {
+ ast_log(LOG_WARNING, "unknown attribute type %s\n", name);
+ }
+
+ return val;
}
static struct ast_format_interface opus_interface = {
@@ -342,11 +367,12 @@
.format_attribute_set = opus_set,
.format_parse_sdp_fmtp = opus_parse_sdp_fmtp,
.format_generate_sdp_fmtp = opus_generate_sdp_fmtp,
+ .format_attribute_get = opus_get
};
static int load_module(void)
{
- if (ast_format_interface_register("opus", &opus_interface)) {
+ if (__ast_format_interface_register("opus", &opus_interface, ast_module_info->self)) {
return AST_MODULE_LOAD_DECLINE;
}
@@ -358,9 +384,9 @@
return 0;
}
-AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_LOAD_ORDER, "Opus Format Attribute Module",
+AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_GLOBAL_SYMBOLS | AST_MODFLAG_LOAD_ORDER, "Opus Format Attribute Module",
.support_level = AST_MODULE_SUPPORT_CORE,
.load = load_module,
.unload = unload_module,
- .load_pri = AST_MODPRI_CHANNEL_DEPEND,
+ .load_pri = AST_MODPRI_REALTIME_DRIVER /* Needs to load before codec_opus */
);
--
To view, visit https://gerrit.asterisk.org/3988
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9f20e7cce00c32464d9a180e81283d49d199d0a3
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
More information about the asterisk-code-review
mailing list