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

Cache multi-line sed replacements across methods to improve runtime

Merge pull request #43 from TheBlueMatt/main

Link libm for debug builds in testing due to new rust libstd dep

Link libm for debug builds in testing due to new rust libstd dep

Merge pull request #42 from TheBlueMatt/main

Update to v0.0.101

Bump lightning dep gitish to v0.0.101

Update C and C++ demos to latest API and test custom messages

Update auto-generated bindings

Call clone explicitly to avoid deref recursion when cloning traits

Add missing C++ header for LDK::Str

Stop tracking unused syn::Type mappings in GenericResolver

They are now unused as of the previous commit

Use `GenericTypes`'s type resolver instead of re-resolving

When we resolve a type with `GenericTypes::maybe_resolve_path` it
returns a resolved-path string as well as a syn::Path. When
resolving we'd previously then re-resolve the Path with the current
type/import information, instead of just accepting what was already
resolved. Simply accepting what was already resolved not only
simplifies a few cases, but also allows us to clean up where the
"crate::" prefix is printed, keeping "crate::" prefixes out of all
type resolution.

Work around broken resolution on non-pub use aliasing in another module

Handle uncallable trait methods that use Infallible in a parameter

Map core::convert::Infallible to a similar manually-defined struct

Support std::io::Read natively by mapping it to and u8slice

Support converting C `OutPoint`s to `bitcoin` `OutPoint`s

Update io::Read manual implementation to upstream API changes

Avoid mapping optionals as owned pointers

Using an owned pointer to map an optional turned out to be quite
annoying to avoid downstream memory issues. Instead, we use the
much more explicit `COption_TypeZ` enums everywhere.

This also fixes Option-al mapping of enums.

Handle `W: X<T = Y>` generics where `Y` is used as `Z<Y>`

This simply requires calling `resolve_type` in a few more places.

Call methods explicitly on traits to avoid to deref recursion

Correct printing generic methods in traits

If a trait method is generic (and not the object itself), we
previously would print the concrete resolved types in the
`impl nativeTrait for TraitStruct` block instead of the generic
names, leading to compilation failures.

Learn supertrait associated types when processing traits

Define a macro for constructing a TypeResolver for a given module

This is used to resolve types referenced in modules other than the
current one.

Support mapping core::fmt::Debug as a supertrait

Test fetching a Vec<Vec<Opaque>> via a generated getter in C++ demo

Support mapping slices which contain slices which contain opaques

Fixes https://github.com/lightningdevkit/ldk-garbagecollected/issues/43

Map Vecs to slices when constructing a getter for a public field

This adds logic to allow constructing a getter for a public Vec
field where the inner items are opaque structs. In order to do this
we map them as slices, which will end up generating Vecs with
elements which all have their `is_owned` flags unset.

Drop debug print that slipped in in a previous PR

Merge pull request #40 from TheBlueMatt/main

Differentiate inner pointers representing None and Some(ZST)

Update git hash to latest upstream

Update auto-generated bindings to latest upstream

Update auto-generated bindings

Build native debug binaries with --cfg=test_mod_pointers

Differentiate `inner` pointers representing `None` and `Some(ZST)`

For zero-sized-types, rust `Box::into_inner(Box::new(ZST {}))`
returns `1usize as *mut ZST`, which confuses our Java bindings
which check for `None` by checking if `inner < 1024`. While we
could convert the Java bindings to check for `inner == 0`, the
magic value for ZST pointers is not, to my knowledge, an ABI
guarantee Rust provides.

Instead, we add an offset to the `inner` pointers to push them
past the zero page for ZSTs, taking this opportunity to clean up
some of our pointer conversion and push them through a common set
of utility functions.

We also add testing infrastructure to add similar offsets to
non-ZSTs to get good test coverage of the offset addition-removal,
though Rust should largely be ignoring pointer values for ZSTs
anyway so there should be little risk in anything going wrong here.

Merge pull request #39 from TheBlueMatt/main

Update to 0.0.100

Use v0.0.100 for lightning-c-bindings rust-lightning dep

Commit latest auto-generated C bindings based on 0.0.100

Add support for bitcoin::hash_types references and NonZeroU8

Merge pull request #38 from TheBlueMatt/main

Provide full (new) struct to trait clone functions

Update auto-generated bindings