[FFmpeg-devel] [PATCH] ffmpeg: insert bitmap subtitles as video in filters.
nicolas.george at normalesup.org
Wed Aug 1 16:04:37 CEST 2012
Le quintidi 15 thermidor, an CCXX, Stefano Sabatini a écrit :
> Maybe mention what the funny labels stand for.
> Comment this block with something along the line:
> |Compute the size for the video subtitles stream. This will be the
> |subtitles stream size if defined, otherwise the minimum size which
> |will contain all the video streams specified in the corresponding
> |input file.
> or a shortened version.
> Maybe you could complain and write only in the overlapping region.
I do not have a sample. I could probably fabricate one (although bitmap
subtitles are currently fragile and hard tu mux). But that would make more
complex code for possibly no real-world reason at all. I'll implement it if
someone has the problem and a sample.
> What's the implication of AV_BUFFERSRC_FLAG_NO_CHECK_FORMAT? (I mean
> can it happens that the size of the buffer changes, in that case what
> happens of the filtergraph?, can it cope with it?).
Currently, the size of the buffer can not change.
> nit: "inf" reminds "info", what about "infile"?
> nit: nb_reqs since it is plural
> Please can you explain what this does?
> s->streams[i]->codec->codec_type could be factored.
Thanks for the review, updated patch in a few minutes.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel