[FFmpeg-devel] [PATCH]lavf/mpegtsenc: Set min PID for data pkt to 0x0010

[Michael Niedermayer]

On Sat, Sep 24, 2016 at 12:06:07PM +0200, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch fixes ticket #1673.
> ISO13818-1 indeed specifies in §2.4.3.3 that values 
> up to 0xF are reserved.
> 
> Please comment, Carl Eugen

>  mpegtsenc.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 60a99310ff7b9ef476fa1d70bb01ffcaaf550226  0001-lavf-mpegtsenc-Set-min-PID-for-data-pkt-to-0x0010.patch
> From 06371416c00eaf73430d1bb7d841167356adbe23 Mon Sep 17 00:00:00 2001
> From: Sylvain Laurent <syllaur at gmail.com>
> Date: Sat, 24 Sep 2016 12:01:34 +0200
> Subject: [PATCH] lavf/mpegtsenc: Set min PID for data pkt to 0x0010.
> 
> Fixes ticket #1673.
> ---
>  libavformat/mpegtsenc.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
> index fd849e5..c10a3bf 100644
> --- a/libavformat/mpegtsenc.c
> +++ b/libavformat/mpegtsenc.c
> @@ -1843,7 +1843,7 @@ static const AVOption options[] = {
>        { .i64 = 0x1000 }, 0x0010, 0x1f00, AV_OPT_FLAG_ENCODING_PARAM },
>      { "mpegts_start_pid", "Set the first pid.",
>        offsetof(MpegTSWrite, start_pid), AV_OPT_TYPE_INT,
> -      { .i64 = 0x0100 }, 0x0020, 0x0f00, AV_OPT_FLAG_ENCODING_PARAM },
> +      { .i64 = 0x0100 }, 0x0010, 0x0f00, AV_OPT_FLAG_ENCODING_PARAM },
>      { "mpegts_m2ts_mode", "Enable m2ts mode.",

the previous limit was set by Zach Swena (in CC)
was there a reason why it was 0x20 and not 0x10 ?

Thanks

[...]


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Never trust a computer, one day, it may think you are the virus. -- Compn
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160927/cad22787/attachment.sig>