[FFmpeg-devel] [ffmpeg-devel] [PATCH 1/4] lavfi: add asrc_abuffer - audio source buffer filter

Mina Nagy Zaki mnzaki at gmail.com
Mon Aug 1 10:59:33 CEST 2011


On Wed, Jul 27, 2011 at 11:56:32AM +0200, Stefano Sabatini wrote:
> On date Wednesday 2011-07-27 01:05:17 +0300, Mina Nagy Zaki encoded:
[...]

I rewrote the docs.

[...]
> What about:
> int av_asrc_buffer_add_audio_buffer_ref(AVFilterContext *abuffer_src,
>                                         AVFilterBufferRef *samplesref, int flags);
> ?
> 
> consistent with vsrc_buffer.h, more API-robust.

Added this to API, and kept others too such that we have entry points on
different 'levels'

[...]
> > +static av_cold int init(AVFilterContext *ctx, const char *args, void *opaque)
> > +{
> > +    ABufferSourceContext *abuffer = ctx->priv;
> > +    char sample_fmt_str[8], chlayout_str[16], sample_rate_str[16];
> 
> 16 is not enough for the layout.

Made it 32, which I think is enough.

[...]

Fixed everything else. Kept API as discussed.

Also, added input normalization, and tested a bit. But normalizing samplerate
has been a problem due to the codec picking up the sample rate of the very last
block on the first block. I can implement a workaround for this, but I think it
would be better to fix the codec.



More information about the ffmpeg-devel mailing list