Skip to content
Snippets Groups Projects

WIP: ancillary: pass through decoders and packetizers

Open Thomas Guillem requested to merge tguillem/vlc:ancillary-decoderc into master
2 unresolved threads

On top of !7072

  • Added ancillary passthrough for all packetizers
  • Added vlc_ancillary_queue: a helper for asynchronous decoders that can't have a context (like MediaCodec)
  • Implemented ancillary passthrough in avcodec via frame_info
  • Implemented simple ancillary passthrough in faad.c (lot of decoders modules will be alike)
  • Implemented ancillary passthrough in mediacodec via vlc_ancillary_queue
  • Implemented ancillary passthrough in videotoolbox via frame_info

WIP:

  • Need to find a proper way to get rid of the timestamp_Fifo helper in MediaCodec, any idea @fcartegnie ?
  • If you agree, I will forward ancillaries in all other decoders modules
Edited by Thomas Guillem

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Denis Charmet
  • Thomas Guillem added 4 commits

    added 4 commits

    • 3375677b - WIP: mediacodec: remove timestamp_Fifo
    • 50bc785d - mediacodec: forward ancillaries
    • 6ce90986 - faad: forward ancillaries
    • 62a8b62a - videotoolbox: forward ancillaries

    Compare with previous version

  • Thomas Guillem changed the description

    changed the description

  • Author Maintainer

    Added videotoolbox handling.

  • Thomas Guillem added 9 commits

    added 9 commits

    • 54b1653d - frame: add vlc_frame_ChainGatherAncillaries()
    • eff4a4f3 - frame: gather ancillaries from vlc_frame_ChainGather()
    • 045101e0 - ancillary: add vlc_ancillary_queue
    • c09e4c20 - packetizer: forward ancillaries
    • 8fcb00d1 - avcodec: forward all input ancillaries to the output
    • 758c2d3e - WIP: mediacodec: remove timestamp_Fifo
    • b258ea90 - mediacodec: forward ancillaries
    • 204a394e - faad: forward ancillaries
    • d2a7b934 - videotoolbox: forward ancillaries

    Compare with previous version

  • Denis Charmet
    • Author Maintainer

      Maybe found a proper workaround for MediaCodec timestamp_Fifo. I guess MediaCodec doesn't care about PTS vs DTS, but he cares about TS always incrementing (the DTS can increase and decrease). So if there is no PTS, I could start from the first DTS and use the date_t API to increase the PTS.

    • Author Maintainer

      cf. !7088

    • Please register or sign in to reply
  • Thomas Guillem mentioned in merge request !7088

    mentioned in merge request !7088

  • Please register or sign in to reply
    Loading