Do not use the FPS field of RealMedia files for the default duration of RealVideo tracks. It may contain the maximum number of frames per second that the encoder should output or has output. Fixes Anthill bug 113.

This commit is contained in:
Moritz Bunkus 2005-04-07 15:17:17 +00:00
parent 35153c7a2b
commit 08e457eaaf
2 changed files with 8 additions and 1 deletions

View File

@ -1,5 +1,12 @@
2005-04-07 Moritz Bunkus <moritz@bunkus.org> 2005-04-07 Moritz Bunkus <moritz@bunkus.org>
* mkvmerge: bug fix: RealMedia files contain a "FPS" field in
their track headers. Unfortunately this field does not always
contain the actual FPS of a video track but the maximum number of
FPS that the encoder has output or should output. Therefore
mkvmerge does not use a "default duration" element for RealVideo
tracks anymore. Fixes Anthill bug 113.
* mkvmerge: bug fix: Failing calls to posix_fadvise upon adding a * mkvmerge: bug fix: Failing calls to posix_fadvise upon adding a
file to mmg caused mmg to think that the file identification file to mmg caused mmg to think that the file identification
failed. Now warnings for posix_fadvise are not output anymore, and failed. Now warnings for posix_fadvise are not output anymore, and

View File

@ -326,7 +326,7 @@ real_reader_c::create_packetizer(int64_t tid) {
mxprints(buffer, "V_REAL/%s", dmx->fourcc); mxprints(buffer, "V_REAL/%s", dmx->fourcc);
dmx->ptzr = dmx->ptzr =
add_packetizer(new video_packetizer_c(this, buffer, dmx->fps, add_packetizer(new video_packetizer_c(this, buffer, 0.0,
dmx->width, dmx->height, ti)); dmx->width, dmx->height, ti));
if ((dmx->fourcc[0] != 'R') || (dmx->fourcc[1] != 'V') || if ((dmx->fourcc[0] != 'R') || (dmx->fourcc[1] != 'V') ||
(dmx->fourcc[2] != '4') || (dmx->fourcc[3] != '0')) (dmx->fourcc[2] != '4') || (dmx->fourcc[3] != '0'))