mirror of
https://github.com/CCExtractor/ccextractor.git
synced 2026-02-03 21:23:48 +00:00
[PR #1341] [MERGED] [FIX] Attempt to fix long-running regression in TeleText #2114
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/CCExtractor/ccextractor/pull/1341
Author: @canihavesomecoffee
Created: 6/1/2021
Status: ✅ Merged
Merged: 6/10/2021
Merged by: @cfsmp3
Base:
master← Head:bugfix/teletext-regression📝 Commits (6)
7c6e3dbAttempt to fix long-running regression in TeleText613047cAdd changesc373d2cClang-format changes31f79f8Improved fix54d241aFixupd5e2bb9Merge branch 'master' into bugfix/teletext-regression📊 Changes
3 files changed (+8 additions, -2 deletions)
View changed files
📝
docs/CHANGES.TXT(+1 -0)📝
src/lib_ccx/general_loop.c(+1 -1)📝
src/lib_ccx/telxcc.c(+6 -1)📄 Description
In raising this pull request, I confirm the following (please check boxes):
My familiarity with the project is as follows (check one):
Regression test 78 (https://sampleplatform.ccextractor.org/regression/test/78/view)
has been broken since #622 was merged to fix other issues.
It's been traced back to be caused by not setting t0 at the correct time
(setting it using a calculated PTS time rather than taking it from the video frame),
and this commits attempts to fix that.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.