[FFmpeg-devel] BUG in use of extradata and extradata_size with dvb subtitles and teletext
u at pkh.me
Sat Jan 11 12:10:54 CET 2014
On Fri, Jan 10, 2014 at 04:15:48PM +0200, Andriy Lysnevych wrote:
> On Fri, Jan 10, 2014 at 3:30 PM, Clément Bœsch <u at pkh.me> wrote:
> > On Fri, Jan 10, 2014 at 01:16:02PM +0000, Carl Eugen Hoyos wrote:
> >> Andriy Lysnevych <andriy.lysnevych <at> gmail.com> writes:
> >> > avcodec and avformat minor versions bumped, patch attached.
> >> If this patch fixes ticket #2024 (or a part of it), please
> >> mention it in the commit message.
> > ...and with the style nit eventually.
> Don't know exactly what "style nit" is, but I did few coding style changes.
> If "style nit" still exists please give more details so I can fix it.
style nitpicks, not exactly important, but appreciated if you honor them.
You didn't honor the small comments in
But really, that's not important, I've made the changes locally and will
redo if necessary before pushing.
> The patch fixes ticket #2024 partly because it addresses two different problems:
> 1) First two samples end with "av_interleaved_write_frame(): Invalid
> argument". This problem is not fixed.
> 2) Last one: "Remuxing only the subtitles work, but the resulting file
> shows no subtitles". This one is fixed.
After testing, it does indeed fix transcoding, but it doesn't fix
remuxing. For instance, this makes a valid output:
ffmpeg -i Starship_Troopers.vob -map 0 -c copy -c:s dvbsub out.ts
but this doesn't:
ffmpeg -i Starship_Troopers.vob -map 0 -c copy out.ts
Can you confirm this?
If so, I'll adjust the commit message and push, unless someone has an
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 490 bytes
Desc: not available
More information about the ffmpeg-devel