Be more conservative about holding a ref to a cloned object

If we clone an object before passing to/from native code, we
generally don't need to hold a reference to that object in the
struct that owns the method.

The one exception we include here is if the object being passed is
either a trait or holds a reference to a trait. Its not clear if
this is required but it seems like a potential sharp edge.

Merge pull request #163 from TheBlueMatt/main

Update to c-bindings 0.0.123.1

[TS] Update auto-generated bindings to LDK-C-Bindings 0.0.123.1

[C#] Update auto-generated bindings to LDK-C-Bindings 0.0.123.1

[Java] Update auto-generated bindings to LDK-C-Bindings 0.0.123.1

[C#] Run tests against release dylib on OSX in CI

[Java] Change OSX CI to just check release JARs, don't build

Update README with a bit more info on how Rusty things are mapped

Fixes #160

[C#] Run tests against release library in determinism CI run

[C#] Add a simple test of BOLT12 offer parsing

[C#] Fix bool passing

It turns out that C# always expects bools to be passed as 4-byte
ints, which matches the classic C definition (which was usually
typedef'd to int) but is platform-dependent. On some platforms we
were building with 1-byte ints which led to bools getting mangled
when passing them to C#.

Luckily we already have bools passed to C# separated from bools
passed to Rust, so we can simply redefine `jboolean` to `int32_t`.

Merge pull request #162 from TheBlueMatt/main

Additional CI Debugging

Merge pull request #161 from TheBlueMatt/main

0.0.123.0

[Java] Print error stack trace when tests fail

[C#] Install mono-devel in determinism check to get semantic diff

Update branch references to 0.0.123 in CI

[C#] Update test to LDK 0.0.123 and pay a BOLT12 offer

[C#] Update auto-generated bindings to LDK 0.0.123

[TS] Update tests to LDK 0.0.123

[TS] Update auto-generated bindings to LDK 0.0.123

[TS] Add one more method style for callback definitions

[Java] Update CMC and Human peer test to LDK 0.0.123

[Java] Update auto-generated bindings to LDK 0.0.123

Only handle `null` objects when its allowed

This should result in hitting NPEs rather than in-native
panics/SEGFAULTs.

Avoid manual memory management in NioPeerHandler

We still free the native memory optimistically, but avoid disabling
the GC being able to free memory entirely.

Fixes #157

Add a new `force_free` in Java for cases where its useful

Map the new `LDKTweakedPublicKey` as a 32-byte array

Support 4-tuples

Merge pull request #158 from TheBlueMatt/main

Update CI references to 0.0.122

Update CI references to 0.0.122

Merge pull request #155 from TheBlueMatt/main

Cut 0.0.121.3

[Java] Upgrade Android deterministic build to Debian bookworm

[Java] Update auto-generated bindings

Also don't pass `--target` to clang on *-androideabi

... but do pass the march/mcpu argument.

Merge pull request #154 from TheBlueMatt/main

Fix typo introduced in 2dc5209c398ec23e5ce4f2a287bb55c33c681811

Fix typo introduced in 2dc5209c398ec23e5ce4f2a287bb55c33c681811

Merge pull request #153 from TheBlueMatt/main

Skip passing the `--target` to clang on Android

Skip passing the `--target` to clang on Android

Merge pull request #152 from siwatanejo/master

CI: fix typo in git clone command

CI: fix typo in git clone command

Introduced in:
94ae3f5ecc951f3d527f97ea10a8a4d897e6b696

Merge pull request #150 from TheBlueMatt/main

Ensure bindings statics run before enums try to call native init

[C#] Very marginally clean up get_java_arr_elem conditionals

Skip the strip step on Windows

This partially reverts 5a78aa805d7c550a855d6b589907cab4ab6ca476

[Java] Ensure bindings statics run before enums call native code

Fixes #149.

Merge pull request #151 from aarani/fixStringArrayDecode

CSharp: fix LDKStr array decoding

CSharp: fix LDKStr array decoding

Strings are encoded as array of string references, those references
are actually pointers to the string location and have 64bit in size,
but decoding logic was decoding those pointers as 32bit which caused
the following exception, this commit fixes that.

Exception:

Fatal error. System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
   at InternalUtils.getArrayLength(Int64)
   at InternalUtils.decodeUint8Array(Int64)
   at InternalUtils.decodeString(Int64)
   at org.ldk.structs.Result_CVec_StrZIOErrorZ+Result_CVec_StrZIOErrorZ_OK..ctor(System.Object, Int64)
   at org.ldk.structs.Result_CVec_StrZIOErrorZ.constr_from_ptr(Int64)
   at org.ldk.structs.KVStore.list(System.String, System.String)

Merge pull request #148 from TheBlueMatt/main

Build macOS binaries in the CI-built release C# Nupkg

Merge pull request #147 from TheBlueMatt/main

Various C# build fixes

Build macOS binaries in the CI-built release C# Nupkg

Always use -march on x86, even when targeting MacOS.

Fix `$IS_MAC` usage in c_sharp generation

Pass -isysroot to $CC when targeting MacOS with MACOS_SDK set

Strip LLVM BC from resulting binaries

[C#] Use new X-Lang LTO for C# Windows builds

Pass --target/-mcpu/-march/-mtune to all $CC calls

[C#] Use `Heap{Alloc,Free}` rather than `malloc`/`free` on Windows

We generally expect to allocate pointers in C which Rust will free,
and vice versa. On Windows, this requires that we use the same
allocation path as Rust's stdlib does, which is actually
`Heap{Alloc,Free}`, not `malloc`/`free`. Here, we swap the heap
allocation functions we call, fixing heap corruption issues on
Windows.

Merge pull request #146 from TheBlueMatt/main

Update to LDK 0.0.121

Restore android builds to Debian bullseye

There was never a reason to upgrade, and while it doesn't matter,
I'm not upgrading locally yet.

Fix Java CI determinism validation to ignore C# output

Upload CI-built release artifacts to enable better debugging

Update CI references to LDK 0.0.121 branches

[C#] Update tests to LDK 0.0.121 API

[C#] Update auto-generated C# bindings

[TS] Update tests for LDK 0.0.121 API

[TS] Update auto-generated bindings

[Java] Update batteries and tests to LDK 0.0.121

[Java] Update auto-generated Java bindings to LDK 0.0.121

Drop old manual implementations of the `Tx{In,Out}` getters

...as these are now provided directly upstream.

Add manual implementation of the new `WitnessVersion` struct

[TS] Add one additional call function semantic

[C#] Free array buffers passed to C# after we've decoded them

[C#] Use an instance of the callback delegate, not the fn itself

When passing the callback delegate to C from C# during static init,
it appears (but only on Windows) this actually creates a C# class
instance for the delegate and passes that. However, that class
instance is eventually GC'd and our callbacks will start failing.

Instead, we create a static instance of of the delegate and pass
that to C.

[C#] Check int lengths at compile time

...cause I was using the wrong headers trying to do debug builds
and it took much too long to debug.

[Java] Fix libcode.version argument to not reference C#

This caused the generated shared library to export no symbols.

[Java] Re-introduce glibc versioning checks, but only up to 2.34

Merge pull request #145 from TheBlueMatt/main

Switch C# builds to use `dotnet` on Fedora

[C#] Upload the release zip as a CI artifact

Treat Fedora like Linux...because they think they're so special 🙄

[C#] Update auto-generated bindings

[C#] Add -lntdll to windows builds as rustc relies on it

[C#] Fix compile warnings which have become errors in newer clang

[C#] Switch to net6.0 for the "target", though unclear if it does much

[C#] Switch to `dotnet` for releases

Apparently some tooling doesn't like mono-compiled libraries, so
we just use `dotnet` for releases instead.

[C#] Set the "build id" to a dummy value to improve determinism

[C#] Use faketime explicitly when compiling

As targeting windows has a timestamp in the resulting dll, which we
need to be deterministic.

Merge pull request #144 from TheBlueMatt/main

Update README to remove stale build instructions, mention C#

Update README to remove stale build instructions, mention C#

Merge pull request #143 from TheBlueMatt/main

[C#] Set license URL to the deprecated marker, fix file path

[C#] Set license URL to the deprecated marker, fix file path

Apparently this is required when using a LICENSE file for nuget.org

Merge pull request #142 from TheBlueMatt/main

Initial 0.0.118.0-alpha0 C# Release

[C#] Add CI testing for C#

[C#] Update build commands to match Java, add win and release bins

Also fixes a mono-csc argument error

[C#] Add packaging artifacts to build a .nuget zip

[C#] Add some initial C# tests

[C#] Update auto-generated C# bindings

[C#] Drop reference to `assert` which makes building on Win hard

[C#] Remove version file which should not have been checked in

[C#] Add `-unsafe` to mono-csc and build C library

[C#] Drop marshalling generation

Given we now do array/string conversion using the TypeScript logic
we no longer need to use the C# marshalling nonsense, which we
remove the remnants of here.

[C#] Move C# side to a TypeScript-based array/string conversion

Using the marshallers properly will be a ton of additional work,
and its not clear we can do it in the C -> C# direction anyway.
Instead, we can trivially just use the TypeScript conversion logic
which converts everything through pointers anyway.

We'd previously done this on the C side of things, but here we do
it on the C# side as well.