Drop `#[must_use]` that rustc complains is ignored

Merge pull request #111 from TheBlueMatt/2023-07-0.0.115-aliasing-fix

[0.0.115] Fix strict-aliasing violation on traits holding inner fields

Update auto-generated bindings with aliasing fix.

Pin quote and proc-macro2 in CI to meet MSRV

Fix strict-aliasing violation on traits holding inner fields

When we map a trait method which returns a reference to a struct,
we map it by storing said struct in the trait implementation
struct. Then, we create a setter method which ensures the new field
is set. Sadly, because the original Rust trait method may take a
non-mutable reference to self, we have to update the field without
a mutable reference to the trait implementation struct.

Previously we did this by simply unsafe-casting a pointer to
mutable, which violates the aliasing rules in Rust. In a recent
rustc (at least on macOS), this broke.

Here, we convert the stored struct to wrap it in an `UnsafeCell`,
which fixes the issue.

Merge pull request #108 from TheBlueMatt/main

Skip address sanitizer builds on OSX

Skip address sanitizer builds on OSX

...cause it appears broken on newer macOS and I'm skeptical its
detecting real leaks.

Merge pull request #107 from TheBlueMatt/main

Support Building OSX binaries on Linux

Update the set of OSX runners since github removed 10.15

Build OSX releases on Linux if `MACOS_SDK` is set

Make `clang-lto-link-osx` work on Linux (by forcing bash)

Merge pull request #106 from TheBlueMatt/main

Make `BitEndianScalar` clonable

Update auto-generated bindings

Make `BitEndianScalar` clonable

Merge pull request #105 from arik-so/2023-05-115-debugging

Add debugging lines to demo.cpp

Add debugging lines to demo.cpp

Merge pull request #103 from TheBlueMatt/main

Update to LDK 0.0.115

Update CI+Cargo.toml bindings branch to 0.0.115

Update auto-generated bindings for LDK 0.0.115

Update C++ demo application to LDK 0.0.115

Ensure an option always unwraps to a non-reference type

Handle Option<Vec<_>> with a much cleaner option converter

Handle use of `#[doc]` in the original source

Replace type resolution assertions with failures

Type resolution now supports failing, but some assertions are still
left over from when it didn't. Replace some of those which now fail
on LDK 0.0.115.

Update hard-coded type definitions

Fix documentation on generated _hash methods for std::hash::Hash

Support the new no-exported tag that is more human readable

Merge pull request #102 from TheBlueMatt/main

Fix C++ demo bindings build due to function signature mismatch

Fix C++ demo bindings build due to function signature mismatch

Merge pull request #101 from TheBlueMatt/main

Update to use `find_route_with_id` to match upstream changes

Update to use `find_route_with_id` to match upstream changes

Merge pull request #100 from TheBlueMatt/main

LDK 0.0.114

Workaround github actions brokenness

Update `rust-lightning` branch to 0.0.114-bindings

Update auto-generated bindings

Note that cross-language LTO requires lld in addition to clang

Print something for custom messages in C++ demo

Fix connection race in C++ demo

Update C/C++ Demos to latest upstream API

Add `lightning-background-processor`'s new std feature

Support new rust-bitcoin type mappings for various hashes

Ignore discriminants on enum variants

While this loses the value information, its better than failing to
generate bindings at all.

Support tuples containing array elements

Add type for 8 u16s (which is used for the scoring data)

Support non-u8 (primitive) arrays

Support supertraits-of-supertraits

If we have a supertrait that requires an in-crate supertrait, we
handle that here by writing the required impls. We don't support
a supertrait-of-a-supertrait-of-a-supertrait.

Fix typo in path resolution leading to a "crate name" of a full path

Support passing `Option<&OpaqueStruct>` to C

Luckily the solution is trivial - just treat the conversion for the
inner pointer as `ptr_for_ref`, because we ultimately want a
pointer on the inside, not on the outside.

Merge pull request #98 from TheBlueMatt/main

Fix OSX building by not redirecting to `/dev/null`

Fix OSX building by not redirecting to `/dev/null`

Somehow this results in a broken pipe error...cause macOS is trash.

Merge pull request #97 from TheBlueMatt/main

Fix LTO compilation with latest ld.lld on OSX

Stop timing C++ demo bindings run cause OSX is, as usual, garbage

Update CI references to OSX clang 15.0.3

Fix LTO compilation with latest ld.lld on OSX

Merge pull request #96 from TheBlueMatt/main

0.0.113 Bindings

Update CI and Cargo.toml references to 0.0.113

Set the C target to wasm32-wasi rather than wasm32

We need at least a partial C stdlib anyway, so there's no real
difference going ahead and linking wasm32-wasi on the C side vs
only on the Rust side. This fixes compilation on Debian testing
with OS packages installed.

Update C/C++ demo bindings to latest upstream API

Update auto-generated bindings to 0.0.113

Support new upstream manually-mapped types

Add support for witnesses via a manual mapping

Support u128 values

Use the resolved name, not original ident, for Vec naming

This will be useful in the next commit when we rename `u5` to `U5`

Ignore async functions as mapping them would be substantial work

Allow enums with lifetime-only generics

Use `Clone::clone(a)` rather than `a.clone` to avoid deref recursion

Some initial support for `pub(.*) use` aliasing in type resolution

Make rustc stop complaining about unused `Box::from_raw`s

Merge pull request #94 from TheBlueMatt/main

Update for 0.0.112

Update Cargo.toml and CI references to 0.0.112

Update auto-generated bindings

Update C/C++ demos for latest upstream API

Map `io::ErrorKind` the same as `io::Error`, we only use the kind

Handle path types that start with `crate::`

Handle multiple `impl`s of the same trait in the same file

This should be easy, but we end up with redundant `use` statements.
Thus, we instead track which use statements we want and put them at
the end when generating new files

(Partially) support supertraits with generic bounds

If we extend a supertrait where the supertrait is generic, we will
ultimately need to genericize the supertrait into a derived object.
This doesn't implement that part, but it does implement the
subtrait-writing code to make it compatible with such a future.

Support trait methods that return enums in the C++ sed script

Merge pull request #93 from TheBlueMatt/main

Expose the `rapid-gossip-sync` crate for no-std builds

Update auto-generated bindings for backport of RGS-no-std support

Expose the `rapid-gossip-sync` crate for no-std builds

Merge pull request #92 from TheBlueMatt/main

Add a convinience constructor for the new BigEndianScalar

Update auto-generated bindings

Add a convinience constructor for the new BigEndianScalar

Merge pull request #91 from TheBlueMatt/main

0.0.111

Update auto-generated bindings

Update C++ bindings demo to latest upstream API

Add Scalar and SharedSecret type mappings for new secp256k1 types

Update bindings branch to 0.0.111 and update rust-bitcoin and secp

Allow liftime bounds on generic bounds

While liftime bounds require bindings users to ensure liftime
requirements are met, they're important for allowing us to export
lock wrappers. Thus, we relax the simple-bounds assertions checks
here.

Don't use a turbofish when all parameters are lifetimes.

For some reason rustc doesn't like this, and its easy, so whatever.

Use trait, not impl, definition for X_as_Trait functions

When we map a trait impl block, we historically didn't know how to
resolve the types in the trait context, so we just used the impl
context definitions and assumed the types were the same. Sadly, if
we have an associated type that is bounded by a trait, we need to
use the trait bound to figure out what to write for the X_as_Trait
method declarations.

Since we now have the ability to generate full trait-context type
resolvers, we do so here.

Support mapping `Readable` `impl`s outside of the `lightning` crate

In order to map `Readable` `impl` blocks in crates other than the
`lightning` main crate we simply have to update the type references
to include crate name, which we do here.

Prefer `Option_` mappings for more types over transparent mappings

In general, the explicit `Option_` mappings are easier for
downstream bindings to match against, and clearer for users anyway.
Sadly, originally we were trying to avoid them where possible, so
some types default to transparent mappings. Here we swap the
default, at least for manually-mapped types, with a few explicit
exceptions (that probably should be changed as well).

Merge pull request #89 from TheBlueMatt/main

Don't force targeting sandybridge on all platforms

Don't force targeting sandybridge on all platforms

This should fix building on macOS when using upstream clang.

Merge pull request #88 from TheBlueMatt/main

Try clone'ing the field before skipping exporting a field getter

Update auto-generated bindings

Try clone'ing the field before skipping exporting a field getter

Merge pull request #87 from TheBlueMatt/main

LDK 0.0.110

Update auto-generated bindings to LDK 0.0.110