From 4c30c48684cd09a057906f30d7a3f8927ecabb52 Mon Sep 17 00:00:00 2001 From: Artur Dryomov Date: Tue, 3 Sep 2013 15:53:03 +0300 Subject: Move building server via TCP servers finder to a separate method. Change-Id: I8def7a5f7ecf924f6cc61f3a037defa9e8116089 Reviewed-on: https://gerrit.libreoffice.org/5815 Reviewed-by: Michael Meeks Tested-by: Michael Meeks --- .../impressremote/communication/TcpServersFinder.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'android/sdremote') diff --git a/android/sdremote/src/org/libreoffice/impressremote/communication/TcpServersFinder.java b/android/sdremote/src/org/libreoffice/impressremote/communication/TcpServersFinder.java index b31e4f66edc3..84f305be9809 100644 --- a/android/sdremote/src/org/libreoffice/impressremote/communication/TcpServersFinder.java +++ b/android/sdremote/src/org/libreoffice/impressremote/communication/TcpServersFinder.java @@ -124,11 +124,7 @@ class TcpServersFinder implements ServersFinder, Runnable { String aFoundServerHostname = aSearchResultScanner.nextLine(); - Server aFoundServer = new Server(Server.Protocol.TCP, - aSearchResultPacket.getAddress().getHostAddress(), - aFoundServerHostname); - - addServer(aFoundServer); + addServer(buildServer(aSearchResultPacket, aFoundServerHostname)); callUpdatingServersList(); } @@ -156,6 +152,12 @@ class TcpServersFinder implements ServersFinder, Runnable { mServers.put(aServer.getAddress(), aServer); } + private Server buildServer(DatagramPacket aSearchResultPacket, String aServerHostname) { + String aServerAddress = aSearchResultPacket.getAddress().getHostAddress(); + + return Server.newTcpInstance(aServerAddress, aServerHostname); + } + private void callUpdatingServersList() { Intent aIntent = Intents.buildServersListChangedIntent(); LocalBroadcastManager.getInstance(mContext).sendBroadcast(aIntent); -- cgit