-
Notifications
You must be signed in to change notification settings - Fork 182
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
capi
Dart todos
#4235
Comments
cc @Manishearth |
For the utf-16 stuff we have a separate issue: rust-diplomat/diplomat#240 and getting a DiplomatWriteable16 . These don't exist right now.
I'm confused, you'd be converting these at the boundary, yes?
Not uint32_t type either?
JS throws. |
I'm confused. The question here is if we can use the
No, there's only
But what does it throw? |
I believe JS throws an FFIError type that wraps the internal error type.
We should use native dart types in the final interface, yes. That's different from what we use under the hood. |
The linked API throws a |
I believe those are just FFIErrors with nothing inside them |
The problem is that we cannot tell whether it should be a |
yeah we need to add a UTF16String type to diplomat. |
For now I would diplomat::skip UTF16 dart things and only handle the more expensive utf8 ones |
Just collecting some notes:
ICU4XReorderedIndexMap::as_slice
returns a&[usize]
. I need to figure out how to port this to DartICU4XDataStruct::create_decimal_symbols_v1
accepts&[char]
. There's no char type in Dart, it allows code point iteration but the items are strings. We could useList<String>
orList<int>
for this&[u16]
is a string or not, so currently I have to putUint16List
on the APIResult<_, ()>
. This is fine in C++, but we want something to throw in Dart. Check what JS does.The text was updated successfully, but these errors were encountered: