Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: Fix crash when accessing planesCount or bytesPerRow #380

Merged
merged 6 commits into from
Aug 20, 2021

Conversation

mrousavy
Copy link
Owner

What

For whatever reason calling getPlanes() from C++ didn't work. No idea what I did wrong. I now just call it from a Java abstraction layer (ImageProxyUtils.java), and it works.

Changes

Tested on

Related issues

@mrousavy mrousavy added 🐛 bug Something isn't working 🤖 android Issue affects the Android platform labels Aug 20, 2021
@mrousavy mrousavy merged commit 0d3f2cb into main Aug 20, 2021
@mrousavy mrousavy deleted the fix/fix-planeproxy-crash branch August 20, 2021 14:02
isaaccolson pushed a commit to isaaccolson/deliveries-mobile that referenced this pull request Oct 30, 2024
…y#380)

* fix: Fix `bytesPerRow` and `planeCount` crashing

* use direct call

* Call `getPlanesCount` and `getBytesPerRow` through java

* revert camerapage changes

* remove removed header import

* Update CameraPage.tsx
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🤖 android Issue affects the Android platform 🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant