Fixing the stopADBlog to not kill any process if pid is 0 (#31)

Author: shridharkalagi

src/main/java/com/thoughtworks/android/AndroidManager.java

@@ -120,14 +120,17 @@ public String startADBLogWithPackage(String udid, String packageName, String fil
         process = cmd.execForProcessToExecute("adb -s " + udid
                 + " logcat | grep -F \"`adb shell ps | grep " + packageName + " | cut -c10-15`\"" + " > " + filePath);
         processUDIDs.put(udid, process);
-        return "Collecting ADB logs for device " + udid + " for package "+packageName+" in file " + filePath;
+        return "Collecting ADB logs for device " + udid + " for package " + packageName + " in file " + filePath;
     }
 
     public String stopADBLog(String udid) throws Exception {
         Process p = processUDIDs.get(udid);
-        int id = (getPid(p) > 0) ? getPid(p) : 0;
-        cmd.runCommandThruProcess("kill -9 " + id);
-        return "Stopped collecting ADB logs " + udid;
+        if (getPid(p) > 0) {
+            cmd.runCommandThruProcess("kill -9 " + getPid(p));
+            return "Stopped collecting ADB logs " + udid;
+        } else
+            return "No process found to kill";
+
     }
 
     public int getPid(Process process) {