[FFmpeg-devel] [PATCH 2/4] tty: disable extension matching

Reimar Döffinger Reimar.Doeffinger at gmx.de
Thu Aug 8 21:49:36 CEST 2013


Michael Niedermayer <michaelni at gmx.at> wrote:
>On Thu, Aug 08, 2013 at 03:17:04AM +0200, wm4 wrote:
>> On Thu, 8 Aug 2013 03:06:05 +0200
>> Michael Niedermayer <michaelni at gmx.at> wrote:
>> 
>> > On Thu, Aug 08, 2013 at 01:04:37AM +0200, wm4 wrote:
>> > > This demuxer isn't very useful and causes more trouble than it
>solves.
>> > > In psrticular, it matches .txt files too eagerly. It should be
>left to
>> > > the application what to do with text files, and ana pplication
>shouldn't
>> > > be forced to blacklist libavformat fringe demuxers.
>> > 
>> > Can you elaborate on the problem ?
>> > what does the application do with .txt files if they are not sucked
>> > up by tty ?
>> 
>> In my case, this prevented a fallback to an old builtin MPL2 subtitle
>> parser to fail. I can also see other things that are stored in text
>> files and which the application might want to handle itself, like
>> playlists.
>> 
>
>> On the other hand, I see no reasonable argument why libavcodec
>> should render text files as video. There are much better ways to view
>> text files.
>
>iam not sure about the lack of use cases
>anyone else has any oppinons ?

Shouldn't it be possible and more flexible if the application just ignored any probe results below a certain score to avoid this issue?




More information about the ffmpeg-devel mailing list