On Wed, Sep 03, 2014 at 02:37:48AM +0100, Kieran Kunhya wrote:
> On 3 September 2014 01:40, Michael Niedermayer <michae...@gmx.at> wrote:
> > On Wed, Sep 03, 2014 at 12:59:53AM +0100, Kieran Kunhya wrote:
> >> Patch not sent to ML so couldn't review:
> >>
> >> Why have you only changed the code to favour DESC_types and not other 
> >> types?
> >
> > because thats what was needed to fix Ticket3766, which was what the
> > commit tried to do.
> >
> > We might want to change others too, ideally would be if we had actual
> > testcases for them that need it. So the changes can be tested.
> >
> > Do you have some testcases ?
> 
> In general you should not be probing if there are descriptors because
> the descriptors are designed to make the codec detection unambiguous.

yes


> 
> How do testcases for stream detection even work in FATE?

see tests/fate/probe.mak

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to