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

VideoPlayer: write state of clocksync to ProcessInfo, make FinishPipeline dependent on this #10222

Merged
merged 3 commits into from Aug 4, 2016

Conversation

Projects
None yet
4 participants
@FernetMenta
Copy link
Member

commented Aug 3, 2016

@Jalle19

This comment has been minimized.

Copy link
Member

commented Aug 3, 2016

Why a lock instead of atomics since it's just one boolean?

@FernetMenta

This comment has been minimized.

Copy link
Member Author

commented Aug 3, 2016

Why a lock instead of atomics since it's just one boolean?

the boolean is supposed to get some company.

@fritsch

This comment has been minimized.

Copy link
Member

commented Aug 3, 2016

Once upon a time there was a boolean feeling so alone...

Am 03.08.2016 7:15 nachm. schrieb "Rainer Hochecker" <
notifications@github.com>:

Why a lock instead of atomics since it's just one boolean?

the boolean is supposed to get some company.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#10222 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABCfHWWyI-wiLq-1g6Ku4uqXMYNYkmeeks5qcMrngaJpZM4JblU2
.

@FernetMenta

This comment has been minimized.

Copy link
Member Author

commented Aug 3, 2016

jenkins build this please

@FernetMenta FernetMenta force-pushed the FernetMenta:finishpipe branch from cb8d229 to 533c018 Aug 4, 2016

@FernetMenta

This comment has been minimized.

Copy link
Member Author

commented Aug 4, 2016

jenkins build this please

@FernetMenta

This comment has been minimized.

Copy link
Member Author

commented Aug 4, 2016

Android build errors unrelated

@FernetMenta FernetMenta merged commit c24a52f into xbmc:master Aug 4, 2016

0 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
jenkins.kodi.tv You are a failure. Fix the code and try again......
Details

@FernetMenta FernetMenta deleted the FernetMenta:finishpipe branch Aug 4, 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.