[FFmpeg-devel] [PATCH] samplefmt: change syntax for av_samples_alloc() and av_samples_fill_arrays()

[Reimar Döffinger]

On Mon, May 23, 2011 at 12:14:19AM +0200, Stefano Sabatini wrote:
> ---
>  libavutil/samplefmt.c |   34 ++++++++++++++++++----------------
>  libavutil/samplefmt.h |   26 ++++++++++++++++----------
>  2 files changed, 34 insertions(+), 26 deletions(-)
> 
> diff --git a/libavutil/samplefmt.c b/libavutil/samplefmt.c
> index ea004d9..346e44e 100644
> --- a/libavutil/samplefmt.c
> +++ b/libavutil/samplefmt.c
> @@ -73,31 +73,29 @@ int av_get_bits_per_sample_fmt(enum AVSampleFormat sample_fmt)
>  }
>  
>  int av_samples_fill_arrays(uint8_t *pointers[8], int linesizes[8],
> -                           uint8_t *buf, int nb_channels, int nb_samples,
> -                           enum AVSampleFormat sample_fmt, int planar, int align)
> +                           uint8_t *buf, int linesize0, int nb_channels,
> +                           enum AVSampleFormat sample_fmt, int planar)

The name indicates that av_samples_fill_arrays is a public function, and this
would then be an API/ABI change...
> @@ -97,7 +103,7 @@ int av_samples_fill_arrays(uint8_t *pointers[8], int linesizes[8],
>   * av_freep(&pointers[0]).
>   *
>   * @param nb_samples number of samples per channel
> - * @param planar 1 if the samples layout is planar, 0 if packed,
> + * @param planar >0 if the samples layout is planar, packed otherwise
>   * @param align the value to use for buffer size alignment
>   * @return the size in bytes required for the samples buffer, a negative
>   * error code in case of failure
> -- 
> 1.7.2.3
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel