commit 1e34a1bead76f4c3fa6e2baec327e5846f4c23fd
author Robyn Coultas <coultasr@android.com> Wed Dec 05 17:12:41 2012 -0800
committer Isaac Katzenelson <isaack@android.com> Wed Dec 05 17:46:18 2012 -0800
tree 77a5206b326bb17d6dc9885f7476d91dfec01f37
parent 68898395ed6315a0f54929d0a6b82abfc4829b36

Always log the alarm time

Bug: 7579100
Bug: 7634943

Intentionally verbose to provide useful information in bug reports.
Tuned format of alarm time to include day of week

Change-Id: I54fb5ee75c28868adf98c0d9d1f3fb25f55a6af6

src/com/android/deskclock/Alarms.java

diff --git a/src/com/android/deskclock/Alarms.java b/src/com/android/deskclock/Alarms.java
index 93c1ec9..637b824 100644
--- a/src/com/android/deskclock/Alarms.java
+++ b/src/com/android/deskclock/Alarms.java
@@ -426,9 +426,9 @@
         AlarmManager am = (AlarmManager)
                 context.getSystemService(Context.ALARM_SERVICE);
 
-        if (Log.LOGV) {
-            Log.v("** setAlert id " + alarm.id + " atTime " + atTimeInMillis);
-        }
+        // Intentionally verbose: always log the alarm time to provide useful
+        // information in bug reports.
+        Log.v("Alarm set for id=" + alarm.id + " " + Log.formatTime(atTimeInMillis));
 
         Intent intent = new Intent(ALARM_ALERT_ACTION);
 
@@ -472,6 +472,9 @@
                 PendingIntent.FLAG_CANCEL_CURRENT);
         am.cancel(sender);
         setStatusBarIcon(context, false);
+        // Intentionally verbose: always log the lack of a next alarm to provide useful
+        // information in bug reports.
+        Log.v("No next alarm");
         saveNextAlarm(context, "");
     }
 

src/com/android/deskclock/Log.java

diff --git a/src/com/android/deskclock/Log.java b/src/com/android/deskclock/Log.java
index 39e7e34..e075442 100644
--- a/src/com/android/deskclock/Log.java
+++ b/src/com/android/deskclock/Log.java
@@ -55,6 +55,6 @@
     }
 
     public static String formatTime(long millis) {
-        return new SimpleDateFormat("HH:mm:ss.SSS aaa").format(new Date(millis));
+        return new SimpleDateFormat("HH:mm:ss.SSS/E").format(new Date(millis));
     }
 }