[FFmpeg-devel] MPEG TS encoding problem on change from stereo to mono, pay for fix

Stefano Sabatini stefano.sabatini-lala
Sat Nov 27 16:54:40 CET 2010


On date Wednesday 2010-11-24 00:10:47 +0100, Stefano Sabatini encoded:
> On date Tuesday 2010-11-23 08:19:08 +0100, Reimar D?ffinger encoded:
> > On Mon, Nov 22, 2010 at 10:27:37AM +0100, Stefano Sabatini wrote:
> > > -    if (ost->audio_resample && !ost->resample) {
> > > +    resample_changed = ost->resample_sample_fmt  != dec->sample_fmt ||
> > > +                       ost->resample_channels    != dec->channels   ||
> > > +                       ost->resample_sample_rate != dec->sample_rate;
> > > +
> > > +    if (ost->audio_resample && !ost->resample || resample_changed) {
> > 
> > Doesn't gcc complain with a warning that it wants some extra () here?
> > Also I think this is not quite right, at the very least you'll keep
> > a resampling context around even if input and output are the same now
> > if they weren't before. I'm not sure if it will actually be used, but
> > I think it will, too.
> 
> See your point, updated.

> -- 
> FFmpeg = Frightening & Fantastic Mythic Puristic Easy Gangster

> From 0bcec33a96f6398d1554ec2199e44b824a1e5bc0 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> Date: Mon, 22 Nov 2010 01:42:11 +0100
> Subject: [PATCH] Fix encoding when the input audio format/rate/channels changes during
>  transcoding.
> 
> Fix issue #2292.
> ---
>  ffmpeg.c |   32 ++++++++++++++++++++++++++++++--
>  1 files changed, 30 insertions(+), 2 deletions(-)

Ping.
-- 
FFmpeg = Fundamentalist Forgiving Moronic Prodigious Elastic Guru



More information about the ffmpeg-devel mailing list