Merge pull request #63 from TheBlueMatt/main

Consider all reference types clonable + handle refs-in-tuples

Update auto-generated bindings

Manually handle generated types containing references to opaques

This is really disgusting, we really shouldn't have references in
generated types at all, its just a mess. Instead, we should break
apart arguments when its a tuple, which would solve this more
cleanly.

Consider all reference types clonable (because they are)

Merge pull request #62 from TheBlueMatt/main

Expose ProbabilisticScorer in Bindings

Update auto-generated bindings with the ProbabilisticScorer

Switch to using the Prob. Scorer in the C++ demo

Allow method call args to be converted with a new_var conversion

The call was already there, not sure why we were asserting on it
not printing anything...

Pass types through `resolve_type` in a few additional places

Over time I'd like to swap all of our generic resolution to pass
through `resolve_type` instead of the current path resolution, as
its strictly more flexible, but for now we just add the few we need.

Separate printing references-with-lifetimes from c-vs-rust types

(and use C types with lifetimes in generic struct creation)

Process all type aliases as C types, leaning on annotations to skip

Previously we'd used the presence of generics on a type alias
definition to decide when to skip creation of a corresponding C
type. However, with `ProbabilisticScorer`, we now have a type alias
which has a generic bound but which we still want to create a C
type for.

Merge pull request #61 from TheBlueMatt/main

Implement `Clone` for `SecretKey`

Implement `Clone` for `SecretKey`

Merge pull request #60 from TheBlueMatt/main

Update to 0.0.105

Update bindings workflow in CI to 0.0.105

Update auto-generated bindings to 0.0.106

Update demo.cpp with latest upstream API

Explicitly refer to `format!()` with an `alloc` prefix for no-std

Add function-call generic parameters when they're resovable

We really shouldn't ever actually need to do this - rustc should be
smart enough to figure out what we're doing given all the
parameters are concrete types, but this appears to be failing
sometimes.

Handle `Option<[u8; 32]>`, mapping it same as `Option<&[u8; 32]>`

Don't mark everything with a feature bound as TestOnly

Convert slices containing non-primitives that are clonable

Update build for lightning-invoice no-std support

Recognize `bitcoin::bech32::u5` instead of only `bech32::u5`

Merge pull request #59 from TheBlueMatt/main

Detect aarch64-osx for using apple sed as well

Detect aarch64-osx for using apple sed as well

May fix #58, but there's probably other broken stuff around.

Merge pull request #57 from TheBlueMatt/main

Optimize more aggressively for WASM builds

Overwrite ldk_ver.h to avoid back-to-back builds duplicating consts

This causes back-to-back builds to have an ldk_ver.h for which the
oldest content dominates.

Stop passing -mcpu=sandybridge to wasm builds on x86

Use -O1 on WASM debug builds as they are otherwise too slow to run.

Drop `cargo rustc` in favor of using $RUSTFLAGS

cargo rustc only passes the given arguments to the linking rustc,
not any building rustc calls, which is not generally what we want.

This further reduces `liblightningjs_release.wasm` from 2.4MB to
2.3 MB.

Optimize more aggressively for WASM builds

Use `-Oz` instead of `-Os` because why not, and include LLVM
bitcode in C dependencies.

This reduces `liblightningjs_release.wasm` from 3.0MB to 2.4MB,
likely mostly from removing secp256k1 symbols that were unused.

Merge pull request #54 from TheBlueMatt/main

Support `no-std`

Update auto-generated bindings

Update git commit hash to instead point to 0.0.104-bindings branch

Support no_std on the generated crate

Swap std references in generated code for core/alloc references

Support building dependent crates with `no-std`

Merge pull request #53 from orenfromberg/patch-1

Improving readability of genbindings.sh

Update genbindings.sh

Merge pull request #52 from TheBlueMatt/main

Correct non-default lld version passing

Correctly detect non-default LLD usage

Passing `-flto=ld*.lld-X` only appears to actually work on debian
testing. The correct argument for an alternative version of LLD is
`-flto=lld-X`.

Revert "Correct LLD detection when a specific version is needed for X-lang LTO"

This reverts commit b33cce1a6255564b88e0615eb5c7941bd7b15759.

Merge pull request #51 from TheBlueMatt/main

0.0.104 Bindings Updates

Only test C++ demo in std mode, as it relies on threading support

Update git reference in checked-in Cargo.toml and CI branch

Update auto-generated bindings

Correct LLD detection when a specific version is needed for X-lang LTO

Also use the full LLD specified as ld*.lld as it seems to be needed
on Linux with LLD 13 now.

Update C++ bindings to latest upstream API

Support Option<()> (and &Option<()>) incl conversion and printing

Drop allow_wallclock_use feature and use std/no-std directly

Map new lightning::chain::keysinterface::KeyMaterial type as [u8; 32]

Expose a 12-byte array instead of 10-byte as OnionV2 has changed

Merge pull request #49 from TheBlueMatt/main

0.0.103.1

Support cross-language LTO on OSX (with upstream clang/lld)

Update git commit in Cargo.toml to upstream bindings changes

Update auto-generated bindings

Expose a method to unmangle pointers in test builds

This is used by Java bindings to compare all pointers against those
returned by its malloc wrap, possibly catching some simple
use-after-free or invalid pointer deref bugs.

It is exposed only when built as debug and defined in C headers
with `LDK_DEBUG_BUILD` defined.

Support `use self::...` statements as they are sometimes use for submodules

Support new c_bindings cfg flag to enable bindings-specific logic

Update C/C++ demos to support new Logger API

Stop special-casing `Logger`, instead map fmt::Arguments to string

Correctly print lifetime parameters when printing generic arguments

Support traits and structs with lifetime bounds (which are ignored)

Support mapping MaybeReadable

This fixes https://github.com/lightningdevkit/ldk-garbagecollected/issues/62

Add helper method on results to return `result_ok`

This fixes https://github.com/lightningdevkit/ldk-garbagecollected/issues/59

Merge pull request #48 from TheBlueMatt/main

0.0.103.0 Updates

Update upstream git tag

Use a branch for bindings, as rustc prevents us from using main

Update auto-generated bindings

Update C++ bindings to latest upstream API + test PaymentRetrier

Avoid double-`ObjOps::nonnull_ptr_to_inner` on options in some cases

Separate logic to print `create_ownable_reference` types

These types aren't actually mapped in the same way as all other
types, so we really shouldn't be trying to use the same conversion
logic for both.

Implement additional `Into<Str>` auto-conversion

Track generic param count and print types in ptr-conversion

This will allow rustc to prevent an issue where we accidentally
map `&&Object as *const _` to get a `*const Object`, instead
getting a pointer to a pointer.

Add constructor for tuple types where all contained types are known

Check if types are understood before we map an impl block for them

Support `use crate::*` statements

Map `PaymentId` to a `[u8; 32]` like other Payemt* IDs.

This should solve https://github.com/lightningdevkit/ldk-garbagecollected/issues/52

Support resolving non-trivial bounds from parent generic contexts

Support converting Option<&[]> to C

impl-Trait on in-crate types in a diff mod from the type definition

Support printing lifetime generics on an impl-Trait block

Support printing impl generics without an immediate bound

(e.g. if you have `fn <A> ... where A: Trait` we'd not have printed
`<A>`)

This further supports printing impl generics with a reference to a
struct (not a trait), which we concretize with a regular reference.

Swap --pretty=expanded with -Zunpretty=expanded

Apparently the `--pretty` option was dropped in newer
rustc, see https://github.com/rust-lang/rust/pull/83491

Merge pull request #46 from TheBlueMatt/main

0.0.102 Bindings Updates

Bump upstream git commit to latest

Update auto-generated bindings

Update C++ demo to latest upstream changes

Handle bitcoin::Transaction aliases instead of the full path

Correctly convert Option<Generic<Inner>> to C

Drop stale manual path mappings for generic containers

These are all mapped as generic containers now so there's no need
for manual mapping entries.

[net] Ensure we don't spin loop by keeping the poll write bit set

Previously we'd not been unsetting POLLOUT, which will result in
us spin-loop'ing with ChannelManager_write_buffer_space_avail.

Further, we'd also not been unsetting the POLLIN bit, which would
result in us never pausing read as the SocketDescriptor API
requires us to.

Merge pull request #45 from TheBlueMatt/main

Correct clone logic for `Str`.

Update auto-generated bindings to latest upstream

Update git commit dependency to latest upstream

Correct clone logic for `Str`.

Previously we'd blindly clone'd the fields, which, if
`data_is_owned` is set, will always result in a a double-free.

Instead, we always clone the underlying bytes, setting
`data_is_owned` on the returned value since its likely the caller
wants to hold onto the string outside of the current context.

Merge pull request #44 from TheBlueMatt/main

Detect a type as clonable with an explicit `impl Clone`

These stay `impl Clone` even after the pretty-print rustc pass so
we cannot only match `core::clone::Clone`

Fix races in demo C++ bindings that can hang valgrind in CI