From bba39b9b678f4fb00511ba88c12eef9082ecc628 Mon Sep 17 00:00:00 2001 From: Kurt Partridge Date: Wed, 27 Feb 2013 14:13:29 -0800 Subject: [PATCH] [FileEncap8] Remove useless "success" variable Change-Id: Icf76bc0b795b6a9221d9a6d61c38df31af54ab7b --- .../android/inputmethod/research/Uploader.java | 16 +++------------- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/java/src/com/android/inputmethod/research/Uploader.java b/java/src/com/android/inputmethod/research/Uploader.java index ac64076cb..df495a88d 100644 --- a/java/src/com/android/inputmethod/research/Uploader.java +++ b/java/src/com/android/inputmethod/research/Uploader.java @@ -116,23 +116,15 @@ public final class Uploader { && !pathname.canWrite(); } }); - // TODO: Remove local variable - boolean success = true; - if (files.length == 0) { - success = false; - } for (final File file : files) { - if (!uploadFile(file)) { - success = false; - } + uploadFile(file); } } - private boolean uploadFile(final File file) { + private void uploadFile(final File file) { if (DEBUG) { Log.d(TAG, "attempting upload of " + file.getAbsolutePath()); } - boolean success = false; final int contentLength = (int) file.length(); HttpURLConnection connection = null; InputStream fileInputStream = null; @@ -154,10 +146,9 @@ public final class Uploader { Log.d(TAG, "| " + reader.readLine()); } reader.close(); - return success; + return; } file.delete(); - success = true; if (DEBUG) { Log.d(TAG, "upload successful"); } @@ -175,7 +166,6 @@ public final class Uploader { connection.disconnect(); } } - return success; } private static void uploadContents(final InputStream is, final OutputStream os)