On Mon, Dec 26, 2016 at 06:21:18PM -0800, Thomas Turner wrote: > Signed-off-by: Thomas Turner <[email protected]> > --- > libavutil/tests/audio_fifo.c | 181 > ++++++++++++++++++++++++++++--------------- > tests/ref/fate/audio_fifo | 34 +++----- > 2 files changed, 130 insertions(+), 85 deletions(-)
The commit message is a bit terse, please describe what/why things
are changed
[...]
> +static void ERROR(const char *str, AVAudioFifo *afifo, Audio_Data *adata)
> {
> - int p, b, f;
> - int byte_offset = av_get_bytes_per_sample(test_sample->format);
> - int buffers = av_sample_fmt_is_planar(test_sample->format)
> - ? test_sample->nb_ch : 1;
> - int line_size = (buffers > 1) ? nb_samples * byte_offset
> - : nb_samples * byte_offset *
> test_sample->nb_ch;
> - for (p = 0; p < buffers; ++p){
> - for(b = 0; b < line_size; b+=byte_offset){
> - for (f = 0; f < byte_offset; f++){
> - int order = !HAVE_BIGENDIAN ? (byte_offset - f - 1) : f;
> - printf("%02x", *((uint8_t*)data_planes[p] + b + order));
> + fprintf(stderr, "%s\n", str);
> +
> + if (afifo)
> + av_audio_fifo_free(afifo);
> + if (adata)
> + free_audio(adata);
> +
> + exit(1);
> +}
the indention depth seems inconsistent compared to the other functions
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
