From 9468c6a28e696f4ddca2c5b9ba312ffb590c3161 Mon Sep 17 00:00:00 2001 From: Alan Viverette Date: Thu, 21 Aug 2014 13:56:54 -0700 Subject: Clean up nullable listeners in date and time picker delegates Removes the NO_OP listener from TimePicker since we actually do null checks everywhere already. BUG: 17179502 Change-Id: I39a7d68253845c004027e31c59afc43be42195b7 --- core/java/android/widget/TimePickerClockDelegate.java | 12 ------------ 1 file changed, 12 deletions(-) (limited to 'core/java/android/widget/TimePickerClockDelegate.java') diff --git a/core/java/android/widget/TimePickerClockDelegate.java b/core/java/android/widget/TimePickerClockDelegate.java index 376e5b4..1b89179 100644 --- a/core/java/android/widget/TimePickerClockDelegate.java +++ b/core/java/android/widget/TimePickerClockDelegate.java @@ -73,16 +73,6 @@ class TimePickerClockDelegate extends TimePicker.AbstractTimePickerDelegate { private boolean mHourWithTwoDigit; private char mHourFormat; - /** - * A no-op callback used in the constructor to avoid null checks later in - * the code. - */ - private static final TimePicker.OnTimeChangedListener NO_OP_CHANGE_LISTENER = - new TimePicker.OnTimeChangedListener() { - public void onTimeChanged(TimePicker view, int hourOfDay, int minute) { - } - }; - public TimePickerClockDelegate(TimePicker delegator, Context context, AttributeSet attrs, int defStyleAttr, int defStyleRes) { super(delegator, context); @@ -216,8 +206,6 @@ class TimePickerClockDelegate extends TimePicker.AbstractTimePickerDelegate { updateMinuteControl(); updateAmPmControl(); - setOnTimeChangedListener(NO_OP_CHANGE_LISTENER); - // set to current time setCurrentHour(mTempCalendar.get(Calendar.HOUR_OF_DAY)); setCurrentMinute(mTempCalendar.get(Calendar.MINUTE)); -- cgit v1.1