Skip to content
This repository was archived by the owner on Jan 28, 2024. It is now read-only.

Check that property types aren't incomplete #415

Merged
merged 8 commits into from
Jul 7, 2022
Merged

Conversation

liamappelbe
Copy link
Contributor

You can't use an incomplete struct as a property (it won't compile), but since bit fields aren't supported in FFI, we treat them as incomplete structs. So you can hit this code path by using a bit field as a property.

Fixes dart-lang/native#248

@liamappelbe liamappelbe requested a review from dcharkes July 6, 2022 17:24
Copy link
Contributor

@dcharkes dcharkes left a comment

Choose a reason for hiding this comment

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

lgtm

I've filed

to track by-value support, which is blocked on dart:ffi not supporting it.

@liamappelbe liamappelbe merged commit c811efd into master Jul 7, 2022
@liamappelbe liamappelbe deleted the bitfield_fix branch July 7, 2022 15:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

NSDecimal error
2 participants