Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-09-27 23:30:08 +01:00
Code Issues Projects Releases Packages Wiki Activity

base: Remove last usage of static_assertions crate (#38687)

Browse source 
Replace static_assertions in the `size_of_test` macro with `const
assert`s. The error message is still informative.
Additionally also remove the `cfg` guard, which caused the assertion to
only be enabled on 64 bit platforms, which is something one would not
expect given the name `size_of_test` of the macro.
`cargo tree -i static_assertions` now points to only `stylo` using
`static_assertions`, so we should be able to remove this dependency
fairly easy, since the macro doesn't seem to be used there either at
first glance.

Testing: Covered by existing tests.

---------

Signed-off-by: Jonathan Schwender <schwenderjonathan@gmail.com>
This commit is contained in:
Jonathan Schwender 2025-08-14 21:02:37 +02:00 • committed by GitHub
parent f492cbf8c1
commit 40e57e95b2
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 1 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

1
components/shared/base/Cargo.toml
View file

@ -20,7 +20,6 @@ malloc_size_of = { workspace = true }
malloc_size_of_derive = { workspace = true }
parking_lot = { workspace = true }
serde = { workspace = true }
static_assertions = { workspace = true }
time = { workspace = true }
webrender_api = { workspace = true }

3
components/shared/base/id.rs
View file

@ -22,8 +22,7 @@ use webrender_api::{ExternalScrollId, PipelineId as WebRenderPipelineId};
/// Asserts the size of a type at compile time.
macro_rules! size_of_test {
($t: ty, $expected_size: expr) => {
#[cfg(target_pointer_width = "64")]
::static_assertions::const_assert_eq!(std::mem::size_of::<$t>(), $expected_size);
const _: () = assert!(std::mem::size_of::<$t>() == $expected_size);
};
}