Use &[ComputedOperation] as the argument type of convert_transform.

Therefore, we don't need to clone the vector.
This commit is contained in:
Boris Chiou 2017-05-10 12:59:43 +08:00
parent 013b4a7fd0
commit fe028e045a
2 changed files with 4 additions and 4 deletions

View file

@ -2229,7 +2229,7 @@ fn static_assert() {
% endfor
}
</%def>
pub fn convert_transform(input: Vec<longhands::transform::computed_value::ComputedOperation>,
pub fn convert_transform(input: &[longhands::transform::computed_value::ComputedOperation],
output: &mut structs::root::RefPtr<structs::root::nsCSSValueSharedList>) {
use gecko_bindings::structs::nsCSSKeyword::*;
use gecko_bindings::sugar::refptr::RefPtr;
@ -2249,7 +2249,7 @@ fn static_assert() {
let servo = iter.next().expect("Gecko_NewCSSValueSharedList should create a shared \
value list of the same length as the transform vector");
unsafe {
match servo {
match *servo {
${transform_function_arm("Matrix", "matrix3d", ["number"] * 16)}
${transform_function_arm("MatrixWithPercents", "matrix3d", ["number"] * 12 + ["lop"] * 2
+ ["length"] + ["number"])}
@ -2275,7 +2275,7 @@ fn static_assert() {
}
return;
};
Self::convert_transform(vec, &mut self.gecko.mSpecifiedTransform);
Self::convert_transform(&vec, &mut self.gecko.mSpecifiedTransform);
}
pub fn copy_transform_from(&mut self, other: &Self) {

View file

@ -455,7 +455,7 @@ pub extern "C" fn Servo_AnimationValue_GetTransform(value: RawServoAnimationValu
let list = unsafe { &mut *list };
match servo_list.0 {
Some(ref servo_list) => {
style_structs::Box::convert_transform(servo_list.clone(), list);
style_structs::Box::convert_transform(servo_list, list);
},
None => unsafe {
list.set_move(RefPtr::from_addrefed(Gecko_NewNoneTransform()));