On Wed, Sep 07, 2011 at 07:36:21PM -0400, Justin Ruggles wrote:
> Move shared tables to a separate file as well.
> ---
>  libavcodec/Makefile     |   40 ++--
>  libavcodec/adpcm.c      |  728 
> ++---------------------------------------------
>  libavcodec/adpcm.h      |   46 +++
>  libavcodec/adpcm_data.c |   78 +++++
>  libavcodec/adpcm_data.h |   37 +++
>  libavcodec/adpcmenc.c   |  654 ++++++++++++++++++++++++++++++++++++++++++
>  6 files changed, 855 insertions(+), 728 deletions(-)
>  create mode 100644 libavcodec/adpcm.h
>  create mode 100644 libavcodec/adpcm_data.c
>  create mode 100644 libavcodec/adpcm_data.h
>  create mode 100644 libavcodec/adpcmenc.c

OK

> --- /dev/null
> +++ b/libavcodec/adpcm.h
> @@ -0,0 +1,46 @@
> +
> +/**
> + * @file
> + * ADPCM encoder/decoder common header.

nit: please drop the period

> --- /dev/null
> +++ b/libavcodec/adpcm_data.c
> @@ -0,0 +1,78 @@
> +
> +/**
> + * @file
> + * ADPCM tables.

ditto

> --- /dev/null
> +++ b/libavcodec/adpcm_data.h
> @@ -0,0 +1,37 @@
> +
> +/**
> + * @file
> + * ADPCM tables.

same

> --- /dev/null
> +++ b/libavcodec/adpcmenc.c
> @@ -0,0 +1,654 @@
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with Libav; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 
> USA
> + */
> +#include "avcodec.h"
> +#include "get_bits.h"
> +#include "put_bits.h"
> +#include "bytestream.h"
> +#include "adpcm.h"
> +#include "adpcm_data.h"

nit: empty line between license boilerplate and #includes.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to