Annotate some NonNulls, some lint cleaning
This commit is contained in:
parent
e29aaaf162
commit
cf213affa2
26 changed files with 55 additions and 52 deletions
|
|
@ -185,7 +185,7 @@ public class DownloadSettingsFragment extends BasePreferenceFragment {
|
|||
}
|
||||
|
||||
@Override
|
||||
public boolean onPreferenceTreeClick(final Preference preference) {
|
||||
public boolean onPreferenceTreeClick(@NonNull final Preference preference) {
|
||||
if (DEBUG) {
|
||||
Log.d(TAG, "onPreferenceTreeClick() called with: "
|
||||
+ "preference = [" + preference + "]");
|
||||
|
|
|
|||
|
|
@ -169,7 +169,7 @@ public class SettingsActivity extends AppCompatActivity implements
|
|||
}
|
||||
|
||||
@Override
|
||||
public boolean onPreferenceStartFragment(final PreferenceFragmentCompat caller,
|
||||
public boolean onPreferenceStartFragment(@NonNull final PreferenceFragmentCompat caller,
|
||||
final Preference preference) {
|
||||
showSettingsFragment(instantiateFragment(preference.getFragment()));
|
||||
return true;
|
||||
|
|
|
|||
|
|
@ -115,6 +115,7 @@ public final class SettingsResourceRegistry {
|
|||
return this;
|
||||
}
|
||||
|
||||
@NonNull
|
||||
public Class<? extends Fragment> getFragmentClass() {
|
||||
return fragmentClass;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -50,7 +50,7 @@ public class NotificationActionsPreference extends Preference {
|
|||
////////////////////////////////////////////////////////////////////////////
|
||||
|
||||
@Override
|
||||
public void onBindViewHolder(final PreferenceViewHolder holder) {
|
||||
public void onBindViewHolder(@NonNull final PreferenceViewHolder holder) {
|
||||
super.onBindViewHolder(holder);
|
||||
|
||||
holder.itemView.setClickable(false);
|
||||
|
|
|
|||
|
|
@ -4,6 +4,7 @@ import androidx.preference.PreferenceCategory;
|
|||
import androidx.preference.PreferenceScreen;
|
||||
|
||||
import java.util.Arrays;
|
||||
import java.util.Collections;
|
||||
import java.util.List;
|
||||
import java.util.Objects;
|
||||
import java.util.stream.Stream;
|
||||
|
|
@ -11,7 +12,7 @@ import java.util.stream.Stream;
|
|||
public class PreferenceSearchConfiguration {
|
||||
private PreferenceSearchFunction searcher = new PreferenceFuzzySearchFunction();
|
||||
|
||||
private final List<String> parserIgnoreElements = Arrays.asList(
|
||||
private final List<String> parserIgnoreElements = Collections.singletonList(
|
||||
PreferenceCategory.class.getSimpleName());
|
||||
private final List<String> parserContainerElements = Arrays.asList(
|
||||
PreferenceCategory.class.getSimpleName(),
|
||||
|
|
|
|||
|
|
@ -58,22 +58,27 @@ public class PreferenceSearchItem {
|
|||
this.searchIndexItemResId = searchIndexItemResId;
|
||||
}
|
||||
|
||||
@NonNull
|
||||
public String getKey() {
|
||||
return key;
|
||||
}
|
||||
|
||||
@NonNull
|
||||
public String getTitle() {
|
||||
return title;
|
||||
}
|
||||
|
||||
@NonNull
|
||||
public String getSummary() {
|
||||
return summary;
|
||||
}
|
||||
|
||||
@NonNull
|
||||
public String getEntries() {
|
||||
return entries;
|
||||
}
|
||||
|
||||
@NonNull
|
||||
public String getBreadcrumbs() {
|
||||
return breadcrumbs;
|
||||
}
|
||||
|
|
@ -94,7 +99,7 @@ public class PreferenceSearchItem {
|
|||
getBreadcrumbs());
|
||||
}
|
||||
|
||||
|
||||
@NonNull
|
||||
@Override
|
||||
public String toString() {
|
||||
return "PreferenceItem: " + title + " " + summary + " " + key;
|
||||
|
|
|
|||
|
|
@ -340,7 +340,7 @@ public abstract class Tab {
|
|||
@DrawableRes
|
||||
@Override
|
||||
public int getTabIconRes(final Context context) {
|
||||
final int kioskIcon = KioskTranslator.getKioskIcon(kioskId, context);
|
||||
final int kioskIcon = KioskTranslator.getKioskIcon(kioskId);
|
||||
|
||||
if (kioskIcon <= 0) {
|
||||
throw new IllegalStateException("Kiosk ID is not valid: \"" + kioskId + "\"");
|
||||
|
|
@ -496,7 +496,7 @@ public abstract class Tab {
|
|||
@DrawableRes
|
||||
@Override
|
||||
public int getTabIconRes(final Context context) {
|
||||
return KioskTranslator.getKioskIcon(getDefaultKioskId(context), context);
|
||||
return KioskTranslator.getKioskIcon(getDefaultKioskId(context));
|
||||
}
|
||||
|
||||
@Override
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue