Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fixed crash when the Bitmap size is zero in the ClusterRenderer #336

Merged

Conversation

kikoso
Copy link
Collaborator

@kikoso kikoso commented Jul 16, 2023

This PR prevents a crash when the Bitmap size is zero in the ClusterRenderer.

measuredWidth and measuredHeight typically return 0 when we call them from onCreate, which does not seem to be the case. This is also not deterministic and not always reproducible. For some reason, the view has not been drawn when we hit this point.

This PR will prevent the crash, although the causes need to be determined.

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #297

@kikoso kikoso requested a review from wangela July 16, 2023 14:06
@kikoso kikoso added the priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. label Jul 16, 2023
@kikoso kikoso force-pushed the fix/fixed_crash_when_bitmap_size_is_zero_in_clusterrenderer branch from c15716f to 016d7de Compare July 17, 2023 16:59
@wangela wangela merged commit 93b9b85 into main Jul 25, 2023
10 checks passed
@wangela wangela deleted the fix/fixed_crash_when_bitmap_size_is_zero_in_clusterrenderer branch July 25, 2023 16:25
googlemaps-bot pushed a commit that referenced this pull request Jul 25, 2023
## [2.11.5](v2.11.4...v2.11.5) (2023-07-25)

### Bug Fixes

* fixed crash when the Bitmap size is zero in the ClusterRenderer ([#336](#336)) ([93b9b85](93b9b85))
@wangela
Copy link
Member

wangela commented Jul 27, 2023

🎉 This PR is included in version 2.11.5 🎉

The release is available on:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clustering causes a crash in Navigation.
2 participants