[FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

wm4 nfxjfg at googlemail.com
Sun Aug 31 14:57:54 CEST 2014

On Sun, 31 Aug 2014 14:25:21 +0200
Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:

> On Sun, Aug 31, 2014 at 01:04:29PM +0200, wm4 wrote:
> > +static int sup_probe(AVProbeData *p)
> > +{
> > +    if (p->buf_size < 2 || memcmp(p->buf, "PG", 2))
> > +        return 0;
> I understand if you consider it not worth the effort,
> but ideally this would scan ahead several packets,
> and if it all matches up return a higher score.
> The MP3 probe function is possibly a good reference (though
> this one should be bit simpler).

Other formats are also relatively lazy and just check the magic atthe
start of the file and call it a day (e.g. flac, some img2dec probers).
Of course it's possible that 2 bytes (and ASCII) is a bit too prone to
false positives, so maybe it should be improved.

Since PGS packets can be only at most ~64KB big, I guess it would be
feasible to check whether there is a second PGS packet after the first
one. Would that be sufficient?

In theory, it would be nice if the general probe code could jzst try to
read a few packets...

More information about the ffmpeg-devel mailing list