Re: [FFmpeg-devel] [PATCH 2/2] avcodec/srtdec: attempt to correct SubRip positioning

2015-05-14 Thread Clément Bœsch
On Sun, Feb 08, 2015 at 12:42:11PM +0100, Clément Bœsch wrote: > On Sun, Feb 08, 2015 at 12:40:19PM +0100, Clément Bœsch wrote: > > The positioning was completely wrong. First, the coordinates are > > expressed in ASS playback resolution (which is by default 384x288). > > Secondly, the coordinates

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/srtdec: attempt to correct SubRip positioning

2015-02-08 Thread Clément Bœsch
On Sun, Feb 08, 2015 at 12:40:19PM +0100, Clément Bœsch wrote: > The positioning was completely wrong. First, the coordinates are > expressed in ASS playback resolution (which is by default 384x288). > Secondly, the coordinates define a drawing rectangle, not a moving area. > The previous code was

[FFmpeg-devel] [PATCH 2/2] avcodec/srtdec: attempt to correct SubRip positioning

2015-02-08 Thread Clément Bœsch
The positioning was completely wrong. First, the coordinates are expressed in ASS playback resolution (which is by default 384x288). Secondly, the coordinates define a drawing rectangle, not a moving area. The previous code was making subtitles move from a random position to another random position