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.

[C#] Rewrite trait impl based on typescript, not Java

C# doesn't let us pass arbitrary C# objects over the FFI as a
pointer, at least not easily, so instead we need something that
looks much more like what we do for TypeScript.

Here we rewrite the trait handling in C# based on the TypeScript
code and instead store weak pointers so we can reference trait
implementation instances by a single `long`.

[C#] Move dont-use-reserved-words handling to a util method

[C#] Correct a few things after converting C array handling to the TS one

Merge pull request #141 from TheBlueMatt/main

LDK 0.0.118

[Java] Check determinism of the leaktracking Linux bin as well

[Java] Give up on javadoc entirely as we just cannot require HTML

[Java] Drop legacy glibc support in release builds

Update CI references to LDK 0.0.118

[TS] Update tests to LDK 0.0.118

[TS] Update auto-generated bindings to LDK 0.0.118

[Java] Update tests to LDK 0.0.118

[Java] Build an `OnionMessenger` in CMC, connect ChanMan for offers

[Java] Update auto-generated bindings to LDK 0.0.118

[TS + C#] Correct indentation in enum variant-fetching C code

Correct unitary enum C type variable name

Merge pull request #140 from TheBlueMatt/main

Add missing cargo env load for CI

Add missing cargo env load for CI

The 0.0.117 release determinism tests ended up using a version
different from what was merged by a CI update, which is re-added
here.

Merge pull request #139 from TheBlueMatt/main

LDK 0.0.117

Pin memchr to meet MSRVs during builds

Simplify and make less scary the current status listing

Fixes #137.

Update CI references to 0.0.117

[TS] Update tests and node-net for new LDK 0.0.117 API

[TS] Update auto-generated bindings to 0.0.117

[TS] Correct trait construction with multiple supertraits

[TS] Handle from-hu string array conversions

[TS] Don't null-check array conversions that aren't nullable

These can generate a warning in calling methods that actually don't
allow null.

[Java] Test random strings in invoice description to tests strings

[Java] Add a new HOPT form with a one level keys wrapper

We had a user report this was broken, but it appears to be working
fine in this case here.

[Java] Update batteries and tests to LDK 0.0.117

[Java] Update auto-generated bindings to 0.0.117

[Java] Properly convert strings to/from Modified UTF-8

Fixes #136

Update TransactionOutputs type aliasing rules in genbindings.sh

Update type listing for LDK 0.0.117 bindings and support doubles

[C#] Drop java-specific crap and upgrade mono to something sane

[C#] Make check_arr_len generic across array types

[C#] Update auto-generated files

[C#] Add marshaling logic for array C# -> C conversion

[C#] Use TypeScript Array/String layouts

C# doesn't have a way to pass arrays with a length, so we simply
re-use the TypeScript Array/String passing, which will be usable
with a CustomMarshaler on the C# side.

[C#] Add support for TxIn, int64_t, and int16_t[]

These were added at various points upstream and were never added to
C#

[C#] Add missing handling of traits with supertraits of supertraits

In 831ad69f1f65d26c224fe18003f1694806effa5a support was added for
handling traits with supertraits of supertraits, but the full patch
was not applied to the C# bindings, which is finished here.

Merge pull request #135 from TheBlueMatt/main

0.0.116

Switch to playwright nightly to fix FF on Debian 12

It appears when playwright officially added support for Debian 12
(https://github.com/microsoft/playwright/issues/23532) they also
started throwing an error when trying to run unsupported browsers
on it (which Firefox was in the latest release). This broke our CI
(even though Firefox ran just fine on Debian 12 for the past year).

Here we simply switch to playwright nightly, which we'll just stick
with for a while.

Pin quote and proc-macro2 in bindings builds in CI to meet MSRV

Update CI references to 0.0.116

[TS] Update tests to 0.0.116 API