diff options
author | Anthony Ramine <n.oxyde@gmail.com> | 2016-08-24 18:00:24 +0200 |
---|---|---|
committer | Anthony Ramine <n.oxyde@gmail.com> | 2016-08-25 13:07:46 +0200 |
commit | fa17814f127bf2581ca27aa7210612a13c41c918 (patch) | |
tree | e554889fe56e13530cedda9d1726ae3f88e9a5ad /components/script | |
parent | d29f61af317d3ab9e8233f0327c25b4925b4f053 (diff) | |
download | servo-fa17814f127bf2581ca27aa7210612a13c41c918.tar.gz servo-fa17814f127bf2581ca27aa7210612a13c41c918.zip |
Make define_constants unsafe
Diffstat (limited to 'components/script')
-rw-r--r-- | components/script/dom/bindings/interface.rs | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/components/script/dom/bindings/interface.rs b/components/script/dom/bindings/interface.rs index 3b2b5dbde36..47a549602b6 100644 --- a/components/script/dom/bindings/interface.rs +++ b/components/script/dom/bindings/interface.rs @@ -70,21 +70,19 @@ pub type NonNullJSNative = /// Defines constants on `obj`. /// Fails on JSAPI failure. -fn define_constants( +unsafe fn define_constants( cx: *mut JSContext, obj: HandleObject, constants: &[ConstantSpec]) { for spec in constants { rooted!(in(cx) let value = spec.get_value()); - unsafe { - assert!(JS_DefineProperty(cx, - obj, - spec.name.as_ptr() as *const libc::c_char, - value.handle(), - JSPROP_ENUMERATE | JSPROP_READONLY | JSPROP_PERMANENT, - None, - None)); - } + assert!(JS_DefineProperty(cx, + obj, + spec.name.as_ptr() as *const libc::c_char, + value.handle(), + JSPROP_ENUMERATE | JSPROP_READONLY | JSPROP_PERMANENT, + None, + None)); } } |