From f41877fbeca8d0aeb8762773b74e254d93c2b53f Mon Sep 17 00:00:00 2001 From: Albert Vaca Date: Tue, 12 Jul 2016 12:38:40 +0200 Subject: [PATCH] Disabled removing links when exiting discovery mode because it was broken. If both devices are in "discovery mode" (ie: both want to keep links established) and the last one to create the link goes out of discovery mode, it will close the link when it shouldn't (because the other end still was "using" it to display it to the user). --- core/backends/lan/landevicelink.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/core/backends/lan/landevicelink.cpp b/core/backends/lan/landevicelink.cpp index a6658e92f..716ef6e02 100644 --- a/core/backends/lan/landevicelink.cpp +++ b/core/backends/lan/landevicelink.cpp @@ -151,7 +151,11 @@ void LanDeviceLink::setPairStatus(PairStatus status) } bool LanDeviceLink::linkShouldBeKeptAlive() { + + return true; //FIXME: Current implementation is broken, so for now we will keep links always established + //We keep the remotely initiated connections, since the remotes require them if they want to request //pairing to us, or connections that are already paired. TODO: Keep connections in the process of pairing - return (mConnectionSource == ConnectionStarted::Remotely || pairStatus() == Paired); + //return (mConnectionSource == ConnectionStarted::Remotely || pairStatus() == Paired); + }