[FFmpeg-devel] [PATCH] lavf/concatdec: remove unrelated change during codecpar merge.

Andrey Utkin andrey_utkin at fastmail.com
Mon Apr 25 12:13:18 CEST 2016


On Mon, Apr 25, 2016 at 08:58:40AM +0200, Nicolas George wrote:
> Le sextidi 6 floréal, an CCXXIV, Andrey Utkin a écrit :
> > Please look at my email in ffmpeg-devel
> > '[PATCH] concatdec: Fix handling of H.264 in MP4 in case of "-auto_convert 0"'
> > it adds a check which allows to correctly handle both cases.
> 
> I will be able to look at this patch when it arrives, not before.

Sorry, it still awaits moderator approval (sent from non-subscribed
address) since yesterday.
I think I shouldn't resubmit but instead wait for original letter to
show up.

> > The chunk which Nicolas proposes to drop makes good work for default
> > (auto_convert=1) case and I wouldn't say that it shouldn't be there,
> > that code just doesn't consider auto_convert=0 case.
> 
> I am not entirely convinced this is quite everything.

Ok, anyway your patch fixes my usecase, too.


More information about the ffmpeg-devel mailing list