diff --git a/mp4parse/src/lib.rs b/mp4parse/src/lib.rs index 140ad669..2d44c6fd 100644 --- a/mp4parse/src/lib.rs +++ b/mp4parse/src/lib.rs @@ -5591,9 +5591,7 @@ fn read_video_sample_entry(src: &mut BMFFBox) -> Result BoxType::PixelAspectRatioBox => { let pasp = read_pasp(&mut b)?; let aspect_ratio = pasp.h_spacing as f32 / pasp.v_spacing as f32; - let is_valid_aspect_ratio = |value: f32| -> bool { - value > 0.0 && value < 10.0 - }; + let is_valid_aspect_ratio = |value: f32| -> bool { value > 0.0 && value < 10.0 }; // Only set pixel_aspect_ratio if it is valid if is_valid_aspect_ratio(aspect_ratio) { pixel_aspect_ratio = Some(aspect_ratio); diff --git a/mp4parse/tests/public.rs b/mp4parse/tests/public.rs index 937b0ddc..33bc91a8 100644 --- a/mp4parse/tests/public.rs +++ b/mp4parse/tests/public.rs @@ -1490,8 +1490,15 @@ fn public_parse_pasp_h264() { assert_eq!(v.codec_type, mp4::CodecType::H264); assert_eq!(v.width, 640); assert_eq!(v.height, 480); - assert!(v.pixel_aspect_ratio.is_some(), "pixel_aspect_ratio should exist"); - assert_eq!(v.pixel_aspect_ratio.unwrap() as f32, 16.0 / 9.0, "pixel_aspect_ratio should be 16/9"); + assert!( + v.pixel_aspect_ratio.is_some(), + "pixel_aspect_ratio should exist" + ); + assert_eq!( + v.pixel_aspect_ratio.unwrap() as f32, + 16.0 / 9.0, + "pixel_aspect_ratio should be 16/9" + ); } }