Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] lavc/libx264: replace ITU-T T35(A/53 CC) SEI type by enum value
@ 2023-08-26  6:32 Jun Zhao
  2023-08-26  6:40 ` Kieran Kunhya
  0 siblings, 1 reply; 3+ messages in thread
From: Jun Zhao @ 2023-08-26  6:32 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Jun Zhao, Jun Zhao

replace ITU-T T35(A/53 CC) SEI type by enum value

Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
---
 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-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [FFmpeg-devel] [PATCH] lavc/libx264: replace ITU-T T35(A/53 CC) SEI type by enum value
  2023-08-26  6:32 [FFmpeg-devel] [PATCH] lavc/libx264: replace ITU-T T35(A/53 CC) SEI type by enum value Jun Zhao
@ 2023-08-26  6:40 ` Kieran Kunhya
  2023-08-27  1:54   ` mypopy
  0 siblings, 1 reply; 3+ messages in thread
From: Kieran Kunhya @ 2023-08-26  6:40 UTC (permalink / raw)
  To: FFmpeg development discussions and patches; +Cc: Jun Zhao, Jun Zhao

On Sat, 26 Aug 2023 at 07:32, Jun Zhao <mypopydev@gmail.com> wrote:

> replace ITU-T T35(A/53 CC) SEI type by enum value
>
> Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
> ---
>  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-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [FFmpeg-devel] [PATCH] lavc/libx264: replace ITU-T T35(A/53 CC) SEI type by enum value
  2023-08-26  6:40 ` Kieran Kunhya
@ 2023-08-27  1:54   ` mypopy
  0 siblings, 0 replies; 3+ messages in thread
From: mypopy @ 2023-08-27  1:54 UTC (permalink / raw)
  To: FFmpeg development discussions and patches; +Cc: Jun Zhao, Jun Zhao

On Sat, Aug 26, 2023 at 2:40 PM Kieran Kunhya <kierank@obe.tv> wrote:
>
> On Sat, 26 Aug 2023 at 07:32, Jun Zhao <mypopydev@gmail.com> wrote:
>
> > replace ITU-T T35(A/53 CC) SEI type by enum value
> >
> > Signed-off-by: Jun Zhao <barryjzhao@tencent.com>
> > ---
> >  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-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-08-27  1:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-26  6:32 [FFmpeg-devel] [PATCH] lavc/libx264: replace ITU-T T35(A/53 CC) SEI type by enum value Jun Zhao
2023-08-26  6:40 ` Kieran Kunhya
2023-08-27  1:54   ` mypopy

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git