Merge the To/FromJSValConvertible implementations for Finite<T>.

This commit is contained in:
Ms2ger 2015-04-07 13:34:06 +02:00
parent 58a8cfda52
commit e3683c8598

View file

@ -58,6 +58,7 @@ use libc;
use std::borrow::ToOwned; use std::borrow::ToOwned;
use std::default; use std::default;
use std::marker::MarkerTrait; use std::marker::MarkerTrait;
use std::num::Float;
use std::slice; use std::slice;
/// A trait to retrieve the constants necessary to check if a `JSObject` /// A trait to retrieve the constants necessary to check if a `JSObject`
@ -257,24 +258,6 @@ impl FromJSValConvertible for f32 {
} }
} }
impl ToJSValConvertible for Finite<f32> {
fn to_jsval(&self, cx: *mut JSContext) -> JSVal {
let value = **self;
value.to_jsval(cx)
}
}
impl FromJSValConvertible for Finite<f32> {
type Config = ();
fn from_jsval(cx: *mut JSContext, val: JSVal, option: ()) -> Result<Finite<f32>, ()> {
let result = FromJSValConvertible::from_jsval(cx, val, option);
let result = result.and_then(|v| {
Finite::<f32>::new(v).ok_or(())
});
result
}
}
impl ToJSValConvertible for f64 { impl ToJSValConvertible for f64 {
fn to_jsval(&self, _cx: *mut JSContext) -> JSVal { fn to_jsval(&self, _cx: *mut JSContext) -> JSVal {
unsafe { unsafe {
@ -290,7 +273,7 @@ impl FromJSValConvertible for f64 {
} }
} }
impl ToJSValConvertible for Finite<f64> { impl<T: Float + ToJSValConvertible> ToJSValConvertible for Finite<T> {
#[inline] #[inline]
fn to_jsval(&self, cx: *mut JSContext) -> JSVal { fn to_jsval(&self, cx: *mut JSContext) -> JSVal {
let value = **self; let value = **self;
@ -298,14 +281,15 @@ impl ToJSValConvertible for Finite<f64> {
} }
} }
impl FromJSValConvertible for Finite<f64> { impl<T: Float + FromJSValConvertible<Config=()>> FromJSValConvertible for Finite<T> {
type Config = (); type Config = ();
fn from_jsval(cx: *mut JSContext, val: JSVal, option: ()) -> Result<Finite<f64>, ()> {
let result = FromJSValConvertible::from_jsval(cx, val, option); fn from_jsval(cx: *mut JSContext, value: JSVal, option: ()) -> Result<Finite<T>, ()> {
let result = result.and_then(|v| { let result = try!(FromJSValConvertible::from_jsval(cx, value, option));
Finite::<f64>::new(v).ok_or(()) match Finite::new(result) {
}); Some(v) => Ok(v),
result None => Err(()),
}
} }
} }