Merge pull request #74 from TheBlueMatt/main

[ldk-net] Correct pollfds array offset after reads

[ldk-net] Correct pollfds array offset after reads

Previously, we were copying the pollfds array at an offset of one
into the handler thread's stack. However, when it was changed to
copying at a 0 offset (adding the pipe read fd at the end instead
of beginning), the disable-read handling code was not updated.

This leads to an assertion failure at runtime if LDK decides we
need to stop reading due to the outbound buffer of a peer being
full.

Merge pull request #73 from TheBlueMatt/main

Handle impl blocks outside of the mod of the being-implemented object

Drop now-unused code from c-bindings-gen

Update auto-generated bindings

Handle impl blocks exist outside of the mod of the being-implemented object

This allows us to implement the feature bit accessor functions.

Use full paths in impl block handling instead of assuming in-file idents

In the next commit we'll want to handle impl blocks in one module
for an object that is defined in a different model. This violates
several existing assumptions in the impl block handling code,
namely that we're implementing for an object declared in the
current module. This relaxes that assumption in a few places.

Merge pull request #71 from TheBlueMatt/main

Make `Str`'s `clone` always clone the underlying bytes

Make `Str`'s `clone` always clone the underlying bytes

Its incredibly unexpected that you can clone a higher-level object
(eg an Event with a ClosureReason that contains an `Str`) and have
a pointer back to the original object. To avoid this, `clone` needs
to actually `clone`.

Merge pull request #70 from TheBlueMatt/main

Expose consistent free+clone methods for `Bech32Error`

Update auto-generated bindings

Expose consistent free+clone methods for `Bech32Error`

Merge pull request #69 from TheBlueMatt/main

Update auto-generated bindings with FromStr errors

Send peers the remote address we have for them in ldk-net

The majority of this commit was authored by
Jeffrey Czyz <jkczyz@gmail.com>

Use `_none()` constructor instead of setting tag in C++ demo

Expose error types when implementing FromStr

Fixes #64.

Add additional error type conversion logic

Merge pull request #68 from jkczyz/2022-03-ldk-0-0-106

Update to 0.0.106

Update bindings workflow in CI to 0.0.106

Update auto-generated bindings to 0.0.106

Add a .gitignore for output of genbindings.sh

Update ldk_net.c for 0.0.106

Update demo.cpp for 0.0.106

Merge pull request #67 from TheBlueMatt/main

Redo tuple-reference mapping and prep for 106

Update auto-generated bindings

Crack `ReadableArgs` params when they're tuples

This replaces the special handling we'd had in 0.0.105.2 for
tuples-containing-references by instead just cracking open the
tuples and converting each field individually when calling
`ReadableArgs` where the argument is a tuple.

Revert "Consider all reference types clonable (because they are)"

This reverts commit 291aea2200e01843623a67d42112500b1810d276.

Revert "Manually handle generated types containing references to opaques"

This reverts commit 3798e35c9b8f23fa1500b8bc9867fef319287133.

Separate out CFLAGS for host builds and separate targets

This should at least fix one bug someone ran into trying to build
for Android from OSX.

Clone for `&Option<Enum>`

Push down type conversion logic around contained-has-inner some

Pin cbindgen as 0.21 broke the MSRV

Dont try to run valgrind on ppcel64 (it barfs on secp256k1)

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