[FFmpeg-devel] [PATCH 2/2] avformat/hls: Add subtitle support
Anssi Hannula
anssi.hannula at iki.fi
Mon Dec 12 18:05:06 EET 2016
Hi,
07.12.2016, 00:04, Franklin Phillips kirjoitti:
> Assuming the reason why my patch wasn't being merged was because it
> didn't use the X-TIMESTAMP-MAP, I have included the changes for that.
>
> Those changes were basically a merge of work done by
> anssi.hannula at iki.fi which is why I've cc'd them.
I'm sorry about the lack of response.
I should be able to take a closer look by Friday this week, but with a
quick look your read_packet_subtitle() seems to contain quite a lot of
duplicated code with other parts of hls.c (unless I'm missing
something), so some refactoring should be done.
--
Anssi Hannula
More information about the ffmpeg-devel
mailing list