[FFmpeg-devel] [PATCH 25/27] vaapi_encode: Unify SEI option mask enum

Mark Thompson sw at jkqxz.net
Fri Jan 1 23:35:35 EET 2021


This was in two disjoint parts in the H.264 and H.265 files.  Unify
them in the header to avoid any confusion, because they are really the
same enum.
---
 libavcodec/vaapi_encode.h      | 8 ++++++++
 libavcodec/vaapi_encode_h264.c | 6 ------
 libavcodec/vaapi_encode_h265.c | 5 -----
 3 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/libavcodec/vaapi_encode.h b/libavcodec/vaapi_encode.h
index b41604a883..980bbb1b7a 100644
--- a/libavcodec/vaapi_encode.h
+++ b/libavcodec/vaapi_encode.h
@@ -474,5 +474,13 @@ int ff_vaapi_encode_close(AVCodecContext *avctx);
     VAAPI_ENCODE_RC_MODE(QVBR, "Quality-defined variable-bitrate"), \
     VAAPI_ENCODE_RC_MODE(AVBR, "Average variable-bitrate")
 
+// SEI type mask used in options for H.264 and H.265.
+enum {
+    SEI_TIMING              = 0x01,
+    SEI_IDENTIFIER          = 0x02,
+    SEI_RECOVERY_POINT      = 0x04,
+    SEI_MASTERING_DISPLAY   = 0x08,
+    SEI_CONTENT_LIGHT_LEVEL = 0x10,
+};
 
 #endif /* AVCODEC_VAAPI_ENCODE_H */
diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaapi_encode_h264.c
index d24462414c..7c352e982b 100644
--- a/libavcodec/vaapi_encode_h264.c
+++ b/libavcodec/vaapi_encode_h264.c
@@ -35,12 +35,6 @@
 #include "internal.h"
 #include "vaapi_encode.h"
 
-enum {
-    SEI_TIMING         = 0x01,
-    SEI_IDENTIFIER     = 0x02,
-    SEI_RECOVERY_POINT = 0x04,
-};
-
 // Random (version 4) ISO 11578 UUID.
 static const uint8_t vaapi_encode_h264_sei_identifier_uuid[16] = {
     0x59, 0x94, 0x8b, 0x28, 0x11, 0xec, 0x45, 0xaf,
diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c
index 2e8e772008..09d5a8b5cb 100644
--- a/libavcodec/vaapi_encode_h265.c
+++ b/libavcodec/vaapi_encode_h265.c
@@ -37,11 +37,6 @@
 #include "put_bits.h"
 #include "vaapi_encode.h"
 
-enum {
-    SEI_MASTERING_DISPLAY       = 0x08,
-    SEI_CONTENT_LIGHT_LEVEL     = 0x10,
-};
-
 typedef struct VAAPIEncodeH265Picture {
     int pic_order_cnt;
 
-- 
2.29.2



More information about the ffmpeg-devel mailing list