[haiku-commits] Change in haiku[master]: ffmpeg: Use ffmpeg-provided timestamp

  • From: Gerrit <review@xxxxxxxxxxxxxxxxxxx>
  • To: waddlesplash <waddlesplash@xxxxxxxxx>, haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 4 Nov 2018 09:05:42 +0000

From Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>:

Adrien Destugues has uploaded this change for review. ( 
https://review.haiku-os.org/667


Change subject: ffmpeg: Use ffmpeg-provided timestamp
......................................................................

ffmpeg: Use ffmpeg-provided timestamp

It makes no sense to use the packet dts as a presentation timestamp.
However, the frames don't reliably have a dts. ffmpeg provides us with a
"best effort" timestamp which is more likely to be correct (or as
correct as possible) in all cases.

Note that this relies on the previous commit to get the relevant
information from the AV Packet.

While I'm at it, also convert the timestamp to microseconds as required
by the media_header struct, and don't assume anything about the ffmpeg
timebase.
---
M src/add-ons/media/plugins/ffmpeg/AVCodecDecoder.cpp
1 file changed, 5 insertions(+), 1 deletion(-)



  git pull ssh://git.haiku-os.org/haiku refs/changes/67/667/1
--
To view, visit https://review.haiku-os.org/667
To unsubscribe, or for help writing mail filters, visit 
https://review.haiku-os.org/settings

Gerrit-Project: haiku
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib922bd0b79baa793798b016ec37763363cac5b7a
Gerrit-Change-Number: 667
Gerrit-PatchSet: 1
Gerrit-Owner: Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>

Other related posts: