From 6c4ce7247fa3e57075dfebfb60dd9694553fe084 Mon Sep 17 00:00:00 2001 From: deror1869107 Date: Mon, 23 Jan 2017 08:11:44 +0800 Subject: [PATCH 1/2] Use TypedArray --- components/script/dom/bindings/conversions.rs | 55 +------------- .../script/dom/webglrenderingcontext.rs | 74 +++++++++---------- 2 files changed, 37 insertions(+), 92 deletions(-) diff --git a/components/script/dom/bindings/conversions.rs b/components/script/dom/bindings/conversions.rs index f381d9c859c..e1869863c6c 100644 --- a/components/script/dom/bindings/conversions.rs +++ b/components/script/dom/bindings/conversions.rs @@ -47,7 +47,7 @@ use js::glue::{GetProxyPrivate, IsWrapper}; use js::glue::{RUST_JSID_IS_INT, RUST_JSID_TO_INT}; use js::glue::{RUST_JSID_IS_STRING, RUST_JSID_TO_STRING, UnwrapObject}; use js::jsapi::{HandleId, HandleObject, HandleValue, JSContext}; -use js::jsapi::{JSObject, JSString, JS_GetArrayBufferViewType}; +use js::jsapi::{JSObject, JSString}; use js::jsapi::{JS_GetLatin1StringCharsAndLength, JS_GetObjectAsArrayBuffer, JS_GetObjectAsArrayBufferView}; use js::jsapi::{JS_GetReservedSlot, JS_GetTwoByteStringCharsAndLength}; use js::jsapi::{JS_IsArrayObject, JS_NewStringCopyN, JS_StringHasLatin1Chars}; @@ -572,59 +572,6 @@ pub unsafe fn array_buffer_view_data<'a, T>(abv: *mut JSObject) -> Option<&'a mu Some(slice::from_raw_parts_mut(ptr as *mut T, byte_length as usize / mem::size_of::())) } -/// Returns a copy of the ArrayBufferView data, viewed as T, without checking -/// the real type of it. -pub unsafe fn array_buffer_view_to_vec(abv: *mut JSObject) -> Option> - where T: ArrayBufferViewContents -{ - array_buffer_view_data(abv).map(|data| data.to_vec()) -} - -/// Returns a mutable slice of the Array Buffer View data, viewed as T, checking -/// that the real type of it is ty. -pub unsafe fn array_buffer_view_data_checked<'a, T>(abv: *mut JSObject) -> Option<&'a mut [T]> - where T: ArrayBufferViewContents -{ - array_buffer_view_data::(abv).and_then(|data| { - if T::is_type_compatible(JS_GetArrayBufferViewType(abv)) { - Some(data) - } else { - None - } - }) -} - -/// Returns a copy of the ArrayBufferView data, viewed as T, checking that the -/// real type of it is ty. -pub unsafe fn array_buffer_view_to_vec_checked(abv: *mut JSObject) -> Option> - where T: ArrayBufferViewContents -{ - array_buffer_view_data_checked(abv).map(|data| data.to_vec()) -} - -/// Similar API as the array_buffer_view_xxx functions, but for ArrayBuffer -/// objects. -pub unsafe fn array_buffer_data<'a, T>(ab: *mut JSObject) -> Option<&'a mut [T]> - where T: ArrayBufferViewContents -{ - assert!(!ab.is_null()); - - let mut byte_length = 0; - let mut ptr = ptr::null_mut(); - let ret = JS_GetObjectAsArrayBuffer(ab, &mut byte_length, &mut ptr); - if ret.is_null() { - return None; - } - Some(slice::from_raw_parts_mut(ptr as *mut T, byte_length as usize / mem::size_of::())) -} - -/// Similar API to array_buffer_view_to_vec, but for ArrayBuffer objects. -pub unsafe fn array_buffer_to_vec(ab: *mut JSObject) -> Option> - where T: ArrayBufferViewContents -{ - array_buffer_data(ab).map(|data| data.to_vec()) -} - /// Returns whether `value` is an array-like object. /// Note: Currently only Arrays are supported. /// TODO: Expand this to support sequences and other array-like objects diff --git a/components/script/dom/webglrenderingcontext.rs b/components/script/dom/webglrenderingcontext.rs index e467aba951c..f4de68e2add 100644 --- a/components/script/dom/webglrenderingcontext.rs +++ b/components/script/dom/webglrenderingcontext.rs @@ -10,8 +10,6 @@ use dom::bindings::codegen::Bindings::WebGLRenderingContextBinding::WebGLRenderi use dom::bindings::codegen::Bindings::WebGLRenderingContextBinding::WebGLRenderingContextMethods; use dom::bindings::codegen::UnionTypes::ImageDataOrHTMLImageElementOrHTMLCanvasElementOrHTMLVideoElement; use dom::bindings::conversions::{ArrayBufferViewContents, ConversionResult, FromJSValConvertible, ToJSValConvertible}; -use dom::bindings::conversions::{array_buffer_to_vec, array_buffer_view_data, array_buffer_view_data_checked}; -use dom::bindings::conversions::{array_buffer_view_to_vec, array_buffer_view_to_vec_checked}; use dom::bindings::error::{Error, Fallible}; use dom::bindings::inheritance::Castable; use dom::bindings::js::{JS, LayoutJS, MutNullableJS, Root}; @@ -499,7 +497,8 @@ impl WebGLRenderingContext { format: TexFormat, data_type: TexDataType, unpacking_alignment: u32, - data: *mut JSObject) + data: *mut JSObject, + cx: *mut JSContext) -> Result { let element_size = data_type.element_size(); let components_per_element = data_type.components_per_element(); @@ -511,12 +510,14 @@ impl WebGLRenderingContext { // if it is UNSIGNED_SHORT_5_6_5, UNSIGNED_SHORT_4_4_4_4, // or UNSIGNED_SHORT_5_5_5_1, a Uint16Array must be supplied. // If the types do not match, an INVALID_OPERATION error is generated. + typedarray!(in(cx) let typedarray_u8: Uint8Array = data); + typedarray!(in(cx) let typedarray_u16: Uint16Array = data); let received_size = if data.is_null() { element_size } else { - if array_buffer_view_data_checked::(data).is_some() { + if typedarray_u16.is_ok() { 2 - } else if array_buffer_view_data_checked::(data).is_some() { + } else if typedarray_u8.is_ok() { 1 } else { self.webgl_error(InvalidOperation); @@ -779,10 +780,6 @@ unsafe fn typed_array_or_sequence_to_vec(cx: *mut JSContext, ::Config: Clone, { assert!(!sequence_or_abv.is_null()); - if let Some(v) = array_buffer_view_to_vec_checked::(sequence_or_abv) { - return Ok(v); - } - rooted!(in(cx) let mut val = UndefinedValue()); sequence_or_abv.to_jsval(cx, val.handle_mut()); @@ -796,13 +793,13 @@ unsafe fn typed_array_or_sequence_to_vec(cx: *mut JSContext, } #[allow(unsafe_code)] -unsafe fn fallible_array_buffer_view_to_vec(abv: *mut JSObject) -> Result, Error> - where T: ArrayBufferViewContents +unsafe fn fallible_array_buffer_view_to_vec(cx: *mut JSContext, abv: *mut JSObject) -> Result, Error> { assert!(!abv.is_null()); - match array_buffer_view_to_vec::(abv) { - Some(v) => Ok(v), - None => Err(Error::Type("Not an ArrayBufferView".to_owned())), + typedarray!(in(cx) let array_buffer_view: ArrayBufferView = abv); + match array_buffer_view { + Ok(mut v) => Ok(v.as_slice().to_vec()), + Err(_) => Err(Error::Type("Not an ArrayBufferView".to_owned())), } } @@ -1181,15 +1178,15 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.5 - unsafe fn BufferData(&self, _cx: *mut JSContext, target: u32, data: *mut JSObject, usage: u32) -> Fallible<()> { + unsafe fn BufferData(&self, cx: *mut JSContext, target: u32, data: *mut JSObject, usage: u32) -> Fallible<()> { if data.is_null() { return Ok(self.webgl_error(InvalidValue)); } - let data_vec = match array_buffer_to_vec::(data) { - Some(data) => data, - // Not an ArrayBuffer object, maybe an ArrayBufferView? - None => try!(fallible_array_buffer_view_to_vec::(data)), + typedarray!(in(cx) let array_buffer: ArrayBuffer = data); + let data_vec = match array_buffer { + Ok(mut data) => data.as_slice().to_vec(), + Err(_) => try!(fallible_array_buffer_view_to_vec(cx, data)), }; let bound_buffer = match target { @@ -1249,15 +1246,15 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.5 - unsafe fn BufferSubData(&self, _cx: *mut JSContext, target: u32, offset: i64, data: *mut JSObject) -> Fallible<()> { + unsafe fn BufferSubData(&self, cx: *mut JSContext, target: u32, offset: i64, data: *mut JSObject) -> Fallible<()> { if data.is_null() { return Ok(self.webgl_error(InvalidValue)); } - let data_vec = match array_buffer_to_vec::(data) { - Some(data) => data, - // Not an ArrayBuffer object, maybe an ArrayBufferView? - None => try!(fallible_array_buffer_view_to_vec::(data)), + typedarray!(in(cx) let array_buffer: ArrayBuffer = data); + let data_vec = match array_buffer { + Ok(mut data) => data.as_slice().to_vec(), + Err(_) => try!(fallible_array_buffer_view_to_vec(cx, data)), }; let bound_buffer = match target { @@ -1287,9 +1284,9 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.8 - unsafe fn CompressedTexImage2D(&self, _cx: *mut JSContext, _target: u32, _level: i32, _internal_format: u32, + unsafe fn CompressedTexImage2D(&self, cx: *mut JSContext, _target: u32, _level: i32, _internal_format: u32, _width: i32, _height: i32, _border: i32, pixels: *mut JSObject) -> Fallible<()> { - let _data = try!(fallible_array_buffer_view_to_vec::(pixels) ); + let _data = try!(fallible_array_buffer_view_to_vec(cx, pixels) ); // FIXME: No compressed texture format is currently supported, so error out as per // https://www.khronos.org/registry/webgl/specs/latest/1.0/#COMPRESSED_TEXTURE_SUPPORT self.webgl_error(InvalidEnum); @@ -1298,10 +1295,10 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.8 - unsafe fn CompressedTexSubImage2D(&self, _cx: *mut JSContext, _target: u32, _level: i32, + unsafe fn CompressedTexSubImage2D(&self, cx: *mut JSContext, _target: u32, _level: i32, _xoffset: i32, _yoffset: i32, _width: i32, _height: i32, _format: u32, pixels: *mut JSObject) -> Fallible<()> { - let _data = try!(fallible_array_buffer_view_to_vec::(pixels)); + let _data = try!(fallible_array_buffer_view_to_vec(cx, pixels)); // FIXME: No compressed texture format is currently supported, so error out as per // https://www.khronos.org/registry/webgl/specs/latest/1.0/#COMPRESSED_TEXTURE_SUPPORT self.webgl_error(InvalidEnum); @@ -2066,15 +2063,16 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.12 - unsafe fn ReadPixels(&self, _cx: *mut JSContext, x: i32, y: i32, width: i32, height: i32, + unsafe fn ReadPixels(&self, cx: *mut JSContext, x: i32, y: i32, width: i32, height: i32, format: u32, pixel_type: u32, pixels: *mut JSObject) -> Fallible<()> { if pixels.is_null() { return Ok(self.webgl_error(InvalidValue)); } - let mut data = match { array_buffer_view_data::(pixels) } { - Some(data) => data, - None => return Err(Error::Type("Not an ArrayBufferView".to_owned())), + typedarray!(in(cx) let mut pixels_data: ArrayBufferView = pixels); + let mut data = match { pixels_data.as_mut() } { + Ok(data) => data.as_mut_slice(), + Err(_) => return Err(Error::Type("Not an ArrayBufferView".to_owned())), }; if !self.validate_framebuffer_complete() { @@ -2749,7 +2747,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.8 #[allow(unsafe_code)] unsafe fn TexImage2D(&self, - _cx: *mut JSContext, + cx: *mut JSContext, target: u32, level: i32, internal_format: u32, @@ -2762,7 +2760,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { let data = if data_ptr.is_null() { None } else { - Some(try!(fallible_array_buffer_view_to_vec::(data_ptr))) + Some(try!(fallible_array_buffer_view_to_vec(cx, data_ptr))) }; let validator = TexImage2DValidator::new(self, target, level, @@ -2788,7 +2786,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { let expected_byte_length = match { self.validate_tex_image_2d_data(width, height, format, data_type, - unpacking_alignment, data_ptr) } { + unpacking_alignment, data_ptr, cx) } { Ok(byte_length) => byte_length, Err(()) => return Ok(()), }; @@ -2859,7 +2857,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { // https://www.khronos.org/registry/webgl/specs/latest/1.0/#5.14.8 #[allow(unsafe_code)] unsafe fn TexSubImage2D(&self, - _cx: *mut JSContext, + cx: *mut JSContext, target: u32, level: i32, xoffset: i32, @@ -2872,7 +2870,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { let data = if data_ptr.is_null() { None } else { - Some(try!(fallible_array_buffer_view_to_vec::(data_ptr))) + Some(try!(fallible_array_buffer_view_to_vec(cx, data_ptr))) }; @@ -2898,7 +2896,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { let expected_byte_length = match { self.validate_tex_image_2d_data(width, height, format, data_type, - unpacking_alignment, data_ptr) } { + unpacking_alignment, data_ptr, cx) } { Ok(byte_length) => byte_length, Err(()) => return Ok(()), }; From baf8c5462a45dfa6dbcc0f41140d1be66290f613 Mon Sep 17 00:00:00 2001 From: Josh Matthews Date: Thu, 2 Feb 2017 12:02:13 -0500 Subject: [PATCH 2/2] Make typed_array_or_sequence_to_vec support typed arrays. --- Cargo.lock | 2 +- .../script/dom/webglrenderingcontext.rs | 52 +++++++++++-------- 2 files changed, 32 insertions(+), 22 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index de3c8529c3a..b9380dc554d 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -1288,7 +1288,7 @@ dependencies = [ [[package]] name = "js" version = "0.1.4" -source = "git+https://github.com/servo/rust-mozjs#4ad9326fc70d13294c77f9f0988fda6abfd0023b" +source = "git+https://github.com/servo/rust-mozjs#cc9185025d2655074b29cc0a4bf5ee450356ac5f" dependencies = [ "cmake 0.1.20 (registry+https://github.com/rust-lang/crates.io-index)", "heapsize 0.3.8 (registry+https://github.com/rust-lang/crates.io-index)", diff --git a/components/script/dom/webglrenderingcontext.rs b/components/script/dom/webglrenderingcontext.rs index f4de68e2add..6c9aa68db9b 100644 --- a/components/script/dom/webglrenderingcontext.rs +++ b/components/script/dom/webglrenderingcontext.rs @@ -9,7 +9,7 @@ use dom::bindings::codegen::Bindings::WebGLRenderingContextBinding::{self, WebGL use dom::bindings::codegen::Bindings::WebGLRenderingContextBinding::WebGLRenderingContextConstants as constants; use dom::bindings::codegen::Bindings::WebGLRenderingContextBinding::WebGLRenderingContextMethods; use dom::bindings::codegen::UnionTypes::ImageDataOrHTMLImageElementOrHTMLCanvasElementOrHTMLVideoElement; -use dom::bindings::conversions::{ArrayBufferViewContents, ConversionResult, FromJSValConvertible, ToJSValConvertible}; +use dom::bindings::conversions::{ConversionResult, FromJSValConvertible, ToJSValConvertible}; use dom::bindings::error::{Error, Fallible}; use dom::bindings::inheritance::Castable; use dom::bindings::js::{JS, LayoutJS, MutNullableJS, Root}; @@ -37,8 +37,9 @@ use dom::window::Window; use euclid::size::Size2D; use ipc_channel::ipc::{self, IpcSender}; use js::conversions::ConversionBehavior; -use js::jsapi::{JSContext, JSObject, JS_GetArrayBufferViewType, Type}; +use js::jsapi::{JSContext, JSObject, JS_GetArrayBufferViewType, Type, Rooted}; use js::jsval::{BooleanValue, DoubleValue, Int32Value, JSVal, NullValue, UndefinedValue}; +use js::typedarray::{TypedArray, TypedArrayElement, Float32, Int32}; use net_traits::image::base::PixelFormat; use net_traits::image_cache_thread::ImageResponse; use offscreen_gl_context::{GLContextAttributes, GLLimits}; @@ -775,15 +776,24 @@ impl Drop for WebGLRenderingContext { #[allow(unsafe_code)] unsafe fn typed_array_or_sequence_to_vec(cx: *mut JSContext, sequence_or_abv: *mut JSObject, - config: ::Config) -> Result, Error> - where T: ArrayBufferViewContents + FromJSValConvertible, - ::Config: Clone, + config: ::Config) + -> Result, Error> + where T: TypedArrayElement, + T::Element: FromJSValConvertible + Clone, + ::Config: Clone, { + // TODO(servo/rust-mozjs#330): replace this with a macro that supports generic types. + let mut typed_array_root = Rooted::new_unrooted(); + let typed_array: Option> = + TypedArray::from(cx, &mut typed_array_root, sequence_or_abv).ok(); + if let Some(mut typed_array) = typed_array { + return Ok(typed_array.as_slice().to_vec()); + } assert!(!sequence_or_abv.is_null()); rooted!(in(cx) let mut val = UndefinedValue()); sequence_or_abv.to_jsval(cx, val.handle_mut()); - match Vec::::from_jsval(cx, val.handle(), config) { + match Vec::::from_jsval(cx, val.handle(), config) { Ok(ConversionResult::Success(v)) => Ok(v), Ok(ConversionResult::Failure(error)) => Err(Error::Type(error.into_owned())), // FIXME: What to do here? Generated code only aborts the execution of @@ -2318,7 +2328,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { uniform: Option<&WebGLUniformLocation>, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ConversionBehavior::Default)); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ConversionBehavior::Default)); if self.validate_uniform_parameters(uniform, UniformSetterType::Int, &data_vec) { self.ipc_renderer @@ -2336,7 +2346,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { uniform: Option<&WebGLUniformLocation>, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if self.validate_uniform_parameters(uniform, UniformSetterType::Float, &data_vec) { self.ipc_renderer @@ -2365,7 +2375,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { uniform: Option<&WebGLUniformLocation>, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if self.validate_uniform_parameters(uniform, UniformSetterType::FloatVec2, @@ -2398,7 +2408,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { uniform: Option<&WebGLUniformLocation>, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ConversionBehavior::Default)); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ConversionBehavior::Default)); if self.validate_uniform_parameters(uniform, UniformSetterType::IntVec2, @@ -2431,7 +2441,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { uniform: Option<&WebGLUniformLocation>, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if self.validate_uniform_parameters(uniform, UniformSetterType::FloatVec3, @@ -2464,7 +2474,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { uniform: Option<&WebGLUniformLocation>, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ConversionBehavior::Default)); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ConversionBehavior::Default)); if self.validate_uniform_parameters(uniform, UniformSetterType::IntVec3, @@ -2498,7 +2508,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { uniform: Option<&WebGLUniformLocation>, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ConversionBehavior::Default)); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ConversionBehavior::Default)); if self.validate_uniform_parameters(uniform, UniformSetterType::IntVec4, @@ -2531,7 +2541,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { uniform: Option<&WebGLUniformLocation>, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if self.validate_uniform_parameters(uniform, UniformSetterType::FloatVec4, @@ -2552,7 +2562,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { transpose: bool, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if self.validate_uniform_parameters(uniform, UniformSetterType::FloatMat2, &data_vec) { @@ -2572,7 +2582,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { transpose: bool, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if self.validate_uniform_parameters(uniform, UniformSetterType::FloatMat3, &data_vec) { @@ -2592,7 +2602,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { transpose: bool, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if self.validate_uniform_parameters(uniform, UniformSetterType::FloatMat4, &data_vec) { @@ -2632,7 +2642,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] unsafe fn VertexAttrib1fv(&self, cx: *mut JSContext, indx: u32, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if data_vec.len() < 1 { return Ok(self.webgl_error(InvalidOperation)); } @@ -2649,7 +2659,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] unsafe fn VertexAttrib2fv(&self, cx: *mut JSContext, indx: u32, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if data_vec.len() < 2 { return Ok(self.webgl_error(InvalidOperation)); } @@ -2666,7 +2676,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] unsafe fn VertexAttrib3fv(&self, cx: *mut JSContext, indx: u32, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if data_vec.len() < 3 { return Ok(self.webgl_error(InvalidOperation)); } @@ -2683,7 +2693,7 @@ impl WebGLRenderingContextMethods for WebGLRenderingContext { #[allow(unsafe_code)] unsafe fn VertexAttrib4fv(&self, cx: *mut JSContext, indx: u32, data: *mut JSObject) -> Fallible<()> { assert!(!data.is_null()); - let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); + let data_vec = try!(typed_array_or_sequence_to_vec::(cx, data, ())); if data_vec.len() < 4 { return Ok(self.webgl_error(InvalidOperation)); }