feat(skip): free and skip boxes are now automatically skipped during parsing

src/atom.rs

@@ -195,6 +195,8 @@ macro_rules! nested {
                         Any::Unknown(kind, _) => {
                             tracing::warn!("unknown box: {:?}", kind);
                         },
+                        Any::Skip(atom) => tracing::debug!("skipping skip box of size {}", atom.zeroed.size),
+                        Any::Free(atom) => tracing::debug!("skipping free box of size {}", atom.zeroed.size),
                         _ => return Err(Error::UnexpectedBox(atom.kind())),
                     }
                 }

src/free.rs

@@ -1,5 +1,9 @@
 use crate::*;
 
+///
+/// Free padding space, will be automatically skipped during parsing.
+/// Also see [`Skip`] for a similar atom.
+///
 #[derive(Debug, Clone, PartialEq)]
 #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 pub struct Free {

src/moov/udta/mod.rs

@@ -8,15 +8,14 @@ use crate::*;
 #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 pub struct Udta {
     pub meta: Option<Meta>,
-    pub skip: Option<Skip>,
 }
 
 impl Atom for Udta {
     const KIND: FourCC = FourCC::new(b"udta");
 
     nested! {
         required: [ ],
-        optional: [ Meta, Skip ],
+        optional: [ Meta ],
         multiple: [ ],
     }
 }
@@ -27,10 +26,7 @@ mod tests {
 
     #[test]
     fn test_udta_empty() {
-        let expected = Udta {
-            meta: None,
-            skip: None,
-        };
+        let expected = Udta { meta: None };
 
         let mut buf = Vec::new();
         expected.encode(&mut buf).unwrap();
@@ -50,7 +46,6 @@ mod tests {
                 },
                 items: vec![],
             }),
-            skip: None,
         };
 
         let mut buf = Vec::new();

src/moov/udta/skip.rs

@@ -1,5 +1,9 @@
 use crate::*;
 
+///
+/// Free padding space, will be automatically skipped during parsing.
+/// Also see [`Free`] for a similar atom.
+///
 #[derive(Debug, Clone, PartialEq, Eq, Default)]
 #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
 pub struct Skip {

src/test/av1.rs

@@ -261,7 +261,6 @@ fn av1() {
                     },
                     items: vec![Ilst::default().into()],
                 }),
-                skip: None,
             })
         }
     );

src/test/bbb.rs

@@ -372,7 +372,6 @@ fn bbb() {
                     hdlr: Hdlr{ handler: FourCC::new(b"mdir"), name: "".into() },
                     items: vec![Ilst::default().into()],
                 }),
-                ..Default::default()
             }),
 
             ..Default::default()

src/test/esds.rs

@@ -297,10 +297,7 @@ fn esds() {
                 ..Default::default()
             }],
             udta: Some(Udta {
-                skip: Some(Skip {
-                    zeroed: 82.into(),
-                }),
-                ..Default::default()
+                meta: None,
             }),
 
             ..Default::default()

src/test/h264.rs

@@ -380,12 +380,7 @@ fn avcc_ext() {
                 udta: None,
             },
         ],
-        udta: Some(Udta {
-            meta: None,
-            skip: Some(Skip {
-                zeroed: Zeroed { size: 81 },
-            }),
-        }),
+        udta: Some(Udta { meta: None }),
     };
 
     assert_eq!(moov, expected, "different decoded result");

src/test/hevc.rs

@@ -584,7 +584,6 @@ fn hevc() {
                     },
                     items: vec![Ilst::default().into(),],
                 }),
-                skip: None,
             })
         }
     );

src/test/uncompressed.rs

@@ -290,7 +290,6 @@ fn uncompressed() {
                     },
                     items: vec![Ilst::default().into()],
                 }),
-                skip: None,
             })
         }
     );