Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-29 Thread Franklin Phillips
On Tue, Nov 29, 2016 at 07:10:52PM +0100, wm4 wrote: > On Tue, 29 Nov 2016 17:44:15 + > Franklin Phillips wrote: > > > Hi, > > > > If there are no more comments can we look at merging this please? > > I think your previous post implied that subtitles are actually timed > correctly, right? T

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-29 Thread wm4
On Tue, 29 Nov 2016 17:44:15 + Franklin Phillips wrote: > Hi, > > If there are no more comments can we look at merging this please? I think your previous post implied that subtitles are actually timed correctly, right? If so, I don't mind it being applied as-is. ___

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-29 Thread Franklin Phillips
Hi, If there are no more comments can we look at merging this please? Thanks ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-16 Thread Franklin Phillips
On Wed, Nov 16, 2016 at 12:33:32PM +0100, wm4 wrote: > On Wed, 16 Nov 2016 11:21:59 + > Franklin Phillips wrote: > > > On Wed, Nov 16, 2016 at 11:41:41AM +0100, wm4 wrote: > > > On Wed, 16 Nov 2016 10:31:10 + > > > Franklin Phillips wrote: > > > > > > > This patch is a fix for ticket

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-16 Thread Michael Niedermayer
On Tue, Nov 15, 2016 at 10:50:18AM +, Franklin Phillips wrote: > Hi, > > I am having trouble getting my patch to show up in > https://patchwork.ffmpeg.org/. This page > (https://www.ffmpeg.org/developer.html#Submitting-patches-1) suggests it > could be the mime type, however I used one of t

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-16 Thread wm4
On Wed, 16 Nov 2016 11:21:59 + Franklin Phillips wrote: > On Wed, Nov 16, 2016 at 11:41:41AM +0100, wm4 wrote: > > On Wed, 16 Nov 2016 10:31:10 + > > Franklin Phillips wrote: > > > > > This patch is a fix for ticket #2833. > > > > > > Each subtitle segment is its own WebVTT file and

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-16 Thread Franklin Phillips
On Wed, Nov 16, 2016 at 11:41:41AM +0100, wm4 wrote: > On Wed, 16 Nov 2016 10:31:10 + > Franklin Phillips wrote: > > > This patch is a fix for ticket #2833. > > > > Each subtitle segment is its own WebVTT file and has to demuxed > > individually. > > > The timing of the subtitles are not p

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-16 Thread Franklin Phillips
On Tue, Nov 15, 2016 at 09:56:39PM +0100, Henrik Gramner wrote: > On Tue, Nov 15, 2016 at 1:39 PM, Franklin Phillips > wrote: > > Sorry, I forgot to mention that my first attempt was using git send-email, > > when that didn't work, I tried mutt, same result using both clients. > > Might be spam-

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-16 Thread Franklin Phillips
On Tue, Nov 15, 2016 at 09:56:39PM +0100, Henrik Gramner wrote: > On Tue, Nov 15, 2016 at 1:39 PM, Franklin Phillips > wrote: > > Sorry, I forgot to mention that my first attempt was using git send-email, > > when that didn't work, I tried mutt, same result using both clients. > > Might be spam-

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-16 Thread wm4
On Wed, 16 Nov 2016 10:31:10 + Franklin Phillips wrote: > This patch is a fix for ticket #2833. > > Each subtitle segment is its own WebVTT file and has to demuxed individually. > The timing of the subtitles are not perfect but it is the best I could What does this mean? > do and it also

[FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-16 Thread Franklin Phillips
This patch is a fix for ticket #2833. Each subtitle segment is its own WebVTT file and has to demuxed individually. The timing of the subtitles are not perfect but it is the best I could do and it also does not take into account the X-TIMESTAMP-MAP header in the WebVTT files which is needed to

[FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-15 Thread Franklin Phillips
This patch is a fix for ticket #2833. Each subtitle segment is its own WebVTT file and has to demuxed individually. The timing of the subtitles are not perfect but it is the best I could do and it also does not take into account the X-TIMESTAMP-MAP header in the WebVTT files which is needed to

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-15 Thread Henrik Gramner
On Tue, Nov 15, 2016 at 1:39 PM, Franklin Phillips wrote: > Sorry, I forgot to mention that my first attempt was using git send-email, > when that didn't work, I tried mutt, same result using both clients. Might be spam-filter related. All your e-mails end up in my spam folder due to failing DMA

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-15 Thread Franklin Phillips
On Tue, Nov 15, 2016 at 10:01:01PM +0800, Steven Liu wrote: > 2016-11-15 20:39 GMT+08:00 Franklin Phillips : > > > On Tue, Nov 15, 2016 at 08:04:21PM +0800, Steven Liu wrote: > > > 2016-11-15 18:50 GMT+08:00 Franklin Phillips < > > franklinphillips...@inbox.lv>: > > > you can use git send-email, i

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-15 Thread 刘歧
Thanks 刘歧|CDN|研发总监 吉林省高升科技有限公司北京分公司 地址:北京市海淀区西三环北路87号国际财经中心B座9层 手机:18611075131 邮箱: li...@gosun.com 电话:010-82602628 > 在 2016年11月15日,下午6:50,Franklin Phillips 写道: > > Hi, > > I am having trouble getting my patch to show up in > https://patchwork.ffmpeg.org/. Thi

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-15 Thread Steven Liu
2016-11-15 20:39 GMT+08:00 Franklin Phillips : > On Tue, Nov 15, 2016 at 08:04:21PM +0800, Steven Liu wrote: > > 2016-11-15 18:50 GMT+08:00 Franklin Phillips < > franklinphillips...@inbox.lv>: > > you can use git send-email, it maybe better, i have the same problem > > before > Sorry, I forgot to

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-15 Thread Franklin Phillips
On Tue, Nov 15, 2016 at 08:04:21PM +0800, Steven Liu wrote: > 2016-11-15 18:50 GMT+08:00 Franklin Phillips : > you can use git send-email, it maybe better, i have the same problem > before Sorry, I forgot to mention that my first attempt was using git send-email, when that didn't work, I tried mut

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-15 Thread Steven Liu
2016-11-15 18:50 GMT+08:00 Franklin Phillips : > Hi, > > I am having trouble getting my patch to show up in > https://patchwork.ffmpeg.org/. This page (https://www.ffmpeg.org/ > developer.html#Submitting-patches-1) suggests it could be the mime type, > however I used one of the recommended types,

Re: [FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-15 Thread Franklin Phillips
Hi, I am having trouble getting my patch to show up in https://patchwork.ffmpeg.org/. This page (https://www.ffmpeg.org/developer.html#Submitting-patches-1) suggests it could be the mime type, however I used one of the recommended types, text/plain, and it is still not appearing on the patchwo

[FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-14 Thread Franklin Phillips
This patch is a fix for ticket #2833. Each subtitle segment is its own WebVTT file and has to demuxed individually. The timing of the subtitles are not perfect but it is the best I could do and it also does not take into account the X-TIMESTAMP-MAP header in the WebVTT files which is needed to

[FFmpeg-devel] [PATCH] avformat/hls: Added subtitle support

2016-11-08 Thread Franklin Phillips
This patch is a fix for ticket #2833. Each subtitle segment is its own WebVTT file and has to demuxed individually. The timing of the subtitles are not perfect but it is the best I could do and it also does not take into account the X-TIMESTAMP-MAP header in the WebVTT files which is needed to