#671 Fixed per code review recommendationbugfix/issue_671

- Use for loop instead of do/while loop

1 files changed, 3 insertions, 4 deletions

diff --git a/sdl_android/src/main/java/com/smartdevicelink/transport/SdlBroadcastReceiver.java b/sdl_android/src/main/java/com/smartdevicelink/transport/SdlBroadcastReceiver.java
index 36f91becd..2176b2457 100644
--- a/sdl_android/src/main/java/com/smartdevicelink/transport/SdlBroadcastReceiver.java
+++ b/sdl_android/src/main/java/com/smartdevicelink/transport/SdlBroadcastReceiver.java
@@ -223,14 +223,13 @@ public abstract class SdlBroadcastReceiver extends BroadcastReceiver{
 								}
 							});
 							serviceIntent = new Intent();
-							String pkgName;
-							do {
-								pkgName = apps.remove(0).activityInfo.packageName;
+							for (ResolveInfo app : apps) {
+								String pkgName = app.activityInfo.packageName;
 								serviceIntent.setComponent(new ComponentName(pkgName, pkgName + ".SdlRouterService"));
 								if (packageManager.resolveService(serviceIntent, PackageManager.MATCH_DEFAULT_ONLY) != null) {
 									break;
 								}
-							} while (apps.size() > 1);
+							}
 						} else{
 							Log.d(TAG, "No router service running, starting ours");
 							//So let's start up our service since no copy is running