On 05/09/2012 10:59 AM, Anton Khirnov wrote:
> +    int bps       = av_get_bytes_per_sample(sample_fmt);
> +    int data_size = bps * nb_samples * (planar ? 1 : nb_channels);
> +    int fill_char = (sample_fmt == AV_SAMPLE_FMT_U8 ||
> +                     sample_fmt == AV_SAMPLE_FMT_U8P) ? 0x80 : 0x00;
> +    int i;
> +
> +    offset *= bps * (planar ? 1 : nb_channels);

one small sugestion:

int block_align = av_get_bytes_per_sample(sample_fmt) * (planar ? 1 :
                                                         nb_channels);
int data_size   = nb_samples * block_align;
...
offset *= block_align;


the rest LGTM.

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

Reply via email to