Skip to content

Conversation

@DaughterOfMars
Copy link
Contributor

Description

Adds FFI for Bls12381 types.

Closes #136

@DaughterOfMars DaughterOfMars linked an issue Aug 26, 2025 that may be closed by this pull request
pub struct ValidatorAggregatedSignature(pub iota_types::ValidatorAggregatedSignature);

#[uniffi::export]
impl ValidatorAggregatedSignature {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for a bitmap getter?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't really sure how to handle it. I could just serialize the bytes for the getter but I can't just return the type.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm kind of lacking the context if it's useful to get it or not, in doubt I'd serialise it yeah

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@thibault-martinez thibault-martinez merged commit 5d4dd33 into sdk-bindings Aug 26, 2025
8 checks passed
@thibault-martinez thibault-martinez deleted the feat/bls12381-ffi branch August 26, 2025 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add FFI types for Bls12381 crypto types

4 participants