Re: [FFmpeg-devel] [PATCH] avcodec/samidec: fix misposition of subtitles caused by rounding

2014-11-30 Thread Jeong-Hoon Seo
Thanks for your point of view. I'll fix properly and submit later. 2014-11-27 5:53 GMT+09:00 Clément Bœsch u...@pkh.me: On Tue, Nov 25, 2014 at 06:07:30PM +0900, Jeong-Hoon Seo wrote: SAMI and ASS have different precision of duration in FFMPEG (SAMI: millisecond, ASS: centisecond)

Re: [FFmpeg-devel] [PATCH] avcodec/samidec: fix misposition of subtitles caused by rounding

2014-11-26 Thread Clément Bœsch
On Tue, Nov 25, 2014 at 06:07:30PM +0900, Jeong-Hoon Seo wrote: SAMI and ASS have different precision of duration in FFMPEG (SAMI: millisecond, ASS: centisecond) Sometimes, start-time of current subtitle is overrun with end-time of previous one by rounding process This led to misposition of

[FFmpeg-devel] [PATCH] avcodec/samidec: fix misposition of subtitles caused by rounding

2014-11-25 Thread Jeong-Hoon Seo
SAMI and ASS have different precision of duration in FFMPEG (SAMI: millisecond, ASS: centisecond) Sometimes, start-time of current subtitle is overrun with end-time of previous one by rounding process This led to misposition of subtitles exmple - input SAMI sync start=1155 p class=ENCC