Re: [FFmpeg-devel] [PATCH] lavc/libx264: replace ITU-T T35(A/53 CC) SEI type by enum value

2023-08-26 Thread myp...@gmail.com
On Sat, Aug 26, 2023 at 2:40 PM Kieran Kunhya  wrote:
>
> On Sat, 26 Aug 2023 at 07:32, Jun Zhao  wrote:
>
> > replace ITU-T T35(A/53 CC) SEI type by enum value
> >
> > Signed-off-by: Jun Zhao 
> > ---
> >  libavcodec/libx264.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c
> > index 1a7dc7bdd5..ce849d6c9a 100644
> > --- a/libavcodec/libx264.c
> > +++ b/libavcodec/libx264.c
> > @@ -563,7 +563,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
> >
> >  sei->payloads[0].payload_size = sei_size;
> >  sei->payloads[0].payload  = sei_data;
> > -sei->payloads[0].payload_type = 4;
> > +sei->payloads[0].payload_type =
> > SEI_TYPE_USER_DATA_REGISTERED_ITU_T_T35;
> >  sei->num_payloads = 1;
> >  }
> >  }
> > --
> > 2.25.1
> >
>
> LGTM
thx,will apply
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".


Re: [FFmpeg-devel] [PATCH] lavc/libx264: replace ITU-T T35(A/53 CC) SEI type by enum value

2023-08-25 Thread Kieran Kunhya
On Sat, 26 Aug 2023 at 07:32, Jun Zhao  wrote:

> replace ITU-T T35(A/53 CC) SEI type by enum value
>
> Signed-off-by: Jun Zhao 
> ---
>  libavcodec/libx264.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c
> index 1a7dc7bdd5..ce849d6c9a 100644
> --- a/libavcodec/libx264.c
> +++ b/libavcodec/libx264.c
> @@ -563,7 +563,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
>
>  sei->payloads[0].payload_size = sei_size;
>  sei->payloads[0].payload  = sei_data;
> -sei->payloads[0].payload_type = 4;
> +sei->payloads[0].payload_type =
> SEI_TYPE_USER_DATA_REGISTERED_ITU_T_T35;
>  sei->num_payloads = 1;
>  }
>  }
> --
> 2.25.1
>

LGTM
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".


[FFmpeg-devel] [PATCH] lavc/libx264: replace ITU-T T35(A/53 CC) SEI type by enum value

2023-08-25 Thread Jun Zhao
replace ITU-T T35(A/53 CC) SEI type by enum value

Signed-off-by: Jun Zhao 
---
 libavcodec/libx264.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c
index 1a7dc7bdd5..ce849d6c9a 100644
--- a/libavcodec/libx264.c
+++ b/libavcodec/libx264.c
@@ -563,7 +563,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
 
 sei->payloads[0].payload_size = sei_size;
 sei->payloads[0].payload  = sei_data;
-sei->payloads[0].payload_type = 4;
+sei->payloads[0].payload_type = 
SEI_TYPE_USER_DATA_REGISTERED_ITU_T_T35;
 sei->num_payloads = 1;
 }
 }
-- 
2.25.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".