On 16/05/16 21:34, Francois Cartegnie wrote:
> Unless specified, users must use non registered
> range for network id
> ---
>  libavformat/mpegtsenc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
> index eca348a..cfc59ae 100644
> --- a/libavformat/mpegtsenc.c
> +++ b/libavformat/mpegtsenc.c
> @@ -1234,7 +1234,7 @@ static const AVOption options[] = {
>        { .i64 = 0x0001 }, 0x0001, 0xffff, AV_OPT_FLAG_ENCODING_PARAM },
>      { "mpegts_original_network_id", "Set original_network_id field.",
>        offsetof(MpegTSWrite, original_network_id), AV_OPT_TYPE_INT,
> -      { .i64 = 0x0001 }, 0x0001, 0xffff, AV_OPT_FLAG_ENCODING_PARAM },
> +      { .i64 = 0xff01 }, 0x0001, 0xffff, AV_OPT_FLAG_ENCODING_PARAM },
>      { "mpegts_service_id", "Set service_id field.",
>        offsetof(MpegTSWrite, service_id), AV_OPT_TYPE_INT,
>        { .i64 = 0x0001 }, 0x0001, 0xffff, AV_OPT_FLAG_ENCODING_PARAM },
> 

I guess doing

#define NON_REGISTERED_NETWORK 0xff01

Might make it self-commented.

I guess all the other fields might get the same treatment later.

Patch Ok with that change.

lu
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to