Skip to content

Conversation

patrick96
Copy link
Member

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Other: Replace this with a description of the type of this PR

Description

If context_state_callback is called before we call
pa_threaded_mainloop_wait in the main thread, the signal is lost and we
wait forever.

Fixes #2707
Ref #2699
Ref #2697

Documentation (check all applicable)

  • This PR requires changes to the Wiki documentation (describe the changes)
  • This PR requires changes to the documentation inside the git repo (please add them to the PR).
  • Does not require documentation changes

If context_state_callback is called before we call
pa_threaded_mainloop_wait in the main thread, the signal is lost and we
wait forever.

Fixes polybar#2707
Ref polybar#2699
Ref polybar#2697
@codecov
Copy link

codecov bot commented May 4, 2022

Codecov Report

Merging #2709 (ce0f504) into hotfix/3.6.3 (efbd8e3) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           hotfix/3.6.3    #2709      +/-   ##
================================================
- Coverage         13.76%   13.76%   -0.01%     
================================================
  Files               153      153              
  Lines             11296    11299       +3     
================================================
  Hits               1555     1555              
- Misses             9741     9744       +3     
Flag Coverage Δ
unittests 13.76% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/adapters/pulseaudio.cpp 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update efbd8e3...ce0f504. Read the comment docs.

@patrick96 patrick96 merged commit f653c3a into polybar:hotfix/3.6.3 May 4, 2022
@patrick96 patrick96 deleted the fix/pulse-hang branch May 4, 2022 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant