Closed Atul9 closed 4 years ago
@nical done, I have updated the PR to exclude transform2d.rs & transform3d.rs from cargo fmt changes, please check.
@bors-servo r+
:pushpin: Commit 66b2abf has been approved by nical
:hourglass: Testing commit 66b2abf8784c718f19877bb86bda527ee2d5922b with merge 1a8b95d46fa33978f7a6dcbb7a4c0f2a0550a1c9...
:sunny: Test successful - checks-travis Approved by: nical Pushing 1a8b95d46fa33978f7a6dcbb7a4c0f2a0550a1c9 to master...
Thanks! Looks good overall, except for the transforms2d/3d code which really benefits from visually laying out parameters/members as matrix rows. Could you exclude transform2d.rs and transform3.rs from the changes?