#2440 Inclusion of opus codec into main CSipSimple package

[Anonymous]

Originally created by: shtinkov

This issue is not a deffect. It is a proposal to move opus codec to package CSipSimple.
I tested it a few months and CSipSimple worked well with opus included into it.
In fact opus codec is open-source just like CSipSimple and both softwares follows the same philosophy, so such inclusion will not create any new issue.

So if you agree with me to move codec here is patch in order to do that.

Best Regards!

--- ../CSipSimple/dispatch_shared_libs.sh       2013-06-13 11:50:02.416825649 +0300
+++ CSipSimple/dispatch_shared_libs.sh  2013-06-07 12:03:21.329643035 +0300
@@ -22,7 +22,8 @@

move_lib "g7221" "Pack"
move_lib "codec2" "Pack"
-move_lib "opus" "Pack"
+#stirlitz
+#move_lib "opus" "Pack"
move_lib "g726" "Pack"
move_lib "aac" "Pack"
move_lib "g729" "G729"

--- ../CSipSimple/AndroidManifest.xml   2013-06-13 11:50:02.405825786 +0300
+++ CSipSimple/AndroidManifest.xml      2013-06-11 11:40:37.159102952 +0300
@@ -545,7 +545,19 @@

         <!-- OPUS? -->
+               <!-- stirlitz receiver got from other package -->
+        <receiver android:name="com.csipsimple.plugins.codecs.ReceiverOpus" >
+            <meta-data
+                android:name="lib_name"
+                android:value="libpj_opus_codec.so" />
+            <meta-data
+                android:name="init_factory"
+                android:value="pjmedia_codec_opus_init" />

+            <intent-filter>
+                <action android:name="com.csipsimple.codecs.action.REGISTER_CODEC" />
+            </intent-filter>
+        </receiver>

         <!-- Extra wizard creation account activities -->
         <activity
@@ -553,4 +565,4 @@
             android:theme="@style/DarkTheme" />
     </application>

-</manifest>
\ No newline at end of file
+</manifest>

[Anonymous]

Originally posted by: r3gis...@gmail.com

From tests I did it's not yet ready to be included as 'stable' codec. There is known problems in the integration that make me decide to not include it yet for everyone.
But, indeed, as soon as will become as stable as it should be, it will be included in trunk.

Status: Invalid