Skip to content

Fix connection to ClientDisconnected signal#3760

Open
dingodoppelt wants to merge 1 commit into
mainfrom
dingodoppelt-patch-1
Open

Fix connection to ClientDisconnected signal#3760
dingodoppelt wants to merge 1 commit into
mainfrom
dingodoppelt-patch-1

Conversation

@dingodoppelt

@dingodoppelt dingodoppelt commented Jun 29, 2026

Copy link
Copy Markdown
Member

Short description of changes

Fixes a regression in jamulusserver/clientDisconnected

CHANGELOG: Fix connection to ClientDisconnected signal

Status of this Pull Request

Ready to be merged

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

@ann0see ann0see requested a review from pljones June 29, 2026 20:24
@dingodoppelt

Copy link
Copy Markdown
Member Author

Reintroducing this as a receiver into the connect function makes Qt::AutoConnection use Qt::QueuedConnection for connections across threads instead of Qt::DirectConnection
https://doc.qt.io/qt-6/qt.html#ConnectionType-enum

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