findbug fixes

	modified:   src/com/android/calendar/AgendaActivity.java
	modified:   src/com/android/calendar/AgendaAdapter.java
	modified:   src/com/android/calendar/CalendarActivity.java
	modified:   src/com/android/calendar/CalendarApplication.java
	modified:   src/com/android/calendar/CalendarView.java
	modified:   src/com/android/calendar/DeleteEventHelper.java
	modified:   src/com/android/calendar/EditEvent.java
	modified:   src/com/android/calendar/Event.java
	modified:   src/com/android/calendar/EventInfoActivity.java
	modified:   src/com/android/calendar/IcsImportActivity.java
	modified:   src/com/android/calendar/MonthActivity.java
	modified:   tests/src/com/android/calendar/FormatDateRangeTest.java
	modified:   tests/src/com/android/calendar/WeekNumberTest.java

13 files changed, 12 insertions, 18 deletions

diff --git a/src/com/android/calendar/AgendaActivity.java b/src/com/android/calendar/AgendaActivity.java
index 97a6e5a7..9f5a53ed 100644
--- a/src/com/android/calendar/AgendaActivity.java
+++ b/src/com/android/calendar/AgendaActivity.java
@@ -204,7 +204,7 @@ public class AgendaActivity extends Activity implements ViewSwitcher.ViewFactory
 
         // Eliminate extra GCs during startup by setting the initial heap size to 4MB.
         // TODO: We should restore the old heap size once the activity reaches the idle state
-        long oldHeapSize = VMRuntime.getRuntime().setMinimumHeapSize(INITIAL_HEAP_SIZE);
+        VMRuntime.getRuntime().setMinimumHeapSize(INITIAL_HEAP_SIZE);
 
         setContentView(R.layout.agenda_activity);
 
diff --git a/src/com/android/calendar/AgendaAdapter.java b/src/com/android/calendar/AgendaAdapter.java
index 1b93b7e2..69649eaf 100644
--- a/src/com/android/calendar/AgendaAdapter.java
+++ b/src/com/android/calendar/AgendaAdapter.java
@@ -27,7 +27,7 @@ import android.widget.ResourceCursorAdapter;
 import android.widget.TextView;
 
 public class AgendaAdapter extends ResourceCursorAdapter {
-    static private String mNoTitleLabel; // todo update on locale change.
+    private String mNoTitleLabel;
     private Resources mResources;
     private int mDeclinedColor;
 
diff --git a/src/com/android/calendar/CalendarActivity.java b/src/com/android/calendar/CalendarActivity.java
index e2621c60..f245c2bb 100644
--- a/src/com/android/calendar/CalendarActivity.java
+++ b/src/com/android/calendar/CalendarActivity.java
@@ -100,7 +100,7 @@ public class CalendarActivity extends Activity implements Navigator {
 
         // Eliminate extra GCs during startup by setting the initial heap size to 4MB.
         // TODO: We should restore the old heap size once the activity reaches the idle state
-        long oldHeapSize = VMRuntime.getRuntime().setMinimumHeapSize(INITIAL_HEAP_SIZE);
+        VMRuntime.getRuntime().setMinimumHeapSize(INITIAL_HEAP_SIZE);
 
         setDefaultKeyMode(DEFAULT_KEYS_SHORTCUT);
         mContentResolver = getContentResolver();
diff --git a/src/com/android/calendar/CalendarApplication.java b/src/com/android/calendar/CalendarApplication.java
index a3dad951..424e4229 100644
--- a/src/com/android/calendar/CalendarApplication.java
+++ b/src/com/android/calendar/CalendarApplication.java
@@ -32,7 +32,7 @@ public class CalendarApplication extends Application {
      * points to the first element in the list (the most recently visited
      * screen).
      */
-    /* package */ class Screen {
+    static class Screen {
         public int id;
         public Screen next;
         public Screen previous;
diff --git a/src/com/android/calendar/CalendarView.java b/src/com/android/calendar/CalendarView.java
index 38185a1c..16142995 100644
--- a/src/com/android/calendar/CalendarView.java
+++ b/src/com/android/calendar/CalendarView.java
@@ -103,7 +103,7 @@ public class CalendarView extends View
 
     private ContinueScroll mContinueScroll = new ContinueScroll();
 
-    private class DayHeader{
+    static private class DayHeader{
         int cell;
         String dateString;
     }
diff --git a/src/com/android/calendar/DeleteEventHelper.java b/src/com/android/calendar/DeleteEventHelper.java
index d0693fa9..bf27a9e5 100644
--- a/src/com/android/calendar/DeleteEventHelper.java
+++ b/src/com/android/calendar/DeleteEventHelper.java
@@ -52,8 +52,6 @@ import android.widget.Button;
  * {@link #delete()} multiple times).
  */
 public class DeleteEventHelper {
-    
-    private static final String TAG = "DeleteEventHelper";
     private final Activity mParent;
     private final ContentResolver mContentResolver;
     
diff --git a/src/com/android/calendar/EditEvent.java b/src/com/android/calendar/EditEvent.java
index 1c69b65a..86097d39 100644
--- a/src/com/android/calendar/EditEvent.java
+++ b/src/com/android/calendar/EditEvent.java
@@ -350,7 +350,7 @@ public class EditEvent extends Activity implements View.OnClickListener,
         }
     }
 
-    private class CalendarsAdapter extends ResourceCursorAdapter {
+    static private class CalendarsAdapter extends ResourceCursorAdapter {
         public CalendarsAdapter(Context context, Cursor c) {
             super(context, R.layout.calendars_item, c);
             setDropDownViewResource(R.layout.calendars_dropdown_item);
diff --git a/src/com/android/calendar/Event.java b/src/com/android/calendar/Event.java
index c15e5b54..9061cda2 100644
--- a/src/com/android/calendar/Event.java
+++ b/src/com/android/calendar/Event.java
@@ -113,7 +113,8 @@ public class Event implements Comparable, Cloneable {
     private static final int MIDNIGHT_IN_MINUTES = 24 * 60;
 
     @Override
-    public final Object clone() {
+    public final Object clone() throws CloneNotSupportedException {
+        super.clone();
         Event e = new Event();
 
         e.title = title;
diff --git a/src/com/android/calendar/EventInfoActivity.java b/src/com/android/calendar/EventInfoActivity.java
index 48702bc9..ec684a46 100644
--- a/src/com/android/calendar/EventInfoActivity.java
+++ b/src/com/android/calendar/EventInfoActivity.java
@@ -547,7 +547,7 @@ public class EventInfoActivity extends Activity implements View.OnClickListener,
             values.put(Events.SELF_ATTENDEE_STATUS, status);
             
             // Create a recurrence exception
-            Uri newUri = cr.insert(Events.CONTENT_URI, values);
+            cr.insert(Events.CONTENT_URI, values);
         } finally {
             cursor.close();
         }
@@ -582,7 +582,6 @@ public class EventInfoActivity extends Activity implements View.OnClickListener,
             return;
         }
         Resources res = getResources();
-        ContentResolver cr = getContentResolver();
 
         String eventName = mEventCursor.getString(EVENT_INDEX_TITLE);
         if (eventName == null || eventName.length() == 0) {
diff --git a/src/com/android/calendar/IcsImportActivity.java b/src/com/android/calendar/IcsImportActivity.java
index c4f5fdea..c5a125df 100644
--- a/src/com/android/calendar/IcsImportActivity.java
+++ b/src/com/android/calendar/IcsImportActivity.java
@@ -60,11 +60,9 @@ public class IcsImportActivity extends Activity {
         }
     }
 
-    private View mView;
     private Button mImportButton;
     private Button mCancelButton;
     private Spinner mCalendars;
-    private ImageView mCalendarIcon;
     private TextView mNumEvents;
 
     private ICalendar.Component mCalendar = null;
@@ -86,9 +84,7 @@ public class IcsImportActivity extends Activity {
     protected void onCreate(Bundle icicle) {
         super.onCreate(icicle);
         setContentView(R.layout.ics_import_activity);
-        mView = findViewById(R.id.import_ics);
 
-        mCalendarIcon = (ImageView) findViewById(R.id.calendar_icon);
         mCalendars = (Spinner) findViewById(R.id.calendars);
         populateCalendars();
 
diff --git a/src/com/android/calendar/MonthActivity.java b/src/com/android/calendar/MonthActivity.java
index 89b07eca..9bb86674 100644
--- a/src/com/android/calendar/MonthActivity.java
+++ b/src/com/android/calendar/MonthActivity.java
@@ -195,7 +195,7 @@ public class MonthActivity extends Activity implements ViewSwitcher.ViewFactory,
 
         // Eliminate extra GCs during startup by setting the initial heap size to 4MB.
         // TODO: We should restore the old heap size once the activity reaches the idle state
-        long oldHeapSize = VMRuntime.getRuntime().setMinimumHeapSize(INITIAL_HEAP_SIZE);
+        VMRuntime.getRuntime().setMinimumHeapSize(INITIAL_HEAP_SIZE);
 
         setContentView(R.layout.month_activity);
         mContentResolver = getContentResolver();
diff --git a/tests/src/com/android/calendar/FormatDateRangeTest.java b/tests/src/com/android/calendar/FormatDateRangeTest.java
index 6f0d775d..548fc1c8 100644
--- a/tests/src/com/android/calendar/FormatDateRangeTest.java
+++ b/tests/src/com/android/calendar/FormatDateRangeTest.java
@@ -30,7 +30,7 @@ import java.util.Calendar;
  */
 public class FormatDateRangeTest extends AndroidTestCase {
 
-    private class DateTest {
+    static private class DateTest {
         public Time date1;
         public Time date2;
         public int flags;
diff --git a/tests/src/com/android/calendar/WeekNumberTest.java b/tests/src/com/android/calendar/WeekNumberTest.java
index 6b3774b7..01f5746c 100644
--- a/tests/src/com/android/calendar/WeekNumberTest.java
+++ b/tests/src/com/android/calendar/WeekNumberTest.java
@@ -29,7 +29,7 @@ import android.util.Log;
  */
 public class WeekNumberTest extends AndroidTestCase {
 
-    private class DateAndWeekNumber {
+    static private class DateAndWeekNumber {
         public Time date;
         public Time allDayDate;
         public int expectedWeekNumber;