Details

Reviewers

• karol
ashoat
• jim

Commits

rCOMMd3f0edd1bcc5: [native] Introduce a JNI network module helper

Summary

This diff makes it possible to expose C++ static methods to java. A first example of such a method will be added in the next diff.

Test Plan

Compile the app - it should not crash.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Feb 8 2022, 6:50 AM

Herald added subscribers: atul, • adrian, ashoat. · View Herald TranscriptFeb 8 2022, 6:50 AM

Missing newline

tomek added inline comments.Feb 8 2022, 6:55 AM

native/android/app/src/cpp/jsiInstaller.cpp
64–69 ↗	(On Diff #9394)	When I tried to have a separate method `JNI_OnLoad` in `GlobalNetworkSingletonJNIHelper` it was not possible to locate the exposed method (unsatisfied link), so it looks like we need to have only one `JNI_OnLoad` per library. I could verify that, but I don't think that having a single point where all the natives are registered is a bad thing - it reduces the chances of conflicts, or make finding conflicts easier.

Harbormaster completed remote builds in B6628: Diff 9393.Feb 8 2022, 6:59 AM

Harbormaster completed remote builds in B6629: Diff 9394.Feb 8 2022, 7:04 AM

tomek requested review of this revision.Feb 8 2022, 7:04 AM

tomek planned changes to this revision.Feb 8 2022, 9:29 AM

Update

Harbormaster completed remote builds in B6670: Diff 9441.Feb 9 2022, 5:01 AM

• karol accepted this revision.Feb 9 2022, 5:05 AM

• karol added inline comments.

native/android/app/src/cpp/GlobalNetworkSingletonJNIHelper.cpp
5–6 ↗	(On Diff #9441)	nit

This revision is now accepted and ready to land.Feb 9 2022, 5:05 AM

Herald added a reviewer: ashoat. · View Herald TranscriptFeb 9 2022, 5:05 AM

Herald added a subscriber: • benschac. · View Herald Transcript

This revision now requires review to proceed.Feb 9 2022, 5:05 AM

• karol mentioned this in D3134: [native] Send PONG when PING is received.Feb 9 2022, 5:07 AM

I really don't know much about what's going on here... @karol-bisztyga and @palys-swm are the only ones on the team who have really explored Android JNI generally or fbjni specifically. Adding @jimpo since I know he's done some Android stuff, but not sure if he has context. Will accept, deferring to @karol-bisztyga and @palys-swm.