-
Notifications
You must be signed in to change notification settings - Fork 19
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
Run Build failed on Android: error: no matching constructor for initialization of <<Object>> #259
Comments
Please paste the small repro code here. |
result:
HybridObject:
On iOS it's completely normal. Smooth. |
Created a fix here: #260 |
Thanks @mrousavy . Please release it 🙏🙏🙏 |
have some patience. 🙂 |
Yah sir !!! |
Oh @mrousavy . |
Hm, can you reproduce this error in the example app here please? I created this example nitro/packages/react-native-nitro-image/src/specs/TestObject.nitro.ts Lines 21 to 31 in 1718912
This one has a |
This is what got generated - a JNI binding to nitro/packages/react-native-nitro-image/nitrogen/generated/android/c++/JCar.hpp Lines 35 to 61 in 1718912
|
My code:
Code gen:
|
@baronha again; can you reproduce this error in the example app here please? |
Error:
|
Okay got it. I fixed it here now: #283 |
Thanks for testing and thanks for posting logs. It would've helped more if you tried to add a SIMPLE reproduction in the But I figured it out anyways now so just for next time keep that in mind |
released in Nitro 0.14.1 :) |
Bravo. This is amazing, Marc 🎉🎉🎉 !!! |
What's happening?
/Users/mac/project/react-native-multiple-image-picker/android/../nitrogen/generated/shared/c++/Result.hpp:33:10: note: candidate constructor (the implicit copy constructor) not viable: requires 1 argument, but 15 were provided
struct Result {
...
My code: https://github.com/baronha/react-native-multiple-image-picker/tree/nitro
PR: baronha/react-native-multiple-image-picker#173
Reproduceable Code
Relevant log output
Device
Emulator Android
Nitro Modules Version
0.13.0
Nitrogen Version
Can you reproduce this issue in the Nitro Example app here?
Yes, I can reproduce the same issue in the Example app here
Additional information
The text was updated successfully, but these errors were encountered: