Telephoto internal NPE

[revonateB0T]

FATAL EXCEPTION: main
Process: moe.tarsin.ehviewer.debug, PID: 13059
java.lang.NullPointerException
	at me.saket.telephoto.zoomable.RealZoomableState.fling-BMRW4eQ$zoomable_release(RealZoomableState.kt:502)
	at me.saket.telephoto.zoomable.ZoomableNode$onTransformStopped$1$1.invokeSuspend(Zoomable.kt:138)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:104)
	at androidx.compose.ui.platform.AndroidUiDispatcher.performTrampolineDispatch(AndroidUiDispatcher.android.kt:81)
	at androidx.compose.ui.platform.AndroidUiDispatcher.access$performTrampolineDispatch(AndroidUiDispatcher.android.kt:41)
	at androidx.compose.ui.platform.AndroidUiDispatcher$dispatchCallback$1.run(AndroidUiDispatcher.android.kt:57)
	at android.os.Handler.handleCallback(Handler.java:958)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loopOnce(Looper.java:205)
	at android.os.Looper.loop(Looper.java:294)
	at android.app.ActivityThread.main(ActivityThread.java:8296)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:978)
	Suppressed: kotlinx.coroutines.internal.DiagnosticCoroutineContextException: [androidx.compose.ui.platform.MotionDurationScaleImpl@3a1253d, androidx.compose.runtime.BroadcastFrameClock@f23a132, StandaloneCoroutine{Cancelling}@bf50e83, AndroidUiDispatcher@e154800]

[revonateB0T]

FooIbar/EhViewer#832

[revonateB0T]

telephoto/zoomable/src/commonMain/kotlin/me/saket/telephoto/zoomable/RealZoomableState.kt

Line 502 in 89bedb5

val start = gestureState!!

[saket]

This is so weird. There are only 2 possible situations when gestureState can be null:

1. No gesture has been recorded yet
2. ZoomableState#resetZoom() was called

If a fling was detected then 1 should be impossible. 2 also sounds unlikely because I wasn't able to find any calls to ZoomableState#resetZoom() in your codebase.

[revonateB0T]

ZoomableNode is reused in LazyLayout

telephoto/zoomable/src/commonMain/kotlin/me/saket/telephoto/zoomable/Zoomable.kt

Line 104 in 89bedb5

private class ZoomableNode(

We should update its state on update

telephoto/zoomable/src/commonMain/kotlin/me/saket/telephoto/zoomable/Zoomable.kt

Line 167 in 89bedb5

fun update(

telephoto/zoomable/src/commonMain/kotlin/me/saket/telephoto/zoomable/Zoomable.kt

Line 105 in 89bedb5

private var state: RealZoomableState,

fun update(
    state: RealZoomableState,
    enabled: Boolean,
    onClick: ((Offset) -> Unit)?,
    onLongClick: ((Offset) -> Unit)?,
  ) {
+  state = state
    transformableNode.update(
      state = state.transformableState,
      canPan = state::canConsumePanChange,
      lockRotationOnZoomPan = false,
      enabled = enabled,
      onTransformStopped = onTransformStopped,
    )
    tappableAndQuickZoomableNode.update(
      onPress = onPress,
      onTap = onClick,
      onLongPress = onLongClick,
      onDoubleTap = onDoubleTap,
      onQuickZoomStopped = onQuickZoomStopped,
      transformableState = state.transformableState,
      gesturesEnabled = enabled,
    )
  }

Then onTransformStopped can callback correct RealZoomableState

telephoto/zoomable/src/commonMain/kotlin/me/saket/telephoto/zoomable/Zoomable.kt

Line 138 in 89bedb5

state.fling(velocity = velocity, density = requireDensity())

@saket

[saket]

Good catch! Let me fix this. Are you able to consistently reproduce this crash?

[saket]

1749177 was deployed to maven. Can you please try out 0.9.0-SNAPSHOT?

[revonateB0T]

Good catch! Let me fix this. Are you able to consistently reproduce this crash?

Fixed in 0.9.0-SNAPSHOT, 👍