commit 5c5b1184a17ceee3b585d3e3eeba414a9b08fb19
author Marc Blank <mblank@google.com> Sat Aug 08 21:21:54 2009 -0700
committer Marc Blank <mblank@google.com> Sat Aug 08 21:23:28 2009 -0700
tree 5ced09e9c414b5cf494ee04e6f60e7d048a6e62c
parent f708e075473f4c186c44b61bc5ad5c73c901b61e

Make small changes as recommended by stadler (for CL9459)

src/com/android/exchange/EasSyncService.java

diff --git a/src/com/android/exchange/EasSyncService.java b/src/com/android/exchange/EasSyncService.java
index 47335ca..39f8cc4 100644
--- a/src/com/android/exchange/EasSyncService.java
+++ b/src/com/android/exchange/EasSyncService.java
@@ -146,7 +146,7 @@
     @Override
     public void stop() {
         mStop = true;
-     }
+    }
 
     @Override
     public int getSyncStatus() {
@@ -965,34 +965,34 @@
                 // If this is the account mailbox, wake up SyncManager
                 // Because this box has a "push" interval, it will be restarted immediately
                 // which will cause the folder list to be reloaded...
+                int status;
+                switch (mExitStatus) {
+                    case EXIT_IO_ERROR:
+                        status = EmailServiceStatus.CONNECTION_ERROR;
+                        break;
+                    case EXIT_DONE:
+                        status = EmailServiceStatus.SUCCESS;
+                        break;
+                    case EXIT_LOGIN_FAILURE:
+                        status = EmailServiceStatus.LOGIN_FAILED;
+                        break;
+                    default:
+                        status = EmailServiceStatus.REMOTE_EXCEPTION;
+                        break;
+                }
                 try {
-                    int status;
-                    switch (mExitStatus) {
-                        case EXIT_IO_ERROR:
-                            status = EmailServiceStatus.CONNECTION_ERROR;
-                            break;
-                        case EXIT_DONE:
-                            status = EmailServiceStatus.SUCCESS;
-                            break;
-                        case EXIT_LOGIN_FAILURE:
-                            status = EmailServiceStatus.LOGIN_FAILED;
-                            break;
-                        default:
-                            status = EmailServiceStatus.REMOTE_EXCEPTION;
-                            break;
-                    }
                     SyncManager.callback().syncMailboxStatus(mMailboxId, status, 0);
-
-                    // Save the sync time and status
-                    ContentValues cv = new ContentValues();
-                    cv.put(Mailbox.SYNC_TIME, System.currentTimeMillis());
-                    String s = "S" + mSyncReason + ':' + status + ':' + mChangeCount;
-                    cv.put(Mailbox.SYNC_STATUS, s);
-                    mContentResolver.update(ContentUris
-                            .withAppendedId(Mailbox.CONTENT_URI, mMailboxId), cv, null, null);
                 } catch (RemoteException e1) {
                     // Don't care if this fails
                 }
+
+                // Save the sync time and status
+                ContentValues cv = new ContentValues();
+                cv.put(Mailbox.SYNC_TIME, System.currentTimeMillis());
+                String s = "S" + mSyncReason + ':' + status + ':' + mChangeCount;
+                cv.put(Mailbox.SYNC_STATUS, s);
+                mContentResolver.update(ContentUris
+                        .withAppendedId(Mailbox.CONTENT_URI, mMailboxId), cv, null, null);
             } else {
                 userLog(mMailbox.mDisplayName + ": stopped thread finished.");
             }