[asterisk-dev] [Code Review] Remove MP3 decoder source code from main Asterisk source tree

Kevin Fleming kpfleming at digium.com
Tue Aug 3 13:42:29 CDT 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/836/#review2520
-----------------------------------------------------------

Ship it!


I suppose some people will complain about having to have Subversion installed to get the source... but we can deal with that if it arises.


/branches/1.8/addons/Makefile
<https://reviewboard.asterisk.org/r/836/#comment5547>

    It's just a bit safer to use 'filter' here instead of 'findstring', since filter only looks at whole words.


- Kevin


On 2010-08-03 13:35:41, Russell Bryant wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/836/
> -----------------------------------------------------------
> 
> (Updated 2010-08-03 13:35:41)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This change was inspired by the following thread on the asterisk-dev mailing list:
> 
> "Asterisk 1.8 and MP3 code?"
> http://lists.digium.com/pipermail/asterisk-dev/2010-August/045532.html
> 
> This patch removes the MP3 decoder source code from the Asterisk tree.  The wrapper, format_mp3.c, remains.  However, to build it, a script must be run that downloads the source code from another location.  Also, if format_mp3 is enabled, but the source has not been downloaded, the Makefile will print out instructions on how to go get it.
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/addons/Makefile 280737 
>   /branches/1.8/addons/mp3/MPGLIB_README 280737 
>   /branches/1.8/addons/mp3/MPGLIB_TODO 280737 
>   /branches/1.8/addons/mp3/Makefile 280737 
>   /branches/1.8/addons/mp3/README 280737 
>   /branches/1.8/addons/mp3/common.c 280737 
>   /branches/1.8/addons/mp3/dct64_i386.c 280737 
>   /branches/1.8/addons/mp3/decode_i386.c 280737 
>   /branches/1.8/addons/mp3/decode_ntom.c 280737 
>   /branches/1.8/addons/mp3/huffman.h 280737 
>   /branches/1.8/addons/mp3/interface.c 280737 
>   /branches/1.8/addons/mp3/layer3.c 280737 
>   /branches/1.8/addons/mp3/mpg123.h 280737 
>   /branches/1.8/addons/mp3/mpglib.h 280737 
>   /branches/1.8/addons/mp3/tabinit.c 280737 
>   /branches/1.8/contrib/scripts/get_mp3_source.sh PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/836/diff
> 
> 
> Testing
> -------
> 
> Did a test build before and after downloading the mp3 source into the tree.
> 
> 
> Thanks,
> 
> Russell
> 
>




More information about the asterisk-dev mailing list