Added support for FFMPEG 8.0

changes.d/pr-641.toml

@@ -0,0 +1,5 @@
+[[scuffle-ffmpeg]]
+breaking = true
+category = "feat"
+description = "Added support for FFMPEG 8.0"
+authors = ["@devilish-crow"]

crates/ffmpeg/src/enums/av_fmt_flags.rs

@@ -92,7 +92,8 @@ nutype_enum! {
         /// Allows flushing of buffered data.
         /// - **Used for**: Streaming formats that support mid-stream flushing.
         /// - **Equivalent to**: `AVFMT_ALLOW_FLUSH`
-        AllowFlush = AVFMT_ALLOW_FLUSH as _,
+        /// Deprecated in FFMPEG 8.0, see https://ffmpeg.org/pipermail/ffmpeg-devel/2023-September/314836.html
+        /// AllowFlush = AVFMT_ALLOW_FLUSH as _,
 
         /// The format does not require strict timestamp ordering.
         /// - **Used for**: Formats where out-of-order timestamps are common.
@@ -231,7 +232,8 @@ nutype_enum! {
         /// - **Used for**: Ensuring synchronization in multi-stream files.
         /// - **Binary representation**: `0b0100000000000000`
         /// - **Equivalent to**: `AVFMT_FLAG_SHORTEST`
-        Shortest = AVFMT_FLAG_SHORTEST as _,
+        /// Deprecated in FFMPEG 8.0, see https://ffmpeg.org/pipermail/ffmpeg-devel/2023-August/313668.html
+        /// Shortest = AVFMT_FLAG_SHORTEST as _,
 
         /// **Automatically apply bitstream filters**.
         /// - **Used for**: Simplifying format conversions.

crates/ffmpeg/src/frame.rs

@@ -376,7 +376,7 @@ impl VideoFrame {
 
     /// Returns true if the frame is a keyframe.
     pub const fn is_keyframe(&self) -> bool {
-        self.0.0.as_deref_except().key_frame != 0
+        (self.0.0.as_deref_except().flags & AV_FRAME_FLAG_KEY as i32) != 0
     }
 
     /// Returns the picture type of the frame.

crates/ffmpeg/src/io/snapshots/scuffle_ffmpeg__io__input__tests__receive_packet.snap

@@ -2172,7 +2172,7 @@ expression: packets
             47104,
         ),
         duration: Some(
-            992,
+            1024,
         ),
         pos: Some(
             247025,

crates/ffmpeg/src/io/snapshots/scuffle_ffmpeg__io__output__tests__test_output_write_mp4_fragmented_header.snap

@@ -1,7 +1,6 @@
 ---
 source: crates/ffmpeg/src/io/output.rs
 expression: get_boxes!(output)
-snapshot_kind: text
 ---
 [
     Ftyp(
@@ -445,7 +444,7 @@ snapshot_kind: text
                         BoxHeader {
                             box_type: b"udta",
                         },
-                        b"\0\0\0Ymeta\0\0\0\0\0\0\0!hdlr\0\0\0\0\0\0\0\0mdirappl\0\0\0\0\0\0\0\0\0\0\0\0,ilst\0\0\0$\xa9too\0\0\0\x1cdata\0\0\0\x01\0\0\0\0Lavf61.7.100",
+                        b"\0\0\0Ymeta\0\0\0\0\0\0\0!hdlr\0\0\0\0\0\0\0\0mdirappl\0\0\0\0\0\0\0\0\0\0\0\0,ilst\0\0\0$\xa9too\0\0\0\x1cdata\0\0\0\x01\0\0\0\0Lavf62.3.100",
                     ),
                 ),
             ],

crates/ffmpeg/src/io/snapshots/scuffle_ffmpeg__io__output__tests__test_output_write_mp4_trailer.snap

@@ -1,7 +1,6 @@
 ---
 source: crates/ffmpeg/src/io/output.rs
 expression: get_boxes!(output)
-snapshot_kind: text
 ---
 [
     Ftyp(
@@ -1123,7 +1122,7 @@ snapshot_kind: text
                         BoxHeader {
                             box_type: b"udta",
                         },
-                        b"\0\0\0Ymeta\0\0\0\0\0\0\0!hdlr\0\0\0\0\0\0\0\0mdirappl\0\0\0\0\0\0\0\0\0\0\0\0,ilst\0\0\0$\xa9too\0\0\0\x1cdata\0\0\0\x01\0\0\0\0Lavf61.7.100",
+                        b"\0\0\0Ymeta\0\0\0\0\0\0\0!hdlr\0\0\0\0\0\0\0\0mdirappl\0\0\0\0\0\0\0\0\0\0\0\0,ilst\0\0\0$\xa9too\0\0\0\x1cdata\0\0\0\x01\0\0\0\0Lavf62.3.100",
                     ),
                 ),
             ],

crates/ffmpeg/src/snapshots/scuffle_ffmpeg__decoder__tests__test_decoder_audio.snap

@@ -1118,7 +1118,7 @@ expression: audio_frames
             47104,
         ),
         duration: Some(
-            992,
+            1024,
         ),
         best_effort_timestamp: Some(
             47104,

crates/ffmpeg/src/snapshots/scuffle_ffmpeg__encoder__tests__test_encoder_encode_video.snap

@@ -1,7 +1,6 @@
 ---
 source: crates/ffmpeg/src/encoder.rs
 expression: "&boxes"
-snapshot_kind: text
 ---
 [
     Ftyp(
@@ -742,7 +741,7 @@ snapshot_kind: text
                         BoxHeader {
                             box_type: b"udta",
                         },
-                        b"\0\0\0Ymeta\0\0\0\0\0\0\0!hdlr\0\0\0\0\0\0\0\0mdirappl\0\0\0\0\0\0\0\0\0\0\0\0,ilst\0\0\0$\xa9too\0\0\0\x1cdata\0\0\0\x01\0\0\0\0Lavf61.7.100",
+                        b"\0\0\0Ymeta\0\0\0\0\0\0\0!hdlr\0\0\0\0\0\0\0\0mdirappl\0\0\0\0\0\0\0\0\0\0\0\0,ilst\0\0\0$\xa9too\0\0\0\x1cdata\0\0\0\x01\0\0\0\0Lavf62.3.100",
                     ),
                 ),
             ],