minor changes to podcast downloader

thasmin · Jun 4, 2012 · 9d05b59 · 9d05b59
1 parent e0475cd
commit 9d05b59
Showing 1 changed file with 18 additions and 17 deletions.
diff --git a/src/com/axelby/podax/PodcastDownloader.java b/src/com/axelby/podax/PodcastDownloader.java
@@ -42,29 +42,30 @@ public void download(int podcastId) {
 			cursor = _context.getContentResolver().query(PodcastProvider.QUEUE_URI, projection,
 					PodcastProvider.COLUMN_ID + " = ?",
 					new String[] { String.valueOf(podcastId) }, null);
-			while (cursor.moveToNext()) {
-				PodcastCursor podcast = new PodcastCursor(_context, cursor);
-				if (podcast.isDownloaded())
-					continue;
+			if (!cursor.moveToNext())
+				return;
 
-				File mediaFile = new File(podcast.getFilename());
+			PodcastCursor podcast = new PodcastCursor(_context, cursor);
+			if (podcast.isDownloaded())
+				return;
 
-				Log.d("Podax", "Downloading " + podcast.getTitle());
-				updateDownloadNotification(podcast, 0);
+			File mediaFile = new File(podcast.getFilename());
 
-				HttpURLConnection c = openConnection(podcast, mediaFile);
-				if (c == null)
-					continue;
+			Log.d("Podax", "Downloading " + podcast.getTitle());
+			updateDownloadNotification(podcast, 0);
 
-				if (!downloadFile(c, mediaFile))
-					continue;
+			HttpURLConnection c = openConnection(podcast, mediaFile);
+			if (c == null)
+				return;
 
-				if (mediaFile.length() == c.getContentLength())
-					podcast.determineDuration();
+			if (!downloadFile(c, mediaFile))
+				return;
 
-				removeDownloadNotification();
-				Log.d("Podax", "Done downloading " + podcast.getTitle());
-			}
+			if (mediaFile.length() == c.getContentLength())
+				podcast.determineDuration();
+
+			removeDownloadNotification();
+			Log.d("Podax", "Done downloading " + podcast.getTitle());
 		} finally {
 			if (cursor != null)
 				cursor.close();