Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use video disptime only in case video is running #11018

Merged
merged 1 commit into from Nov 30, 2016

Conversation

Projects
None yet
2 participants
@peak3d
Copy link
Contributor

commented Nov 30, 2016

See title

Description

If Inputstream provides only audio data, the not initialized disptime from video is taken to calculate time offsets wich are displayed later in the UI. This PR fixes the use of this not initialzed value.

How Has This Been Tested?

Win10 before / after test

Types of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@peak3d peak3d merged commit 9de42ad into xbmc:master Nov 30, 2016

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@MartijnKaijser MartijnKaijser added this to the L 18.0-alpha1 milestone Nov 30, 2016

@peak3d

This comment has been minimized.

Copy link
Contributor Author

commented Nov 30, 2016

@MartijnKaijser its a bug. agreed with @FernetMenta to merge.

@peak3d peak3d deleted the peak3d:timeoffset branch Nov 30, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.