On Mon, 3 Jul 2017 17:57:06 +0200
Michael Niedermayer <mich...@niedermayer.cc> wrote:
> On Mon, Jul 03, 2017 at 01:54:50PM +0200, Clément Bœsch wrote:
> > On Mon, Jul 03, 2017 at 01:43:43PM +0200, wm4 wrote:
> > > Some .srt files use this tag.
> > >
> > > (An alternative implementation would be correctly ignoring unknown tags,
> > > and treating them as whitespace. libass can do automatic line wrapping.)
> > > ---
> > > libavcodec/htmlsubtitles.c | 2 ++
> > > 1 file changed, 2 insertions(+)
> > >
> > > diff --git a/libavcodec/htmlsubtitles.c b/libavcodec/htmlsubtitles.c
> > > index be5c9316ca..fe991678d5 100644
> > > --- a/libavcodec/htmlsubtitles.c
> > > +++ b/libavcodec/htmlsubtitles.c
> > > @@ -167,6 +167,8 @@ int ff_htmlmarkup_to_ass(void *log_ctx, AVBPrint
> > > *dst, const char *in)
> > > }
> > > } else if (tagname[0] && !tagname[1] &&
> > > strspn(tagname, "bisu") == 1) {
> > > av_bprintf(dst, "{\\%c%d}", tagname[0],
> > > !tag_close);
> > > + } else if (!strcmp(tagname, "br")) {
> > > + av_bprintf(dst, "\\N");
> > > } else {
> > > unknown = 1;
> > > snprintf(tmp, sizeof(tmp), "</%s>", tagname);
> >
> > So this supports <br>, < br > and <br/>?
>
> i dont think this supports <br/> from a quick look
Tried < br /> and < br / >, both turn out correctly. Pushed.
Unfortunately I just noticed <br/> does not work. Well, whatever.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
