Add a build mode for "release optimizations with leaks checks", handle a jar in CI

Implement partial clones when passing complex enums containing traits

Add missing clone when passing a complex enum to Java

Expand the criteria for cloning when passing to Rust

If a method takes an object by ownership, we always need to clone.

Remap non-clonable move-passed optional fn args into Option<&Type>

This may break in the future, so we print a huge warning.

Fix calling Java/TS methods that return strings

Fix conflicting overuse of "ret" as a variable name

Print error information when we fail to load LDK native library

Correctly handle reference tracking for array elements, construct multi-dimentional arrays

Rename ChannelManagerConstructor.ChannelManagerPersister to EventHandler

ChannelManagerPersister is a bit of a misnomer for the struct which
now handles events as well. Thus, we rename it to focus on the
event-handling function here.

Drop dup PeerManager tiemr_tick_occurred calls from NioPeerHandler

We call PeerManager's timer_tick_occurred in the
lightning-background-processor crate, initialized from the
ChannelManagerConstructor. Prior to the use of the
lightning-background-processor we'd needed to call PeerManager's
timer_tick_occurred from NioPeerHandler, but we never dropped it
after the switch. Thus, every ~30 seconds we'll call
PeerManager::timer_tick_occurred twice in a row, disconnecting all
of our peers.

Merge pull request #42 from TheBlueMatt/main

Cut 0.0.100.2 which should be uploadable to Maven Central

Bump version to 0.0.100.2

Skip javadoc generation on OSX since its broken

Update pom.xml to comply with Maven Central requirements

Merge pull request #40 from TheBlueMatt/main

v0.0.100.1

Update autogenerated bindings to new version

Test use of IgnoringMessageHandler with ChannelManagerConstructor

Merge pull request #39 from TheBlueMatt/main

v0.0.100.0

Update auto-generated files

Switch to the 0.0.100 branch on ldk-c-bindings

Update tests to latest upstream master

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