code review comments

MyTracks/src/com/google/android/apps/mytracks/WriteProgressController.java

@@ -88,26 +88,26 @@ class WriteProgressController {
   }
 
   private final TrackWriter.OnCompletionListener writerCompleteListener =
-    new TrackWriter.OnCompletionListener() {
+      new TrackWriter.OnCompletionListener() {
-      @Override
+        @Override
-      public void onComplete() {
+        public void onComplete() {
-        DialogManager.dismissDialogSafely(activity, dialog);
+          DialogManager.dismissDialogSafely(activity, dialog);
 
-        if (onCompletionListener != null) {
+          if (onCompletionListener != null) {
-          onCompletionListener.onComplete(writer);
+            onCompletionListener.onComplete(writer);
+          }
         }
-      }
+      };
-    };
 
   private final TrackWriter.OnWriteListener writerWriteListener =
-    new TrackWriter.OnWriteListener() {
+      new TrackWriter.OnWriteListener() {
-      @Override
+        @Override
-      public void onWrite(int number, int max) {
+        public void onWrite(int number, int max) {
-        if (number % 500 == 0) {
+          if (number % 500 == 0) {
-          dialog.setIndeterminate(false);
+            dialog.setIndeterminate(false);
-          dialog.setMax(max);
+            dialog.setMax(max);
-          dialog.setProgress(Math.min(number, max));
+            dialog.setProgress(Math.min(number, max));
+          }
         }
-      }
+      };
-    };
 }

MyTracksTest/src/com/google/android/apps/mytracks/WriteProgressControllerTest.java

@@ -24,6 +24,9 @@ import android.test.ActivityInstrumentationTestCase2;
 import java.util.concurrent.Semaphore;
 import java.util.concurrent.atomic.AtomicReference;
 
+/**
+ * @author Matthew Simmons
+ */
 public class WriteProgressControllerTest extends ActivityInstrumentationTestCase2<MyTracks> {
   public WriteProgressControllerTest() {
     super(MyTracks.class);

MyTracksTest/src/com/google/android/apps/mytracks/io/file/MockTrackWriter.java

@@ -41,6 +41,7 @@ public class MockTrackWriter implements TrackWriter {
 
   @Override
   public void setDirectory(File directory) {
+    throw new RuntimeException("not implemented");
   }
 
   @Override