8000 [Linux] BlueZ changes to improve the use of the BluezEndpoint object by DamMicSzm · Pull Request #8 · DamMicSzm/connectedhomeip · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[Linux] BlueZ changes to improve the use of the BluezEndpoint object #8

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

[Linux] BlueZ changes to improve the use of the BluezEndpoint object #8

wants to merge 8 commits into from

Conversation

DamMicSzm
Copy link
Owner
@DamMicSzm DamMicSzm commented Dec 13, 2023 •
edited
Loading

Problem

BLE advertising and scanner classes cache adapter object as a private member. In case of D-Bus service restart (in that case BlueZ), the D-Bus connection ID changes, however, these cached objects will not be updated. This PR is a preliminary change required to fix issue project-chip#30792

Changes

  • Always get adapter object from the BluezEndpoint class as a source of trust

  • Ensure the existence of the object in the compilation process.

  • Convert some glib object to GAutoPtr<> in BlueZ

Testing

CI will test for potential build breaks.

@DamMicSzm DamMicSzm force-pushed the linux-bluez-endpoint-ref branch 4 times, most recently from 86cee55 to d47e4a6 Compare December 14, 2023 10:50
@DamMicSzm DamMicSzm force-pushed the linux-bluez-endpoint-ref branch from d47e4a6 to 793d041 Compare December 14, 2023 12:26
@DamMicSzm
Use reset() for GAutoPtr<>:
8c36b95 
- Restore run cleanup on the CHIPoBluez thread

- Use for GAutoPtr<> object clear

- Use for re-init GAutoPtr<>
@DamMicSzm DamMicSzm force-pushed the linux-bluez-endpoint-ref branch 3 times, most recently from a59cd1d to f49e236 Compare December 14, 2023 16:07
@DamMicSzm
Cleanup:
0684477 
- Remove unnecessary delete operator from classes

- Fix lost self in lambda function

- Remove not needed comment

- Restyled by clang-format
@DamMicSzm
Fixes CI and other
48559cf 
- Fix missing mpC3 getter
@DamMicSzm
Reducing duplicate types in BLE manager
576fac8 
- Restyled by clang-format
@DamMicSzm
WIP - Python BLE API
7fba468 
- Restyled by clang-format
@DamMicSzm DamMicSzm force-pushed the linux-bluez-endpoint-ref branch from f49e236 to 7fba468 Compare December 15, 2023 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@DamMicSzm
0