Ticket #1754 (closed defect: fixed)
Error in Matroska Muxer
|Reported by:||Dan203||Owned by:|
|Blocking:||Reproduced by developer:||no|
|Analyzed by developer:||no|
On line 635 of matroskaenc.c there is an error. The line is currently...
put_ebml_uint(pb, MATROSKA_ID_VIDEODISPLAYUNIT, 3);
it should either be deleted completely or set to 0 instead of 3. (0 is default so deleting it should be OK)
According to the MKV specification...
...the DisplayUnit? value should only be set to 3 if the DisplayHeight? and DisplayWidth? values are set to aspect ratio values. However the lines above that one clearly show that DisplayHeight? and DisplayWidth? are being set to pixel values that are adjusted for aspect.
Most players ignore this value, but some hardware players have issues with files created with this incorrect DisplayUnit? value.
- Keywords mkv regression added; MKV, Matroska removed
- Status changed from new to open
- Version set to git-master