From 9c12611dd348dd7ce095ba7c862dce8a9738a8fe Mon Sep 17 00:00:00 2001 From: Mathias-Boulay Date: Thu, 21 Nov 2024 20:19:28 +0100 Subject: [PATCH] Refactor(quick settings): remove generic leading to unchecked typecast --- .../src/main/java/com/kdt/SideDialogView.java | 7 +++---- .../customcontrols/handleview/EditControlSideDialog.java | 6 +++--- .../net/kdt/pojavlaunch/prefs/QuickSettingSideDialog.java | 2 +- 3 files changed, 7 insertions(+), 8 deletions(-) diff --git a/app_pojavlauncher/src/main/java/com/kdt/SideDialogView.java b/app_pojavlauncher/src/main/java/com/kdt/SideDialogView.java index 6b056051b..80b9cc86f 100644 --- a/app_pojavlauncher/src/main/java/com/kdt/SideDialogView.java +++ b/app_pojavlauncher/src/main/java/com/kdt/SideDialogView.java @@ -28,15 +28,14 @@ import net.kdt.pojavlaunch.Tools; /** * The base class for side dialog views * A side dialog is a dialog appearing from one side of the screen - * @param The type of the content view root */ -public abstract class SideDialogView { +public abstract class SideDialogView { private final ViewGroup mParent; private final @LayoutRes int mLayoutId; private ViewGroup mDialogLayout; private DefocusableScrollView mScrollView; - protected T mDialogContent; + protected View mDialogContent; protected final int mMargin; private ObjectAnimator mSideDialogAnimator; @@ -103,7 +102,7 @@ public abstract class SideDialogView { mTitleDivider = mDialogLayout.findViewById(R.id.side_dialog_title_divider); LayoutInflater.from(mParent.getContext()).inflate(mLayoutId, mScrollView, true); - mDialogContent = (T) mScrollView.getChildAt(0); + mDialogContent = mScrollView.getChildAt(0); // Attach layouts mParent.addView(mDialogLayout); diff --git a/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/customcontrols/handleview/EditControlSideDialog.java b/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/customcontrols/handleview/EditControlSideDialog.java index 171fdad7f..4e00c17c2 100644 --- a/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/customcontrols/handleview/EditControlSideDialog.java +++ b/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/customcontrols/handleview/EditControlSideDialog.java @@ -40,7 +40,7 @@ import net.kdt.pojavlaunch.utils.interfaces.SimpleTextWatcher; import java.util.List; -public class EditControlSideDialog extends SideDialogView { +public class EditControlSideDialog extends SideDialogView { private final Spinner[] mKeycodeSpinners = new Spinner[4]; public boolean internalChanges = false; // True when we programmatically change stuff. @@ -326,8 +326,8 @@ public class EditControlSideDialog extends SideDialogView { } private void setDefaultVisibilitySetting() { - for (int i = 0; i < mDialogContent.getChildCount(); ++i) { - mDialogContent.getChildAt(i).setVisibility(VISIBLE); + for (int i = 0; i < ((ViewGroup)mDialogContent).getChildCount(); ++i) { + ((ViewGroup)mDialogContent).getChildAt(i).setVisibility(VISIBLE); } for(Spinner s : mKeycodeSpinners) { s.setVisibility(View.INVISIBLE); diff --git a/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/prefs/QuickSettingSideDialog.java b/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/prefs/QuickSettingSideDialog.java index fc467ceed..a09017b6b 100644 --- a/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/prefs/QuickSettingSideDialog.java +++ b/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/prefs/QuickSettingSideDialog.java @@ -29,7 +29,7 @@ import net.kdt.pojavlaunch.utils.interfaces.SimpleSeekBarListener; * Side dialog for quick settings that you can change in game * The implementation has to take action on some preference changes */ -public abstract class QuickSettingSideDialog extends com.kdt.SideDialogView { +public abstract class QuickSettingSideDialog extends com.kdt.SideDialogView { private SharedPreferences.Editor mEditor; @SuppressLint("UseSwitchCompatOrMaterialCode")