-
Notifications
You must be signed in to change notification settings - Fork 55
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet! I love how many lines of code we delete here (both generated and non generated)!
final struct1 = bindings.getStruct1(); | ||
var expectedValue = 1; | ||
for (var i = 0; i < struct1.ref.data.dimensions[0]; i++) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah the old api gave access to .dimensions
? Then this is kind of a breaking change.
But we should likely add that kind of access to Array
in dart:ffi
as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then this is kind of a breaking change
Should I update the version to 3.0.0-beta.1 then?
But we should likely add that kind of access to Array in dart:ffi as well.
Yeah, I was wondering that there's no way to get the length of an array. (even though its constant and the user would know it beforehand)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, if we want to be true to semantic versioning, we probably should make it 3.0.0-beta.0
, good catch!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, the key is probably rejected in the config right now, instead of just deprecated. So that is also a breaking change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the key is probably rejected
It will simply warn about an unknown key called array-workaound
, the tool would still work since an unknown key is not a severe error.
Updated version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've also removed the previously deprecated llvm-lib
config.
Closes dart-lang/native#542
array-workaround
config key. (Using this will emit a warning ->unknown key in config
).