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
WebIDL: use Uint8Array #31145
WebIDL: use Uint8Array #31145
Conversation
8bfd716
to
1af44fc
Compare
@@ -112,3 +102,21 @@ impl HeapFloat32Array { | |||
} | |||
} | |||
} | |||
|
|||
pub fn create_typed_array<T, S>( |
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.
@gterzian can you please review this one?
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.
Great work, thanks! LGTM, with just one nit. Leaving the issue open because there are still other typed arrays to use(right?).
@@ -2405,6 +2405,7 @@ impl<'a> ContentSizesComputation<'a> { | |||
had_non_whitespace_content_yet: false, | |||
linebreaker: None, | |||
ending_inline_pbm_stack: Vec::new(), | |||
}.traverse(inline_formatting_context) | |||
} | |||
.traverse(inline_formatting_context) |
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 think it's best to revert this one so as to keep it focused.
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.
Yes I will revert, that was result of fmt
Yes, and I will make other PR to address other typed arrays |
Signed-off-by: Bentaimia Haddadi <haddadi.taym@gmail.com>
1af44fc
to
611d253
Compare
Part of #31064
./mach build -d
does not report any errors./mach test-tidy
does not report any errors