Merge pull request #38 from TheBlueMatt/main

Provide full (new) struct to trait clone functions

Update auto-generated bindings

Update git hash to latest upstream

Bump rust-bitcoin dependency to match rust-lightning

Add utility method to construct new complex enums

This is incredibly useful for downstream languages eg Java as it
avoids having to manually implement constructors and can just use
common function-mapping logic.

Note which parameters or return values are (secretly) Options

Because `Option<OpaqueType>` is mapped the same as `OpaqueType` its
not immediately clear which values in the API are actually
`Option<>`al. Thus, we should at least have documentation noting
this.

Expose struct method calls on trait structs to C++ directly

We can add method calls for non-trait structs later, but this is
particularly useful as otherwise you need to extract both the
method and the `this_arg` to make the call.

Update C++ demo and ldk_net to new clone API

Update C++ demo to new upstream changes

Provide full (new) struct to trait clone functions

Previously, when we went to clone a trait-implementation struct,
we'd provide only the `this_arg` void pointer, requiring that the
`this_arg` pointer be cloned into a new object while all remaining
fields are copied over exactly.

This has a few important limitations:
 * When a struct is cloned, it is not possible to set a `free`
   function to free any new data placed in `this_arg` without it
   also being set on the original struct.
 * Supertrait fields cannot be updated in the subtrait clone
   method, including the `this_arg` and `free` methods.

The first limitation prevents us from setting `free` after a clone
when the original trait may or may not have had `free` set. For
example, if the original trait was created with a `Obj_as_Trait`
method, cloned copies would never be free'd.

The second limitation prevents us from keeping the super and
subtrait `this_arg` fields in sync, in addition to limitations
similar to the above.

This resulted in Java code double-free'ing the `this_arg` field in
`InMemorySigner` objects which were accessed as both `Sign` and
`BaseSign` traits.

Correct args passed to supertrait C methods in subtrait impls

Copy PeerManager pointer in ldk_net

Swap include <> for include "" to make downstream Swift projects happy

If we're leaving binaries around, use -fPIC for ldk_net.o

This is needed if downstream projects want to just take our
ldk_net.o binary and link them in directly, eg with addrsan to
avoid figuring out exact compile flags.

Fix LLVM version-suffixed binary name checking

Most hosts will install clang/LLVM binaries with the -MAJOR_VER
suffix, but we were searching for -MAJOR_VER.MINOR_VER suffix.

Merge pull request #36 from TheBlueMatt/main

Fix Determinism with Newer Rustc and support additional targets (for Android)

Enable macos-11 in CI

Apply metadata replacement to ldk library as well for new rustc

Add extra targets env variable to build for extra targets (eg Android)

Merge pull request #35 from TheBlueMatt/main

Update build-std parameters

As suggested at https://github.com/rust-lang/wg-cargo-std-aware/issues/71#issuecomment-877403993

Set compiler flags earlier to get -isysroot in all RL builds

Merge pull request #34 from TheBlueMatt/main

Add a POSIX C LDK networking library

Update Cargo git hash to upstream v0.0.99

Update auto-generated bindings to latest upstream

Update C++ bindings demo to latest upstream API

Use correct lld to match rustc LLVM version (if available)

Drop -flto from non-cross-language-lto C++ demo builds

It appears somehow the LLVM IR is slipping into the Rust library,
causing Ubunto link to fail with the following error:

/usr/bin/ld: error: LLVM gold plugin has failed to create LTO module: Invalid record
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Add a POSIX C LDK networking library

Skip trait typedef in extra header as its unnecessary

... and OSX complains that "redefinition of typedef...is a C11
feature"

Drop rust-secp256k1 patch as it has been merged into a release upstream

Merge pull request #31 from TheBlueMatt/main

Fix aarch64 targeting and print an error if its disabled

Drop clang GH actions cache as it appears broken on OSX

Drop MacOS 11 runner until we get access to private preview

Fix aarch64 targeting and print an error if its disabled

Merge pull request #30 from TheBlueMatt/main

Clean up OSX support and build for aarch64-apple-darwin

Add OSX CI

Also build for aarch64-apple-darwin if we're building on x86_64 OSX

Clean up OSX support and rustc path selection somewhat

Merge pull request #29 from TheBlueMatt/main

Update to upstream 0.0.98

Update auto-generated bindings to upstream 0.0.98

Switch rust-lightning dep to upstream now that we're "stable"

Update git HEAD to upstream 0.0.98

Update bindings C++ demo to latest upstream (0.0.98) API

Only set -mcpu=sandybridge on x86_64

Drop spurious deref when converting Vec entries to C

Merge pull request #28 from TheBlueMatt/2021-06-fix-nondeterminism

Fix various sources of build non-determinism and default to targeting sandybridge (ie AVX)

Ensure we sanitize rustc random values for all lightning crates

Use 16 hexits for git describe to avoid nondeterminism

Set LC_ALL and -frandom-seed to avoid common nondeterminism sources

Swap default-cpu for sandybridge and use it when compiling libsecp

sandybridge is a reasonable choice because it supports AVX but is
also 10+ years old (with AMD support for AVX at similar times).

The next option would be haswell with AVX2, which doesn't add as
much and for which AMD processors only came about in mid-2015.

Merge pull request #27 from TheBlueMatt/main

Fix race in C++ demo where num_txs_broadcasted ++s before =0s

Fix race in C++ demo where num_txs_broadcasted ++s before =0s

Merge pull request #26 from TheBlueMatt/main

Update rust-secp patch to match latest version and bump upstream

Regenerate auto-generated bindings

Update RL git hash to latest upstream (hopefully next release)

Update rust-secp patch to match latest version

Merge pull request #25 from TheBlueMatt/2021-04-bp-inclusion

Add background-processor to the with-std bindings

Regenerate auto-generated bindings

Add background-processor to the with-std bindings

Take advantage of clone in more manually-defined c_types structs

Implement Clone for a few additional manually-defined c_types

Implement conversion of std::io::Error to Rust

Impl Send+Sync for every trait, not just those with it explicit

Because Send+Sync are generated by the compiler for us, we don't
really know when we should or should not generate it. That said,
Send+Sync aren't exposed outside of Rust, so it only impacts
whether we can use a type when passed to Rust, not how users will
ultimately interact with types.

Handle concrete Deref<Target=X> bounds in generic args

Merge pull request #24 from TheBlueMatt/main

Update Bindings for latest Upstream LDK

Update auto-generated bindings

Bump git depedency to latest upstream

Update auto-generated bindings to match latest upstream

Update bindings demo clients to the new event handling API

Update bindings demo to use invoices and the new inbound payment API

Create mappings for std::hasher::Hash and std::cmp::Eq

Add an export tag to indicate users cannot implement a given trait

Specifically, for the LDK `EventsProvider` trait, we can't
(trivially) implement the `process_pending_events` function,
defined as:
`fn process_pending_events<H: Deref>(&self, handler: H) where H::Target: EventHandler;`

We currently do not support implementing generic methods, as we'd
need to wrap them in a generic Rust-trait-to-C-trait conversion
utility, which we do not currently have. Thus, because users
almost certainly have no reason to implement the `EventsProvider`
themselves, its simpler to simply prevent the use of the C trait
struct as the Rust trait.

Concretely, this means just skipping the
`impl rustEventsProvider for CEventsProvider` block, which works
fine as no Rust functions take an `EventsProvider` as an argument.

Handle core:: and alloc::paths instead of just std:: (or prelude)

Merge pull request #23 from TheBlueMatt/main

Fix handling of returning Option<&Object> from Rust functions

Rebuild auto-generated bindings

Fix handling of returning Option<&Object> from Rust functions

Specifically, these were previously mapped by calling
`<object.as_ref().unwrap() as *const _>`, taking a pointer to the
option on stack instead of a pointer to the option itself.

We just need to drop the `as_ref()` to make it correct.

Merge pull request #22 from TheBlueMatt/main

Redo wasm bindings

Fix double-free of String bytes after converting an owned Str.

Redo wasm bindings

Merge pull request #20 from TheBlueMatt/main

Update CI to work around missing version.rs

Update auto-generated bindings (without new version files)

Add ability to get the LDK and LDK-C-Bindings version

Fix no-std+WASM builds by not mixing -flto and -fembed-bitcode

Merge pull request #19 from TheBlueMatt/2021-04-invoice-incl

Bump git hashes to latest upstream

Update auto-generated bindings with new upstream invoice creation

Map enum tuple variants with no contents as unitary enum variants

cbindgen maps `enum A { V() }` with a `struct A_V_Body {}` which
has different size on C and C++ (0 and 1 bytes). While this is
fine if a different variant has non-0 size as the union will be
correctly sized. However, there isn't a lot of reason to rely on
this when we can just have a unitary variant as well.

Resolve issues with Str introduced when it became ownable

When converting from a Str to an &str, we previously took ownership
of the Str, copied the pointer to its characters to an &str and
returned the &str. This made the returned bytes always unreadable.

Further, there is now no reason to use Vec_u8Z for owned Strs, so
we swap for Str as well.

Resolve generic params with default (eg `<A=B(>) as their default.

Drop stale () -> u8 mapping (now) used in Rust-only code

Handle generics on enums

Take parameters to trait functions as mut

In keeping with "generally, make all variables mut so that
converters can assume inputs are mut", we should do the same for
trait implementation blocks.

Support secp256k1 RecoverableSignatures

Drop debug print that slipped into git

Update module paths for hash wrappers

Correct handling of Display trait conversion

Previously we were incorrectly taking the *Rust* objects as
parameters in C functions instead of the C-mapped ones.

Merge pull request #18 from TheBlueMatt/2021-04-invoice-incl

Bump to latest upstream and include lightning-invoice bindings

Update auto-generated bindings with lightning-invoice