[PR #95] [MERGED] [Linux] Implement proper disconnection detection #281

Closed
opened 2026-03-02 12:40:26 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/kavishdevar/librepods/pull/95
Author: @tim-gromeyer
Created: 4/14/2025
Status: Merged
Merged: 4/14/2025
Merged by: @tim-gromeyer

Base: mainHead: linux-disconnection-detection


📝 Commits (4)

  • a313a80 [Linux] Implement proper disconection detection
  • 29b167a Only trigger on airpod devices
  • b7c4fca Remove unused code
  • 10047ae [Linux] Fix SegmentedControl text not shown in release build (but debug builds showed text)???

📊 Changes

5 files changed (+156 additions, -108 deletions)

View changed files

linux/BluetoothMonitor.cpp (+93 -0)
linux/BluetoothMonitor.h (+26 -0)
📝 linux/CMakeLists.txt (+2 -0)
📝 linux/SegmentedControl.qml (+12 -0)
📝 linux/main.cpp (+23 -108)

📄 Description

By watching the bluz dbus, we now have proper disconnection detection.

The previous code did not work, for l2cap sockets, Qt's disconnected event isn't emitted reliably.

This code even triggers when we disable bluetooth.

Fixed:

  • Disconnection was not detected
  • Automatically allow reconnecting (no need to restart app anymore)
  • Fixed text not visible in SegmentedControl in release builds (it worked in debug builds??)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/kavishdevar/librepods/pull/95 **Author:** [@tim-gromeyer](https://github.com/tim-gromeyer) **Created:** 4/14/2025 **Status:** ✅ Merged **Merged:** 4/14/2025 **Merged by:** [@tim-gromeyer](https://github.com/tim-gromeyer) **Base:** `main` ← **Head:** `linux-disconnection-detection` --- ### 📝 Commits (4) - [`a313a80`](https://github.com/kavishdevar/librepods/commit/a313a80c86e6cafd04bbb7d6a6afc608a9ac7fdf) [Linux] Implement proper disconection detection - [`29b167a`](https://github.com/kavishdevar/librepods/commit/29b167ad988e9863eb89d93421634f37ba44ba1f) Only trigger on airpod devices - [`b7c4fca`](https://github.com/kavishdevar/librepods/commit/b7c4fcad13ca546009b15d41d5288956c9ed7aa0) Remove unused code - [`10047ae`](https://github.com/kavishdevar/librepods/commit/10047ae1f1ed0ef1a50e58d656517b0bb06a6bbf) [Linux] Fix SegmentedControl text not shown in release build (but debug builds showed text)??? ### 📊 Changes **5 files changed** (+156 additions, -108 deletions) <details> <summary>View changed files</summary> ➕ `linux/BluetoothMonitor.cpp` (+93 -0) ➕ `linux/BluetoothMonitor.h` (+26 -0) 📝 `linux/CMakeLists.txt` (+2 -0) 📝 `linux/SegmentedControl.qml` (+12 -0) 📝 `linux/main.cpp` (+23 -108) </details> ### 📄 Description By watching the bluz dbus, we now have proper disconnection detection. The previous code did not work, for l2cap sockets, Qt's disconnected event isn't emitted reliably. This code even triggers when we disable bluetooth. #### Fixed: - [x] Disconnection was not detected - [x] Automatically allow reconnecting (no need to restart app anymore) - [x] Fixed text not visible in SegmentedControl in release builds (it worked in debug builds??) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-02 12:40:26 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/librepods#281
No description provided.