-
Notifications
You must be signed in to change notification settings - Fork 575
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
Fixed https://github.com/Polidea/RxAndroidBle/issues/60 #62
Conversation
@@ -76,6 +76,12 @@ public void addAdvertisedUUID(UUID advertisedUUID) { | |||
} | |||
|
|||
@Override | |||
public BluetoothDevice getBluetoothDevice() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this change of places needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, just autoformatter. Do you want me to revert it to the previous state?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, for pull requests these reorderings only make it worse to read. For refactorings that are not changing code but only reordering - you can push directly.
Done, also I've rebased to current master in order to avoid checkstyle fail due to an unused import in other part of the library. |
Rebased and merged into master ( f1a67da ) |
Connection was emitted incorrectly in
establishConnection
method. Observable was callingonComplete
after emission which then caused thatdoOnUnsubscribe
. This is the reason thatDISCONNECTED
state was emitted every time, and it should only be emitted after explicit unsubscription from connection observable.