[FFmpeg-devel] [PATCH] avformat/matroskaenc: write DisplayWidth and DisplayHeight elements only if they differ from PixelWidth and PixelHeight
jamrial at gmail.com
Sat Oct 22 21:53:55 EEST 2016
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:
>> could be simplified to this (as DisplayHeight if unstored, defaults to PixelHeight)
> Even apart from the drawbacks you expect, I do not think this
> micro-optimization is very interesting. Logically, width and height go
> together, using half a default value seems unnatural.
I agree. Pushed as is.
More information about the ffmpeg-devel