From d230fc86db509ef1d55243a983983106b5822272 Mon Sep 17 00:00:00 2001
From: NissimHadar <nissim.hadar@gmail.com>
Date: Thu, 28 Mar 2019 12:23:25 -0700
Subject: [PATCH] CR comments.

---
 tools/nitpick/src/TestRunnerMobile.cpp | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/tools/nitpick/src/TestRunnerMobile.cpp b/tools/nitpick/src/TestRunnerMobile.cpp
index 969da02c9e..53a74da82f 100644
--- a/tools/nitpick/src/TestRunnerMobile.cpp
+++ b/tools/nitpick/src/TestRunnerMobile.cpp
@@ -117,12 +117,10 @@ void TestRunnerMobile::connectDevice() {
             _modelName = "UNKNOWN";
             for (int i = 0; i < tokens.size(); ++i) {
                 if (tokens[i].contains(MODEL)) {
-                    if (i < tokens.size()) {
-                        QString modelID = tokens[i].split(':')[1];
+                    QString modelID = tokens[i].split(':')[1];
 
-                        if (modelNames.count(modelID) == 1) {
-                            _modelName = modelNames[modelID];
-                        }
+                    if (modelNames.count(modelID) == 1) {
+                        _modelName = modelNames[modelID];
                     }
                     break;
                 }
@@ -223,7 +221,7 @@ void TestRunnerMobile::runInterface() {
         startCommand = "io.highfidelity.hifiinterface/.PermissionChecker";
     }
 
-        QString serverIP { getServerIP() };
+    QString serverIP { getServerIP() };
     if (serverIP == NETWORK_NOT_FOUND) {
         _runInterfacePushbutton->setEnabled(false);
         return;