Skip to content

Commit b9290f2

Browse files
committed
fix: remove unnecessary safe property access
these were non-null properties and do not need safe access, it is flagged as a warning (which we fail on with -Werror) in new kotlin
1 parent c0a4fe0 commit b9290f2

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

AnkiDroid/src/main/java/com/ichi2/anki/dialogs/customstudy/TagLimitFragment.kt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -132,8 +132,8 @@ class TagLimitFragment : DialogFragment() {
132132
override fun onStart() {
133133
super.onStart()
134134
lifecycleScope.launch {
135-
binding.loadingViews?.isVisible = true
136-
binding.contentViews?.isVisible = false
135+
binding.loadingViews.isVisible = true
136+
binding.contentViews.isVisible = false
137137
val customStudyDefaults = withCol { sched.customStudyDefaults(deckId) }
138138
binding.requireOneOrMoreCheckBox.isChecked =
139139
customStudyDefaults.tagsList.any { tag -> tag.include }
@@ -143,8 +143,8 @@ class TagLimitFragment : DialogFragment() {
143143
}
144144
tagsIncludedAdapter.tags = tags.toMutableList() // make a copy
145145
tagsExcludedAdapter.tags = tags.toMutableList() // make a copy
146-
binding.loadingViews?.isVisible = false
147-
binding.contentViews?.isVisible = true
146+
binding.loadingViews.isVisible = false
147+
binding.contentViews.isVisible = true
148148
}
149149
}
150150

AnkiDroid/src/test/java/com/ichi2/anki/AnalyticsTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ class AnalyticsTest {
117117

118118
// prepare analytics so we can inspect what happens
119119
val spyHit = spy(ExceptionHit())
120-
doReturn(spyHit).whenever(analytics)?.exception()
120+
doReturn(spyHit).whenever(analytics).exception()
121121
try {
122122
UsageAnalytics.sendAnalyticsException(grandparentException, false)
123123
} catch (e: Exception) {

0 commit comments

Comments
 (0)