summaryrefslogtreecommitdiffstats
path: root/libhb/sync.c
Commit message (Collapse)AuthorAgeFilesLines
* sync: fix some p-to-p issuesJohn Stebbins2016-06-091-72/+123
| | | | | | | | | After finding the start position, some data prior to the start from other streams could leak through causing duplicate timestamps in the output. Also, improves alignment of stop times of all streams when a stop time is set.
* sync: fix CC burn-inJohn Stebbins2016-05-251-1/+2
| | | | | | It was dropping subtitles because the "end of CC" marker buffer can have the same time as the next valid CC which triggered the subtitle overlap dropping code.
* sync: correct timestamp discontinuities in sync instead of reader (#192)John Stebbins2016-05-241-42/+459
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * sync: correct timestamp discontinuities in sync instead of reader This patch passes discontinuity information through the pipeline till it reaches sync.c. The timestamps are passed through the pipeline as read and unmodified to sync.c (instead of attempting to correct discontinuities in reader). In sync, when we see a discontinuity, we know where the next timestamp should be based on the timestamp and duration of the previous buffer (before the discontinuity). So we calculate an "SCR" offset based on the timestamp after the discontinuity and what we calculate it should be. The old discontinuity handling code was broken due to the following. The MPEG STD timing model relies heavily on the decoder having an STC that is phase lock looped to the PCRs in the stream. When decoding a broadcast stream, the decoder can count on the time measure between PCRs using the STC to match to a high degree of accuracy. I.e. STC - lastSTC == PCR - lastPCR. When a discontinuity occurs, the decoder calculates a new PCR offset = PCR - STC. I.e. the offset is the new PCR value minus what it would have been if there had been no discontinuity. The above does not work without a reliable STC, which we do not have. We have been attempting to approximate one by avereraging the duration of received packets and extrapolating an "STC" based on the last PTS and the average packet duration. But this is highly variable and unreliable. * decavcodec: fix data type of next_pts It needs to be double so that partial ticks are not lost * deccc608sub: clarify comment * sync: allow queueing more audio Audio is small, and there is often a significant amount of audio in the stream before the first video frame. * sync: improve handling of damaged streams When data is missing, the audio decoder was extrapolating timestamps from the last pts before the error caused by the missing data which caused sync issues. Also, missing data can cause the video decoder to output a frame out of order with the wrong scr sequence. Drop such frames.
* sync: fill queues completelyJohn Stebbins2016-05-211-1/+1
| | | | | | essentially an off-by-one error. OutputBuffer had to wait for one more buffer before any output was performed after the queue should have already been filled to it's minimum levels.
* sync: silence compiler warningJohn Stebbins2016-05-201-1/+1
|
* sync: detect and drop 0 duration initial video frameJohn Stebbins2016-05-191-7/+12
|
* sync: increse min video queue depth for deeper dejitteringJohn Stebbins2016-05-181-1/+1
|
* sync: fix potential dropped chapter marksJohn Stebbins2016-05-181-10/+31
|
* sync: improve code readabilityJohn Stebbins2016-05-181-18/+19
|
* work: don't allocate fifos that are not needed for indepth_scanJohn Stebbins2016-05-181-6/+0
|
* sync: set start and stop when resampling audioJohn Stebbins2016-05-181-0/+2
| | | | | It's not strictly necessary because it gets done elsewhere as well. But putting it here makes the code more understandable.
* sync: fix progress statistics when title longer than sum of chaptersJohn Stebbins2016-05-181-1/+21
|
* sync: dejitter larger video jittersJohn Stebbins2016-05-181-1/+1
|
* sync: handle coarsely interleaved streams betterJohn Stebbins2016-05-181-2/+2
| | | | | ... by allowing a deeper initial buffer when looking for the fist PTS of each stream.
* sync: add timestamp loggingJohn Stebbins2016-05-181-3/+41
| | | | ... at log level 11 ;)
* sync: move comment to more appropriate locationJohn Stebbins2016-05-181-10/+12
|
* sync: do not drop audio before frame p-to-p startJohn Stebbins2016-05-181-11/+33
| | | | | | | We were dropping all buffers before the start frame was found regardless of the buffers start time. Now we keep track of the start time of the last video frame seen and only drop buffers that start before that frame.
* sync: estimate video frame drop duration betterJohn Stebbins2016-05-171-1/+8
|
* sync: cosmeticsJohn Stebbins2016-05-171-39/+39
|
* sync: drop overlapping subtitlesJohn Stebbins2016-05-171-6/+43
|
* sync: interleave subtitle output with other streamsJohn Stebbins2016-05-171-20/+3
|
* sync: send output directly to sync output fifosJohn Stebbins2016-05-171-43/+54
| | | | | | | | since sync interleaves it's output by PTS, the stream of the incoming buffer is mostly not the same as the stream of the outgoing buffer. This causes a delay in the data to get to it's respective fifo until the sync work function for the output stream is called next. Writing directly to the output fifo fixes this.
* sync: sanitize subtitles in OutputBufferJohn Stebbins2016-05-171-31/+53
|
* sync: handle initial pts < 0 betterJohn Stebbins2016-05-171-99/+102
|
* sync: fix race condition that caused double freeJohn Stebbins2016-05-171-0/+4
|
* sync: handle very short streams betterJohn Stebbins2016-05-171-0/+4
| | | | | If the entire stream fits in the sync queues, the first PTS was not detected and initial offsets were not applied.
* sync: fix conversions of AV_NOPTS_VALUE to doubleJohn Stebbins2016-05-171-5/+5
| | | | | The way the constant is defined requires an (int64_t) cast to force it to be signed.
* sync: remove unused variableJohn Stebbins2016-05-171-1/+0
|
* sync: fix merging of multiple SSA to tx3gJohn Stebbins2016-05-171-108/+233
| | | | | | When more than 2 subtitles overlapped in time, they were not merged properly and could result in cases where the subtitle time went backwards
* sync: move chapter logging to sync.cJohn Stebbins2016-05-171-0/+30
|
* move cadence (Film->Video) logging to sync.cJohn Stebbins2016-05-171-0/+95
|
* sync: fix hang at end of p-to-p encoding with subtitlesJohn Stebbins2016-03-111-1/+8
| | | | | If there are no more subtitles in a subtitle track after the stop time of a p-to-p encoding, the encode would never finish.
* sync: fix yet another subtitle issue.John Stebbins2016-03-111-1/+2
| | | | | | subtitle stop time was getting incorrectly squashed when special value AV_NOPTS_VALUE. it should have been set to the next subtitles start time.
* sync: fix handling of end-of-subtitle markersJohn Stebbins2016-03-061-0/+4
| | | | | Under some discontinuity cases, these markers were getting passed through to the muxer, which caused a crash.
* sync: improve video dejitterJohn Stebbins2016-02-151-1/+1
| | | | | Found a sample that requires 12 frames to detect and fix video jitter. So extend the minimum frame buffer from 10 to 12.
* sync: fix crash when sync init failsJohn Stebbins2016-02-121-1/+2
|
* sync: fix subtitle timestamps that go backwardsJohn Stebbins2016-02-101-3/+22
| | | | | This should be really difficult to trigger, but someone found a sample that does.
* sync: fix hang in subtitle scan passJohn Stebbins2016-01-271-0/+9
| | | | | if there are no subtitles in the file, our check that would terminate the subtitle scan pass does not get executed.
* sync: fix delayed subtitles when burned-inJohn Stebbins2016-01-271-0/+14
| | | | sync was delaying delivery of subtitles which caused burn-in late.
* sync: fix hang in ssa subtitle syncJohn Stebbins2016-01-211-1/+5
| | | | Failed to pass along EOF buffer.
* sync: gut and rewriteJohn Stebbins2016-01-211-1201/+1615
| | | | | | | | | | | sync.c was difficult to read, fragile, and prone to difficult to diagnose bugs (see https://forum.handbrake.fr/viewtopic.php?f=12&t=33147) This rewrite simplifies the code, removes signals, locking and yield that probably cause the above problem and is much more flexible. It fixes a wider variety of timestamp issues than before and is much easier to extend if other timestamp analysis is desired.
* libhb: fix hang during subtitle scanJohn Stebbins2016-01-191-0/+6
|
* sync: fix typo that could result in dropping subtitlesJohn Stebbins2016-01-151-1/+1
|
* Update copyright dates to 2016.Bradley Sepos2016-01-011-1/+1
|
* libhb: use LL for int64 constants to make mingw happyJohn Stebbins2015-12-261-1/+1
|
* libhb: make muxer, sync, and reader behave like other work objectsJohn Stebbins2015-11-091-46/+33
| | | | | | | | | | | | | simplify job initialization sequence, clean up code, and document dependencies in the sequence better. Make hb_add set job->sequence_id. It is no longer necessary for the frontend to do this. If the frontend needs the sequence_id, it is returned by hb_add(). Clean up use of interjob. do_job() now uses sequence_id to detect when a new sequence of related jobs is running and automatically clears interjob.
* libhb: add hb_buffer_listJohn Stebbins2015-09-241-79/+46
| | | | | This brings together several independent implementations of a simple buffer list manager.
* libhb: remove unused hb_buffer_t.subJohn Stebbins2015-09-041-1/+0
|
* cli: fix initialization of encode rangesjstebbins2015-05-131-0/+2
| | | | | | | | | | | The default chapter start or end was used if one but not the other of start-at/stop-at were used. Also don't overwrite current pass and pass_count status in hb_state_t while searching for the position to start encoding. git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@7177 b64f7644-9d1e-0410-96f1-a4d463321fa5
* libhb: Use a buffer flat to indicate EOFjstebbins2015-05-011-8/+8
| | | | | | | | | | | | ... instead of a 0 length buffer. This fixes this issue: https://forum.handbrake.fr/viewtopic.php?f=12&t=31959 Theora can create 0 length output. These 0 length frames indicate duplicate frames. So we can't use 0 length buffers to indicate the end of the stream. git-svn-id: svn://svn.handbrake.fr/HandBrake/trunk@7143 b64f7644-9d1e-0410-96f1-a4d463321fa5
generated by cgit v1.2.3 (git 2.39.1) at 2024-11-29 09:38:22 +0000