search

delight-im / Android-DDP

[UNMAINTAINED] Meteor's Distributed Data Protocol (DDP) for clients on Android
Apache License 2.0
274 stars 54 forks source link

mReconnectAttempts issue #62

Open chopikboy opened 8 years ago

chopikboy commented 8 years ago

Method onClose (Meteor.java 148). I think mReconnectsAttempts can't be more than 1. Because we increase this one only if lostConnection == true, and lostConnection is true only after we do mConnected = true, but we do mConnected = true only in one place - onOpen() (Meteor.java 140), where we also do mReconnectAttempts = 0.

ocram commented 8 years ago

Thank you very much, and sorry for the late reply!

You're right :) Is there anything you would suggest as a fix?

Just remove lostConnection and the check for lostConnection to be true?