diff options
author | Timo Rothenpieler <timo@rothenpieler.org> | 2023-06-15 00:45:44 +0200 |
---|---|---|
committer | Timo Rothenpieler <timo@rothenpieler.org> | 2023-06-16 22:35:24 +0200 |
commit | cb3453eb25e249b7c428badd24d8dbcfc74ec3e7 (patch) | |
tree | 929be10aeb22d006ce2da6405497be9378201fe3 /LICENSE.md | |
parent | 549430e14da798e776ac6543351c5b98aecf2d1a (diff) | |
download | ffmpeg-cb3453eb25e249b7c428badd24d8dbcfc74ec3e7.tar.gz |
Revert "avcodec/nvenc: fix b-frame DTS behavior with fractional framerates"
This reverts commit 9a245bdf5d7860b8bc5e5c21a105a075925b719a.
This commit basically broke all samples with fractional framerates,
rather than fixing them.
I at this point do not understand the original issue anymore, and I'm
not sure how this slipped my initial testing.
All my test samples must have happened to have a simple timebase.
The actual dts values pretty much always are just a simple chain of
1,2,3,4,5,... Or maybe slightly bigger steps. Each increase by one means
an advance in time by one unit of the timebase.
So a fractional framerate/timebase is already not an issue.
So with this patch applied, the calculation might end up substracting
huge values (1001 is a common one) from the dts, which would be an
offset of that many frames, not of that many fractions of a second.
This broke at least muxing into mp4, if the sample happened to have a
fractional framerate.
I do not thing the original issue this patch tried to fix existed in the
first place, so it can be reverted without further consequences.
Diffstat (limited to 'LICENSE.md')
0 files changed, 0 insertions, 0 deletions