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
Implement raqote backend for 2D canvas rendering #24201
Conversation
I think we should merge these changes after squashing the bitshifting commit and rebasing. |
rust-azure's ellipse() C++ implementation copy/pasted and kind of ported to Rust. Obviously needs refactor to turn it into idiomatic Rust.
@bors-servo r+ |
📌 Commit 2a44994 has been approved by |
Implement raqote backend for 2D canvas rendering <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [ ] These changes fix (part of) #23431 <!-- Either: --> - [ ] There are tests for these changes <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/24201) <!-- Reviewable:end -->
💔 Test failed - status-taskcluster |
Tidy failure in linux and windows error: attempt to shift left with overflow
--> components/canvas/raqote_backend.rs:636:27
|
636| let color: u32 = (self.color.alpha << 8 * 3 |
| ^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: `#[deny(exceeding_bitshifts)]` on by default
error: attempt to shift left with overflow
--> components/canvas/raqote_backend.rs:637:13
|
637| self.color.red << 8 * 2 |
| ^^^^^^^^^^^^^^^^^^^^^^^
error: attempt to shift left with overflow
--> components/canvas/raqote_backend.rs:638:13
|
638| self.color.green << 8 * 1 |
| ^^^^^^^^^^^^^^^^^^^^^^^^^
error: aborting due to 3 previous errors
error: Could not compile `canvas`.
warning: build failed, waiting for other jobs to finish... the mac one failed with #23290 intermittent |
You'll need to cast those u8 values to u32 before shifting. |
Thanks, I'm already on it! Building takes some time since the rebase.. |
@bors-servo r+ |
📌 Commit 308908e has been approved by |
Implement raqote backend for 2D canvas rendering <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [ ] These changes fix (part of) #23431 <!-- Either: --> - [ ] There are tests for these changes <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/24201) <!-- Reviewable:end -->
☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster |
./mach build -d
does not report any errors./mach test-tidy
does not report any errorsThis change is