Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7f0727af authored by Alexander Martinz's avatar Alexander Martinz
Browse files

LightSettingsDialog: remove unused OnColorChangedListener



We never set the listener but use it, wrapped with null checks.
Remove it.

Change-Id: I223ab27b6892c5809389a936a6ed7ea768bc3ae4
Signed-off-by: Alexander Martinz's avatarAlexander Martinz <amartinz@shiftphones.com>
parent c0af837e
Loading
Loading
Loading
Loading
+0 −10
Original line number Diff line number Diff line
@@ -49,7 +49,6 @@ import org.lineageos.internal.notification.LedValues;
import org.lineageos.internal.notification.LightsCapabilities;
import org.lineageos.internal.notification.LineageNotification;
import org.lineageos.lineageparts.R;
import org.lineageos.lineageparts.notificationlight.ColorPickerView.OnColorChangedListener;

import java.util.ArrayList;
import java.util.Locale;
@@ -73,8 +72,6 @@ public class LightSettingsDialog extends AlertDialog implements
    private Spinner mPulseSpeedOff;
    private LayoutInflater mInflater;

    private OnColorChangedListener mListener;

    private NotificationManager mNotificationManager;

    private boolean mReadyForLed;
@@ -243,10 +240,6 @@ public class LightSettingsDialog extends AlertDialog implements
        mNewColor.setColor(color);
        mHexColorInput.setText(String.format(Locale.US, format, color & mask));

        if (mListener != null) {
            mListener.onColorChanged(color);
        }

        updateLed();
    }

@@ -447,9 +440,6 @@ public class LightSettingsDialog extends AlertDialog implements
                mColorPicker.setColor(color);
                mNewColor.setColor(color);
                updateLed();
                if (mListener != null) {
                    mListener.onColorChanged(color);
                }
            } catch (IllegalArgumentException ex) {
                // Number format is incorrect, ignore
            }