aboutsummaryrefslogtreecommitdiffstats
path: root/COPYING.GPLv2
diff options
context:
space:
mode:
authorTimo Rothenpieler <timo@rothenpieler.org>2023-06-15 00:45:44 +0200
committerTimo Rothenpieler <timo@rothenpieler.org>2023-06-15 00:59:02 +0200
commitac7c265b33b52f914ebe05e581bbe9343eca1186 (patch)
tree78f07e4b3ab34655ef0e91fe3bfa119d2a62d7e5 /COPYING.GPLv2
parentcfcd6e2108a7c43394840ca857ba1b6e8f6351c2 (diff)
downloadffmpeg-ac7c265b33b52f914ebe05e581bbe9343eca1186.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 'COPYING.GPLv2')
0 files changed, 0 insertions, 0 deletions