Remove noop_proc_macro dependency

Cargo.toml

@@ -17,7 +17,6 @@ tracing = [
 [dependencies]
 num-traits = "0.2"
 num-derive = "0.4"
-noop_proc_macro = "0.3.0"
 serde = { version = "1.0", features = ["derive"], optional = true }
 profiling = { version = "1" }
 tracing = { version = "0.1.40", optional = true }

src/frame.rs

@@ -10,10 +10,13 @@
 use crate::math::*;
 use crate::pixel::*;
 use crate::plane::*;
-use crate::serialize::{Deserialize, Serialize};
+
+#[cfg(feature = "serialize")]
+use serde::{Deserialize, Serialize};
 
 /// Represents a raw video frame
-#[derive(Debug, Clone, Eq, PartialEq, Serialize, Deserialize)]
+#[derive(Debug, Clone, Eq, PartialEq)]
+#[cfg_attr(feature = "serialize", derive(Serialize, Deserialize))]
 pub struct Frame<T: Pixel> {
     /// Planes constituting the frame.
     pub planes: [Plane<T>; 3],

src/lib.rs

@@ -83,22 +83,6 @@ pub mod math;
 pub mod pixel;
 pub mod plane;
 
-mod serialize {
-    #[cfg(feature = "serialize")]
-    pub use serde::*;
-
-    #[cfg(not(feature = "serialize"))]
-    pub use noop_proc_macro::{Deserialize, Serialize};
-}
-
-mod wasm_bindgen {
-    #[cfg(all(target_arch = "wasm32", target_os = "unknown"))]
-    pub use wasm_bindgen::prelude::*;
-
-    #[cfg(not(all(target_arch = "wasm32", target_os = "unknown")))]
-    pub use noop_proc_macro::wasm_bindgen;
-}
-
 pub mod prelude {
     pub use crate::math::*;
     pub use crate::pixel::*;

src/pixel.rs

@@ -7,8 +7,11 @@
 // Media Patent License 1.0 was not distributed with this source code in the
 // PATENTS file, you can obtain it at www.aomedia.org/license/patent.
 
-use crate::serialize::{Deserialize, Serialize};
-use crate::wasm_bindgen::*;
+#[cfg(feature = "serialize")]
+use serde::{Serialize, Deserialize};
+
+#[cfg(feature = "wasm")]
+use wasm_bindgen::prelude::*;
 
 use num_derive::FromPrimitive;
 use num_traits::{AsPrimitive, PrimInt, Signed};
@@ -146,8 +149,9 @@ impl Coefficient for i32 {
 }
 
 /// Chroma subsampling format
-#[wasm_bindgen]
-#[derive(Copy, Clone, Debug, Default, PartialEq, Eq, FromPrimitive, Serialize, Deserialize)]
+#[derive(Copy, Clone, Debug, Default, PartialEq, Eq, FromPrimitive)]
+#[cfg_attr(feature = "wasm", wasm_bindgen)]
+#[cfg_attr(feature = "serialize", derive(Serialize, Deserialize))]
 #[repr(C)]
 pub enum ChromaSampling {
     /// Both vertically and horizontally subsampled.

src/plane.rs

@@ -17,10 +17,13 @@ use aligned_vec::{ABox, AVec, ConstAlign};
 
 use crate::math::*;
 use crate::pixel::*;
-use crate::serialize::{Deserialize, Serialize};
+
+#[cfg(feature = "serialize")]
+use serde::{Deserialize, Serialize};
 
 /// Plane-specific configuration.
-#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
+#[derive(Debug, Clone, PartialEq, Eq)]
+#[cfg_attr(feature = "serialize", derive(Serialize, Deserialize))]
 pub struct PlaneConfig {
     /// Data stride.
     pub stride: usize,
@@ -147,7 +150,8 @@ impl<T: Pixel> PlaneData<T> {
 /// One data plane of a frame.
 ///
 /// For example, a plane can be a Y luma plane or a U or V chroma plane.
-#[derive(Clone, PartialEq, Eq, Serialize, Deserialize)]
+#[derive(Clone, PartialEq, Eq)]
+#[cfg_attr(feature = "serialize", derive(Serialize, Deserialize))]
 pub struct Plane<T: Pixel> {
     // TODO: it is used by encoder to copy by plane and by tiling, make it
     // private again once tiling is moved and a copy_plane fn is added.