[FFmpeg-devel] [PATCH]Support linebreaks in onCaption subtitles

Clément Bœsch u at pkh.me
Mon May 4 11:48:04 CEST 2015


On Mon, May 04, 2015 at 09:41:35AM +0000, Carl Eugen Hoyos wrote:
> Carl Eugen Hoyos <cehoyos <at> ag.or.at> writes:
> 
> > Clément Bœsch <u <at> pkh.me> writes:
> > 
> > > > +    text->linebreaks = "<br/>";
> > > 
> > > i'm afraid this won't be enough to support 
> > > "<br>", "<BR>", "<br />", ...
> > 
> > I only looked at the sample we have, are they 
> > all valid line breaks?
> 
> Ping, I don't understand your comment.
> 

There is nothing to understand: is it explicit in a specs somewhere that
the only markup is a line return expressed by the specific string "<br/>",
or it's just another HTML-based markup, where "<br>", "<BR>", "<br />" can
happen?

You are the one implementing this, don't ask me about the details. I'm
just wondering about the amount of work it will require to remove that
code before implementing it correctly if necessary.

The main point being that I'm wondering how we are going to explain to our
users that we support subtitles with "<br/>", but supporting "<br />" will
require large changes.

Note: there is a work in progress on the mailing list about factoring out
the HTML code from codecs, which includes the support of "br" based tags;
it might make more sense then to change the codec to SubRip (because it
will support that tag) instead of adding a half-assed codec that will be
deprecated soon.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150504/720e6e3f/attachment.asc>


More information about the ffmpeg-devel mailing list