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

Nicolas George george at nsup.org
Thu Oct 20 20:05:55 EEST 2016


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 to this (as DisplayHeight if unstored, defaults to PixelHeight)
> PixelWidth=720
> PixelHeight=480
> DisplayWidth=640

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.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161020/f389bf08/attachment.sig>


More information about the ffmpeg-devel mailing list