[FFmpeg-devel] [PATCH 2/3] textdec: Rename all generic parts from srt to text.

Nicolas George nicolas.george at normalesup.org
Thu Aug 2 12:12:55 CEST 2012


Le sextidi 16 thermidor, an CCXX, Clément Bœsch a écrit :
> That tag soup really is SubRip specific; WebVTT for example¹ defines
> another taste for the soup, so I think it would be better to have
> CODEC_ID_SUBRIP.

The difference is that WebVTT clearly defines the accepted syntax, while
SubRip does not.

> and ASS-tags? huh?

That is what we discussed a few hours ago: some file have them. And that is
precisely why I think "TAGSOUP" is more descriptive than SUBRIP: this is an
unholy mix of various standards in the hope that bogus players will display
them as expected.

> > - newlines marked by CR?LF;
> some SRT files might only have \n, both should be supported.

That is exactly what the '?' in the pseudo-regexp stands for.

> > - heading and trailing newlines (spaces) must be ignored.
> Is that different from the current state?

Probably not, but now it is specified.

Note that I expect the decoded text (as opposed to the demuxed text) to have
just LF newlines and no heading/trailing spaces.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120802/2842f45f/attachment.asc>


More information about the ffmpeg-devel mailing list