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

  • From: Gerrit <review@xxxxxxxxxxxxxxxxxxx>
  • To: waddlesplash <waddlesplash@xxxxxxxxx>, haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 2 Jan 2019 13:45:57 +0000

From Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>:

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


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, 1 insertion(+), 1 deletion(-)



  git pull ssh://git.haiku-os.org/haiku refs/changes/13/813/1
--
To view, visit https://review.haiku-os.org/813
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: Ie54f38082bc26e7a56f78a1bce1a1abe1b0a3663
Gerrit-Change-Number: 813
Gerrit-PatchSet: 1
Gerrit-Owner: Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>

Other related posts: