[FFmpeg-devel] [PATCH v4] libavcodec: Add FLAC API test

[James Almer]

On 19/04/15 9:03 PM, Ludmila Glinskih wrote:

> +static int run_test(AVCodec *enc, AVCodec *dec, AVCodecContext *enc_ctx,
> +                    AVCodecContext *dec_ctx)
> +{
> +    AVPacket enc_pkt;
> +    AVFrame *in_frame, *out_frame;
> +    uint8_t *raw_in = NULL, *raw_out = NULL;
> +    int in_offset = 0, out_offset = 0;
> +    int frame_data_size = 0;

This

[...]

> +    if (memcmp(raw_in, raw_out, frame_data_size * NUMBER_OF_FRAMES) != 0)

And this are the only lines where frame_data_size is used. This means it's always 0.
Is that intended? Because unless i'm missing something, that doesn't seem right for 
a memcmp.