Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: use display aspect ratio for DisplayWidth and DisplayHeight when possible

2016-11-02 Thread Kieran O Leary
Hi On 2 Nov 2016 10:15 p.m., "Carl Eugen Hoyos" wrote: > > > > > This fixes ticket #5743, implementing the suggestion from ticket #5903. > > Thank you for fixing this! > > Carl Eugen > +1 Thanks James! ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: use display aspect ratio for DisplayWidth and DisplayHeight when possible

2016-11-02 Thread Carl Eugen Hoyos
2016-10-22 22:33 GMT+02:00 James Almer : > This avoids potential rounding errors and guarantees the source aspect > ratio is preserved. > Keep writing pixel values when Stereo 3D Mode is enabled and for WebM, > as the format doesn't support anything else. > > This fixes ticket #5743, implementing t

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: use display aspect ratio for DisplayWidth and DisplayHeight when possible

2016-11-02 Thread James Almer
On 10/28/2016 12:03 AM, James Almer wrote: > On 10/22/2016 5:33 PM, James Almer wrote: >> This avoids potential rounding errors and guarantees the source aspect >> ratio is preserved. >> Keep writing pixel values when Stereo 3D Mode is enabled and for WebM, >> as the format doesn't support anything

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: use display aspect ratio for DisplayWidth and DisplayHeight when possible

2016-10-27 Thread James Almer
On 10/22/2016 5:33 PM, James Almer wrote: > This avoids potential rounding errors and guarantees the source aspect > ratio is preserved. > Keep writing pixel values when Stereo 3D Mode is enabled and for WebM, > as the format doesn't support anything else. > > This fixes ticket #5743, implementing

[FFmpeg-devel] [PATCH] avformat/matroskaenc: use display aspect ratio for DisplayWidth and DisplayHeight when possible

2016-10-22 Thread James Almer
This avoids potential rounding errors and guarantees the source aspect ratio is preserved. Keep writing pixel values when Stereo 3D Mode is enabled and for WebM, as the format doesn't support anything else. This fixes ticket #5743, implementing the suggestion from ticket #5903. Signed-off-by: Jam