Changeset 3661 for pjproject/branches/projects/2.0-dev/pjmedia/src/pjmedia/codec.c

Timestamp:

Jul 15, 2011 9:51:46 AM (13 years ago)

Author:

bennylp

Message:

Fixed #1294: New pjmedia_codec_register_audio_codecs() API to register all known codecs, and also added implementation to deregister all codecs when the media endpoint is destroyed

File:

• pjproject/branches/projects/2.0-dev/pjmedia/src/pjmedia/codec.c (modified) (2 diffs)

pjproject/branches/projects/2.0-dev/pjmedia/src/pjmedia/codec.c

@@ -73 +73 @@
 PJ_DEF(pj_status_t) pjmedia_codec_mgr_destroy (pjmedia_codec_mgr *mgr)
 {
+    pjmedia_codec_factory *factory;
     unsigned i;
 
     PJ_ASSERT_RETURN(mgr, PJ_EINVAL);
+
+    /* Destroy all factories in the list */
+    factory = mgr->factory_list.next;
+    while (factory != &mgr->factory_list) {
+        pjmedia_codec_factory *next = factory->next;
+        (*factory->op->destroy)();
+        factory = next;
+    }
 
     /* Cleanup all pools of all codec default params */
@@ -111 +120 @@
 
     PJ_ASSERT_RETURN(mgr && factory, PJ_EINVAL);
+
+    /* Since 2.0 we require codec factory to implement "destroy" op. Please
+     * see: https://trac.pjsip.org/repos/ticket/1294
+     *
+     * Really! Please do see it.
+     */
+    PJ_ASSERT_RETURN(factory->op->destroy != NULL, PJ_ENOTSUP);
 
     /* Enum codecs */