[Java] Map `LDKReadOnly*` the same as `LDKLocked*` ie via `AutoClosable`

Pass unnamed enum field comments through to field processing

Add handling for the new WitnessVersion type, similar to u5 but 4-bits

Use fully-qualified types more often in conversions

This ensures to-human conversions still work correctly inside
enums where there is an enum variant named identically to the type
which is being converted to (eg in the new `GraphSyncError` enum).

Merge pull request #100 from TheBlueMatt/main

Add note about garbage collection of the chan man constructor

Add note about garbage collection of the chan man constructor

Merge pull request #98 from tnull/2022-05-fix-genbindings-aarch64

Some fixes for aarch64 macos

Some fixes for aarch64 macos

Merge pull request #97 from TheBlueMatt/main

[Java] Fix PATH order to use upstream clang in OSX CI

[Java] Fix PATH order to use upstream clang in OSX CI

Thanks to @tnull for spotting this one.

Merge pull request #96 from TheBlueMatt/main

[Java] Correct trivial race condition in HumanObjectPeerTest

Update ts bindings version to 0.0.106.0beta1

[Java] Correct trivial race condition in HumanObjectPeerTest

On my (very slow) old armv7 device, HumanObjectPeerTest can fail as
`maybe_exchange_peer_messages` will set `ran = false` *after* a
message is added to the runqueue, but before it runs, then when we
call `process_events` there will be no events to process, then
before we get into the `runqueue` `synchronized` block the message
will be delivered, causing us to think we're done, even though
there are now events pending. Because going around the loop twice
is free, we should just not set `ran` to false at the top of
`maybe_exchange_peer_messages`.

Merge pull request #95 from TheBlueMatt/main

Convert primitives before passing to host language

Update OSX CI clang to 14.0.1

[TS] Update auto-generated bindings

[Java] Update auto-generated bindings

Convert primitives before passing to host language

This resolves an issue where we map unsigned primitives to their
signed variants, and occasionally pass the unsigned variant (which
is outside the bounds of the signed variant) to the host language.

In Java this can cause a panic as the VM sees an int outside the
range of a short (for example with network ports).

Merge pull request #94 from TheBlueMatt/main

Fix spurious inclusion of the OSX leaktracking bins in the release jar

Upload OSX CI builds for analysis to debug nondeterminism

Update clang to 14.0.0 (sadly 14.0.1 darwin binaries are not yet up)

Update CI branch to 0.0.106

Pin cbindgen in CI as 0.21 broke the MSRV

[TS] Remove checked-in WASM binary - it should be in ldk-java-bins

[TS] Update tests to latest upstream API

[TS] Update auto-generated bindings

[TS] Implement trait_struct_inc_refcnt properly (identical to java)

[Java] Update auto-generated bindings

[Java] Update LDK batteries to latest upstream API

[Java] Update tests to latest upstream API

Add a new manual type - UnqualifiedError/LDKError

Fully-Qualify Paths in some contexts instead of assuming enums

Instead of assuming enum variants that have a name the same as a
field's type, actually build a (simple) qualified-name method and
start using it.

Merge pull request #91 from TheBlueMatt/main

[Java] Correct the checked-in bindings to match generated ones

[Java] Correct the checked-in bindings to match generated ones

Merge pull request #90 from TheBlueMatt/main

Expose ProbabilisticScorer

[Java] Link libmath for `log10` (REVERT FOR 0.0.106)

[TS/CI] Update dependent package list

[TS] Update auto-generated bindings

[Java] Update auto-generated bindings to include ProbabilisticScorer

[Java] Use ProbabilisticScorer in HumanObjectPeerTest

Don't assume from-var conversion in opaque ret conv

When we do a ret-conv of an opaque struct, don't assume that the
variable as-parsed exists, instead use the variable that's being
created in the ret-conv string if we need to clone.

[Java] Make HumanObjectPeerTest status print a callback

...to make it easier to improve the Android LDKTester app

Merge pull request #88 from TheBlueMatt/main

Fix 32-bit Java builds

[TS] Update auto-generated C bindings with latest changes

[Java] Update auto-generated bindings with latest changes

Make pointer integer type language-dependent, instead of intptr_t

This is more consistent as we already did this in some places and
resolves issues on 32-bit Java platforms (ie Android ARMv7)
introduced in b0ba503b308bcdcbf1b7609876705c12408e44e6 and
071b5fa40465132896ffca3c8ad29d0374e983d2.

[Android] Drop i686 as a build target

I don't believe i686 phones ever existed, and emulators should
prefer x86_64 instead.

Merge pull request #86 from TheBlueMatt/main

Update to 0.0.105.0

Update CI to the ldk-c-bindings 0.0.105 branch

Bump clang to 13.0.1 on OSX in CI

[TS] Update auto-generated bindings for 0.0.105.0

[TS] Handle UInt5s and arrays of UInt5s

[Java] Update HumanObjectPeerTest to test Scorer serialization

This doesn't appear to change much, but it seems nicer to.

As of this commit, the total leaks during a full Java test run are
    25 allocations remained for 1115532 bytes.

[Java] Update auto-generated bindings to 0.0.105.0

[Java] Add a reachabilityFence on underlying trait impl calls

Because the implementation instance is implicitly held as a
reference in the `held` field, it seems unlikely this could ever
matter, hwoever, in the simplified implementation in `PeerTest`
(where there is no Human instance to hold a reference to the
underlying bindings instance), we do see the implementation being
free'd while its still in use. Thus, to ensure we can't hit any
such issue in the future, we add one in normal human objects as
well.

[Java] Update tests + batteries to latest upstream API

[Java] Handle UInt5 array conversion

Correct handling of arrays of 32-byte-array equivalents (Payment*)

Properly match warning comments, not structs with "Warning"

Merge pull request #82 from TheBlueMatt/main

TypeScript Package cleanups and Java socket closure

[TS] Fix CI when we log while the browser is shutting down

[Java] Close all sockets cleanly on `NioPeerHandler::interrupt`

[Java] More carefully ensure sockets are closed in `NioPeerHandler`

There is no indication that the sockets are leaking in the handling
loop, but its good to be careful and ensure we always both close
the SocketChannel and cancel the key registration.

During connection, however, it appears we may leak a SocketChannel
if the connection times out, leaking a file descriptor at least
until the GC kicks in and cleans up after us.  Here we are more
careful.

[Java] Stop processing net before we stop the background processor

See https://github.com/lightningdevkit/rust-lightning/issues/1237A

Fixes #77.

[TS] Copy README.md instead of symlinking, so npm publish accepts it

[TS] Add a types tag in package.json to point to TS types

Merge pull request #81 from TheBlueMatt/main

[TS] Unify Web + Node Codebases (with different init functions)

[TS] Bump package version to 0.0.104.1alpha4

Update auto-generated TypeScript files (now universal Node+Web!)

[TS] Unify Web and Node.JS output so it can be universal

Merge pull request #80 from TheBlueMatt/main

Fix TypeScript CI + Package Missing FIles

[TS] Bump package version to 0.0.104.1alpha3

[TS] Include missing version.mjs file

[TS] spawn second HTTP server for release run, killing previous one

[TS] Manually GC on chromium and skip leak checks otherwise

[TS] Log console messages in browser tests via playwright

Merge pull request #79 from TheBlueMatt/main

TypeScript Cleanups and Fixes

[TS] Do not rely on a symlink as npm publish ignores it

Update auto-generated Java Bindings

[TS] Update auto-generated TypeScript Bindings

[TS] Update tests to build a funding tx and get to broadcast

Use uintptr_t in generic conversion when we're talking about pointers

This may make it harder to support 32-bit Java platforms in the
future if we have a reason to do so, but that seems unlikely and
its possible we can simply redefine the type.

Use intptr_t in C when talking about a pointer.

This fixes a bug in TS where we'd return an int64_t when we intended
to return a pointer, confusing JavaScript as to why it has a bigint
instead of a number.

This may make it harder to support 32-bit Java platforms in the
future if we have a reason to do so, but that seems unlikely and
its possible we can simply redefine the type.

[TS] Run both debug and release builds in CI

[TS] Bump version to 0.0.104.1alpha2

[TS] Include most doc comments

[TS] Fix Some array leaks, mark others, treat < 10 leaks as "okay"

[TS] Empty environment variables as WASI's environ fetch leaks memory

[TS] Use -O1 on debug builds as otherwise they can be unusable slow

[TS] Free native -> human arrays after we're done converting

[TS] Wrap `aligned_alloc` as well as other malloc's.

[TS] Debug allocations somewhat further, incl prints in test.

[TS] Move C header code to next to code that uses it in constr

[TS] Fix Debug builds by removing ((destructor)) attributes

[TS] Remove race-y test for malloc-free-malloc re-using memory

[TS] Swap Os for Oz because its a minor change and a minor win

[TS] Make bindings.mts export nothing, fix some indentation

[TS] Set LDK_GARBAGECOLLECTED_GIT_OVERRIDE so that versions work

[TS] Add bindings-binary-version-check to initializeWasm