diff mbox

[FFmpeg-devel,V1] examples/encode_video: only add sequence end code for mpeg1/2 video

Message ID 1565001644-9105-1-git-send-email-mypopydev@gmail.com
State Accepted
Commit e512d893bfcf6eb0fb60a7601ba1b02d8b44452a
Headers show

Commit Message

Jun Zhao Aug. 5, 2019, 10:40 a.m. UTC 
From: Jun Zhao <barryjzhao@tencent.com>

Only add sequence end code for mpeg1/mpeg2 video, or else use the encoder
libx264 or libx265 in this sample, decoding the output file will get
unknow NALU type error.

Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
---
 doc/examples/encode_video.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
diff mbox

Patch

diff --git a/doc/examples/encode_video.c b/doc/examples/encode_video.c
index 6731b2a..d9ab409 100644
--- a/doc/examples/encode_video.c
+++ b/doc/examples/encode_video.c
@@ -186,7 +186,8 @@  int main(int argc, char **argv)
     encode(c, NULL, pkt, f);
 
     /* add sequence end code to have a real MPEG file */
-    fwrite(endcode, 1, sizeof(endcode), f);
+    if (codec->id == AV_CODEC_ID_MPEG1VIDEO || codec->id == AV_CODEC_ID_MPEG2VIDEO)
+        fwrite(endcode, 1, sizeof(endcode), f);
     fclose(f);
 
     avcodec_free_context(&c);