Skip to content

Commit ec9ac81

Browse files
mdvaccameta-codesync[bot]
authored andcommitted
Fix UnsafeDereference: Remove !! operators in FpsView (#56115)
Summary: Pull Request resolved: #56115 Fixed UnsafeDereference lint warnings in FpsView.kt. Changed the `reactContext` parameter type from `ReactContext?` to `ReactContext` since the superclass FrameLayout requires a non-null Context anyway. This eliminates the need for !! operators. changelog: [internal] internal Reviewed By: alanleedev Differential Revision: D96784792 fbshipit-source-id: 7409bba7037775d24ddc83fa7981a5ad7aa86fc7
1 parent aab8df3 commit ec9ac81

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/FpsView.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,15 @@ import java.util.Locale
2323
*
2424
* NB: Requires API 16 for use of FpsDebugFrameCallback.
2525
*/
26-
internal class FpsView(reactContext: ReactContext?) : FrameLayout(reactContext!!) {
26+
internal class FpsView(reactContext: ReactContext) : FrameLayout(reactContext) {
2727
private val textView: TextView
2828
private val frameCallback: FpsDebugFrameCallback
2929
private val fpsMonitorRunnable: FPSMonitorRunnable
3030

3131
init {
3232
inflate(reactContext, R.layout.fps_view, this)
3333
textView = findViewById<View>(R.id.fps_text) as TextView
34-
frameCallback = FpsDebugFrameCallback(reactContext!!)
34+
frameCallback = FpsDebugFrameCallback(reactContext)
3535
fpsMonitorRunnable = FPSMonitorRunnable()
3636
setCurrentFPS(0.0, 0.0, 0, 0, frameCallback.isRunningOnFabric)
3737
}

0 commit comments

Comments
 (0)