Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: write DisplayWidth and DisplayHeight elements only if they differ from PixelWidth and PixelHeight

2016-10-22 Thread James Almer
On 10/20/2016 2:05 PM, Nicolas George wrote: > Le nonidi 29 vendémiaire, an CCXXV, Dave Rice a écrit : >> You could also make the storage simpler by considering whether or not to >> write DisplayWidth and DisplayHeight independently. For instance: >> PixelWidth=720 >> PixelHeight=480 >> DisplayWidt

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: write DisplayWidth and DisplayHeight elements only if they differ from PixelWidth and PixelHeight

2016-10-20 Thread Nicolas George
Le nonidi 29 vendémiaire, an CCXXV, Dave Rice a écrit : > You could also make the storage simpler by considering whether or not to > write DisplayWidth and DisplayHeight independently. For instance: > PixelWidth=720 > PixelHeight=480 > DisplayWidth=640 > DisplayHeight=480 > > could be simplified t

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: write DisplayWidth and DisplayHeight elements only if they differ from PixelWidth and PixelHeight

2016-10-20 Thread Dave Rice
Hi James, > On Oct 18, 2016, at 11:20 PM, James Almer wrote: > > Signed-off-by: James Almer > --- > libavformat/matroskaenc.c | 6 -- > tests/fate/matroska.mak | 2 +- > 2 files changed, 5 insertions(+), 3 deletions(-) > > diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c

[FFmpeg-devel] [PATCH] avformat/matroskaenc: write DisplayWidth and DisplayHeight elements only if they differ from PixelWidth and PixelHeight

2016-10-18 Thread James Almer
Signed-off-by: James Almer --- libavformat/matroskaenc.c | 6 -- tests/fate/matroska.mak | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 6f2957c..03d5326 100644 --- a/libavformat/matroskaenc.c +++ b/libavform