#11677: ffplay does not properly sync LRC subtitles with 3-digit milliseconds
-------------------------------------+-------------------------------------
Reporter: gbr | Owner: Kacper
| Michajłow <kasper93@…>
Type: defect | Status: closed
Priority: normal | Component: avformat
Version: unspecified | Resolution: fixed
Keywords: lrc | Blocked By:
Blocking: | Reproduced by developer: 0
Analyzed by developer: 0 |
-------------------------------------+-------------------------------------
Changes (by Kacper Michajłow <kasper93@…>):
* owner: (none) => Kacper Michajłow <kasper93@…>
* resolution: => fixed
Comment:
In [changeset:"bc3cc0a6af44adc63caf4e5097fcfebd7a7475b4/ffmpeg"
bc3cc0a6/ffmpeg]:
{{{#!CommitTicketReference repository="ffmpeg"
revision="bc3cc0a6af44adc63caf4e5097fcfebd7a7475b4"
avformat/lrcdec: support arbitrary precision timestamp
Apparently files with milliseconds exist in the wild. And since it cost
nothing to support arbitrary number of digits, extend format to support
that.
Depending on number of digits, the time base of fractional part is
changing. Most LRCs use 2 digits and centiseconds base, but subs with 3
digits and miliseconds exist too.
Set internal time base to AV_TIME_BASE, which in parcitice allows to
hold microseconds with 6 digits. Totally artificial, but who knows maybe
someone wants that.
Fixes: #11677
Signed-off-by: Kacper Michajłow <kaspe...@gmail.com>
}}}
--
Ticket URL: <https://trac.ffmpeg.org/ticket/11677#comment:11>
FFmpeg <https://ffmpeg.org>
FFmpeg issue tracker
_______________________________________________
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-trac
To unsubscribe, visit link above, or email
ffmpeg-trac-requ...@ffmpeg.org with subject "unsubscribe".
