[FFmpeg-devel] [PATCH] avutil: Added selftest for libavutil/audio_fifo.c

[Michael Niedermayer]

On Mon, Dec 19, 2016 at 06:39:03PM -0800, Thomas Turner wrote:
> Signed-off-by: Thomas Turner <thomastdt at googlemail.com>
> ---
>  libavutil/Makefile           |   1 +
>  libavutil/tests/audio_fifo.c | 195 ++++++++++++++++++++++++++++++++++++
>  tests/fate/libavutil.mak     |   4 +
>  tests/ref/fate/audio_fifo    | 228 +++++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 428 insertions(+)
>  create mode 100644 libavutil/tests/audio_fifo.c
>  create mode 100644 tests/ref/fate/audio_fifo
> 
> diff --git a/libavutil/Makefile b/libavutil/Makefile
> index 9841645..2dd91b8 100644
> --- a/libavutil/Makefile
> +++ b/libavutil/Makefile
> @@ -182,6 +182,7 @@ SKIPHEADERS-$(CONFIG_OPENCL)           += opencl.h
>  TESTPROGS = adler32                                                     \
>              aes                                                         \
>              atomic                                                      \
> +            audio_fifo                                                  \
>              avstring                                                    \
>              base64                                                      \
>              blowfish                                                    \
> diff --git a/libavutil/tests/audio_fifo.c b/libavutil/tests/audio_fifo.c
> new file mode 100644
> index 0000000..7e166b1
> --- /dev/null
> +++ b/libavutil/tests/audio_fifo.c
> @@ -0,0 +1,195 @@
> +/*
> + * This file is part of FFmpeg.
> + *
> + * FFmpeg is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU Lesser General Public
> + * License as published by the Free Software Foundation; either
> + * version 2.1 of the License, or (at your option) any later version.
> + *
> + * FFmpeg is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> + * Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with FFmpeg; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> + */
> +
> +#include <stdlib.h>
> +#include <stdio.h>
> +#include <inttypes.h>
> +#include "libavutil/audio_fifo.c"
> +
> +#define MAX_CHANNELS    32
> +

> +#define ERROR(str)                    \
> +        fprintf(stderr, "%s\n", str); \
> +        exit(1);

this should use do{}while; like other multiline macros
see for example:
tests/tiny_ssim.c:#define FFSWAP(type,a,b) do{type SWAP_tmp= b; b= a; a= SWAP_tmp;}while(0)

or it should use a function instead of a macro
a function is probably a better idea anyway


> +
> +typedef struct TestStruct {
> +    const enum AVSampleFormat format;
> +    const int nb_ch;
> +    void const *data_planes[MAX_CHANNELS];
> +    int nb_samples_pch;
> +} TestStruct;
> +
> +static const uint8_t data_U8 [] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11                        };
> +static const int16_t data_S16[] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11                        };
> +static const float   data_FLT[] = {0.0, 1.0, 2.0, 3.0, 4.0, 5.0, 6.0, 7.0, 8.0, 9.0, 10.0, 11.0};
> +
> +
> +static const TestStruct test_struct[] = {
> +    {.format = AV_SAMPLE_FMT_U8   , .nb_ch = 1, .data_planes = {data_U8 ,             }, .nb_samples_pch = 12},
> +    {.format = AV_SAMPLE_FMT_U8P  , .nb_ch = 2, .data_planes = {data_U8 , data_U8 +6, }, .nb_samples_pch = 6 },
> +    {.format = AV_SAMPLE_FMT_S16  , .nb_ch = 1, .data_planes = {data_S16,             }, .nb_samples_pch = 12},
> +    {.format = AV_SAMPLE_FMT_S16P , .nb_ch = 2, .data_planes = {data_S16, data_S16+6, }, .nb_samples_pch = 6 },
> +    {.format = AV_SAMPLE_FMT_FLT  , .nb_ch = 1, .data_planes = {data_FLT,             }, .nb_samples_pch = 12},
> +    {.format = AV_SAMPLE_FMT_FLTP , .nb_ch = 2, .data_planes = {data_FLT, data_FLT+6, }, .nb_samples_pch = 6 }
> +};
> +

> +static void* allocate_memory(size_t size)
> +{
> +    void *ptr = malloc(size);
> +    if (ptr == NULL){
> +        fprintf(stderr, "failed to allocate memory!\n");
> +        exit(1);

this could use ERROR() too

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161220/22646ac0/attachment.sig>