Skip to content
Snippets Groups Projects

RFC: libVLC: use microseconds (us) for libvlc_time_t ?

Open Thomas Guillem requested to merge tguillem/vlc:libvlc-new-apis-us into master

Only the last commit should be reviewed.

  • libvlc_clock() returned int64_t and used us.
  • New player watch timer API used us.
  • Lot of set/get time use libvlc_time_t as ms.
  • timeout are using int as ms (not changed)

**Questions: **

  • OK with the change ?
  • Should we keep timeout as int / ms ? (Or use libvlc_time_t/us)

If accepted, will be merged on top of !3503 (In order to have all major libVLC changes in one shot)

Merge request reports

Members who can merge are allowed to add commits.

Merge request pipeline #518624 passed

Merge request pipeline passed for 87677937

Test coverage 18.61% (0.15%) from 1 job
Approval is optional
Ready to merge by members who can write to the target branch.

Merge details

  • The source branch is 2645 commits behind the target branch.
  • 19 commits and 1 merge commit will be added to .
  • Source branch will not be deleted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading