Correct Rust -> Java method call semantics to avoid double-conversion

Fix cloning trait structs via native clone method

When cloning a trait struct, we need to reference the
already-converted-to-struct value, not the original, integer,
value.

Merge pull request #33 from TheBlueMatt/main

Expose supertrait implemetations to the Java API

Update auto-generated bindings to latest upstream

Note that the inclusion of enum constructor functions fixes #30.

Drop fail-fast in CI to run jobs even after another fails

Move release jar comparisons to run before functional tests in CI

Automatically run diffoscope on changed compressed files in CI

Use the 0.0.99 branch of ldk-c-bindings in CI

Use explicit paths for function arguments in Java

When we have an enum with a variant named the same as a struct we
need to specify a full concrete path when referring to it. The
simplest approach is to just use full concrete paths for all
function arguments, which we do here.

Add an accessor to UInt5 to get the value and correct conversion

Test function returning Option::None returns null to Java

Use new TxOut constructor in tests

Add constructors and expose fields in TxOut

Fixes #34.

Add a simple wrapper for `disconnect_by_node_id` in `NioPeerHandler`

Fixes #36.

Add docs for and mark nullable complex enum fields

Correctly handle passing a null to Rust for an array

Mark function arguments and return values Nullable if relevant

Fixes #35.

Correctly handle null pointers returned from Rust representing None

Fixes the SEGFAULT in #35.

Use findbugs for Nullable annotations instead of jetbrains

Use a full Sign wrapper again in HumanObjectPeerTest

With the new upstream clone API and this, memory leaks during tests are
now:
  418 allocations remained for 1160528 bytes.

Expose supertrait implemetations to the Java API

Support new cloned upstream API instead of clone

Free constructed LDK strings when relevant.

With LDK C Bindings v0.0.99.2, this reduces memory leaks during tests from:
  538 allocations remained for 1188720 bytes.
to
  358 allocations remained for 1141342 bytes.

Deliver messages in PeerTest more robustly

Merge pull request #31 from TheBlueMatt/main

Build deterministic Android AARs, Test built OSX bins

Update auto-generated bindings to 0.0.99.1

Add script to build deterministic android aar

Enable macos-11 in CI

Don't explicitly override __cxa_thread_atexit_impl, rely on LLVM to optimize it out

Merge pull request #29 from TheBlueMatt/main

Update to LDK 0.0.99

Update auto-generated bindings to LDK v0.0.99

Drop disconnect_socket blocking in NioPeerHandler for 0.0.99

0.0.99 no longer requires that we block disconnect_socket and
simplifes our side of the SocketDescriptor interface substantially,
which we implement here.

Merge pull request #28 from TheBlueMatt/main

Provide Better Diagnostics when a Java method called from Rust panics

Update auto-generated bindings with new error messages and version

Test that channel_manager_bytes is not null when persisting

We had an apparently-spurious report that this was the case, though
it doesn't seem to be.

Specify the method which threw an exception when aborting

Merge pull request #25 from TheBlueMatt/main

Fix glibc compat tweaks, make jars deterministic

Bump auto-generated bindings version to 0.0.98.4

Drop binaries from the repo, they are now at https://git.bitcoin.ninja/index.cgi?p=ldk-java-bins;a=summary

Redo the glibc compatibility issues workaround, see comment for details

Deterministically build the jars and check in CI, from a new repo

Add a utility script to make release jars deterministic

Set a library suffix for all platforms, useful for android builds

Merge pull request #24 from TheBlueMatt/main

Support glibc 2.17 and make exception failures explicit

Update auto-generated bindings

Handle Java exceptions thrown in calls from C, printing and aborting

Closes #23.

Support glibc versions prior to 2.18 for linux builds

Merge pull request #22 from TheBlueMatt/main

Support building on OSX

Re-generate auto-generated bindings + add Mac OS generated bindings

Drop the liblightningjni link, creating it in CI instead

Add a CI run on OSX

Build a test jar which we can run to test the built jar is runnable

Include native libraries in jar by shifting folder structure

Drop most instances of Thread.sleep in HumanObjectPeerTest

Do not rely on assertions being enabled in ChannelManagerConstructor

Update README with notes about OSX addrsan

Adjust compile flags for OSX, set trace on genbindings.sh

Skip manual leak checking on OSX as -Wl,-wrap is too complicated

Support OSX (ie builds where int64_t is not an alias for jlong)

Drop _test_only_get_ptr as it is no longer used

Merge pull request #21 from TheBlueMatt/main

Android Fixes

Update auto-generated bindings with new release tag

Avoid use of covariant return type change fns fixing backwards compat

See, eg, https://jira.mongodb.org/browse/JAVA-2559. This results
in errors like the following:

```
2021-06-12 17:15:58.893 13340-13412/com.example.rnldk E/AndroidRuntime: FATAL EXCEPTION: NioPeerHandler NIO Thread
    Process: com.example.rnldk, PID: 13340
    java.lang.NoSuchMethodError: No virtual method clear()Ljava/nio/ByteBuffer; in class Ljava/nio/ByteBuffer; or its super classes (declaration of 'java.nio.ByteBuffer' appears in /system/framework/core-oj.jar)
        at org.ldk.batteries.NioPeerHandler.lambda$new$0$NioPeerHandler(NioPeerHandler.java:185)
        at org.ldk.batteries.-$$Lambda$NioPeerHandler$f9v_DKRzmlYYXidLTlb3OtA-tVY.run(Unknown Source:2)
        at java.lang.Thread.run(Thread.java:764)
```

Do not cast JavaEnv to void** for AttachCurrentThread on Android

Better support alternative targets in genbindings.sh

Merge pull request #20 from TheBlueMatt/main

Update auto-generated bindings with missed files and v0.0.98 tags

Update auto-generated bindings with missed files and v0.0.98 tags

Merge pull request #18 from TheBlueMatt/main

Update to latest upstream C bindings with Invoice support

Update auto-generated bindings

Force CPU target to Sandy Bridge when compiling

See the similar commit on ldk-c-bindings for more info.

Use newly-exposed get_route in HumanObjectPeerTest

Update CI to not require git tags checked in and fix WASM32 paths.

Avoid git version always appearing as dirty

Update tests and Constructor to new APIs

Expand the list of things exposed via UtilMethods

Drop the "LDK" prefix from unitary enums

Expand and update ChannelManagerConstructor+tests for BP and Event API

Handle methods taking non-ref self by using standard conversion

Handle C -> Java calls from Rust-spawned threads

Turns out you need to "attach" threads to the JVM before you can
make Java calls.

Redo constructor function names to feel more natural in Java

Slightly reduce race conditions in NioPeerHandler

Sadly there's some restructuring that needs to happen to PeerHandler
before all possible race conditions go away.

Update test ChannelManagerConstructor and invoice parsing as well

Use uint64_t instead of long for pointers represented in C

While long is generally correct, the conversion of pointers to
long, then int64_t (passed to Java), then back to pointers
results in poor sign-extension, giving us invalid pointers.
Instead, convert to uint64_t everywhere and things work.

Expose bindings versions and print on startup in debug mode

Also updates to latest upstream ldk-c-bindings include structure.

Co-Authored-By: Arik Sosman <git@arik.io>

Support debug builds on Android by calling native log and not using BT

Handle LDKStr in other structs and fix string conversion overread

Handle empty variant structs and enums in variants

Update HumanObjectPeerTest to print invoice in string form

Update tests for new upstream API and using invoicse

Use explicit return type var names instead of guessing

Map more methods that go in UtilMethods

Don't FREE things when we holds_ref if we're not calling *_free

After this we still only have minor leaks across 128 runs of
HumanObjectPeerTest:
   314 allocations remained for 1126840 bytes.

Handle u5, LDKTwentyBytes, and LDKRecoverableSignature

Fix genbindings.sh sed call on OSX

Handle calls which take strings as an argument in Java

Merge pull request #17 from TheBlueMatt/main

Update to latest upstream

Fix CI

Update tests for latest upstream API, sadly disabling the Sign wrapper test

Test new block connection API and update tests to new API

Update auto-generated bindings