CastError - Null check operator used on a null value

[larssn]

positioned_tap_detector_2.dart in _TapPositionDetectorState._onLongPressEvent at line 117 within positioned_tap_detector_2

It seems _pendingTap can be null occasionally when _onLongPressEvent is called. Instead of using the bang operator, wouldn't it be better to just use actual null checks?

[arsamme]

@larssn
Hi, As I don't have free time to work on side projects, Would you please solve this and create PR?

[larssn]

Same, very busy. If I get time, I'll squeeze in a PR.

[greensopinion]

I had the same issue:

Null check operator used on a null value
#0      _TapPositionDetectorState._onLongPressEvent (package:positioned_tap_detector_2/positioned_tap_detector_2.dart:117)
#1      PositionedTapController.onLongPress (package:positioned_tap_detector_2/positioned_tap_detector_2.dart:179)
#2      GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:182)
#3      LongPressGestureRecognizer._checkLongPressStart (package:flutter/src/gestures/long_press.dart:423)
#4      LongPressGestureRecognizer.didExceedDeadline (package:flutter/src/gestures/long_press.dart:372)
#5      PrimaryPointerGestureRecognizer.didExceedDeadlineWithEvent (package:flutter/src/gestures/recognizer.dart:501)
#6      PrimaryPointerGestureRecognizer.addAllowedPointer.<anonymous closure> (package:flutter/src/gestures/recognizer.dart:454)
#7      _rootRun (dart:async/zone.dart:1346)
#8      _CustomZone.run (dart:async/zone.dart:1258)

The PR above should address it.

[arsamme]

@greensopinion Thank you, I've merged your PR and I'll release it on pub soon

[greensopinion]

Thank you. When this gets published we will consume it in flutter_map

…

On Wed, Sep 29, 2021 at 12:22 AM Arsam ***@***.***> wrote: @greensopinion <https://github.com/greensopinion> Thank you, I've merged your PR and I'll release it on pub soon — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEKDQ7SGFTYXJM7MFZL6CN3UEK5CXANCNFSM463BXIGQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[arsamme]

@greensopinion sorry for updating too late. done. I'll close this issue.