From a4cbbeef14071419120b2732b6664b38caf7a285 Mon Sep 17 00:00:00 2001 From: Jonathan Chan Date: Tue, 27 Jun 2017 15:59:00 -0700 Subject: [PATCH] Have unsafe_no_jsmanaged_fields! macro take a type as argument rather than an ident. We only write unsafe_no_jsmanaged_fields!(SomeType), and the documentation says it is to be used for types. --- components/script/dom/macros.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/script/dom/macros.rs b/components/script/dom/macros.rs index 14c05bd3cee8..5bf3257d1d12 100644 --- a/components/script/dom/macros.rs +++ b/components/script/dom/macros.rs @@ -303,7 +303,7 @@ macro_rules! make_nonzero_dimension_setter( /// For use on non-jsmanaged types /// Use #[derive(JSTraceable)] on JS managed types macro_rules! unsafe_no_jsmanaged_fields( - ($($ty:ident),+) => ( + ($($ty:ty),+) => ( $( #[allow(unsafe_code)] unsafe impl $crate::dom::bindings::trace::JSTraceable for $ty {