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

Pull new commits from forked repository #1

Merged
merged 48 commits into from
Aug 12, 2019

Conversation

francoisadam
Copy link
Collaborator

No description provided.

…ctory (#573)

Due to a race condition and subscribing/unsubscribing from different threads it may have happened that unregistering was made before registering finished.
The issue was hidden by a try/catch. This should not happen when using Schedulers.trampoline() to subscribe/unsubscribe.
The access to the callback is available through `RxBleConnection.observeConnectionParametersUpdates()`

Additionally it is possible to get it in `RxBleCustomOperation`:
- `RxBleGattCallback.getConnectionParametersUpdates()`
- `RxBleGattCallback.setHiddenGattCallback(HiddenBluetoothGattCallback)`

The callback is not used for completing `RxBleConnection.requestConnectionPriority()`. It is not possible to universally distinguish which callback is the one that should complete the request. Why? Android OS does change connection parameters on its own at the beginning of the connection to speedup service discovery process. After the discovery parameters are relaxed. If a request is made during this period it is not possible to know which callback is related to the user's request. It is worth noting that making a request early will make it quickly overwritten by the above mentioned process.

This is _not_ a part of public Android API and is not guaranteed to be called.
Usually there are a lot of peripherals advertising nearby and it may be considered too spammy.
# Conflicts:
#	gradle.properties
io.reactivex.rxjava2:rxjava:2.2.7 -> 2.2.8
org.codehaus.groovy:groovy:2.5.6 -> 2.5.7
cglib:cglib-nodep:3.2.10 -> 3.2.12
This reverts commit b3dca6b.
This reverts commit e23018a.
It is now possible to set a whitelist of branches that may produce snapshots
Use “true”/“false” and camelCase, remove error return
@francoisadam francoisadam merged commit 6ac2a80 into GuillaumeBourge:master Aug 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants