On 9 March 2015 at 12:19, Andreas Cadhalpun <
andreas.cadhal...@googlemail.com> wrote:
> On 09.03.2015 03:42, Michael Niedermayer wrote:
>
>> allowing access to the size but not the extradata itself is not useful
>> and could lead to potential problems if writing happens through this field
>>
>> Signed-off-by: Michael Niedermayer <michae...@gmx.at>
>> ---
>> libavcodec/options_table.h | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h
>> index 442b212..a906864 100644
>> --- a/libavcodec/options_table.h
>> +++ b/libavcodec/options_table.h
>> @@ -103,7 +103,6 @@ static const AVOption avcodec_options[] = {
>> {"hex", "hex motion estimation", 0, AV_OPT_TYPE_CONST, {.i64 = ME_HEX
>> }, INT_MIN, INT_MAX, V|E, "me_method" },
>> {"umh", "umh motion estimation", 0, AV_OPT_TYPE_CONST, {.i64 = ME_UMH
>> }, INT_MIN, INT_MAX, V|E, "me_method" },
>> {"iter", "iter motion estimation", 0, AV_OPT_TYPE_CONST, {.i64 =
>> ME_ITER }, INT_MIN, INT_MAX, V|E, "me_method" },
>> -{"extradata_size", NULL, OFFSET(extradata_size), AV_OPT_TYPE_INT, {.i64
>> = DEFAULT }, INT_MIN, INT_MAX},
>> {"time_base", NULL, OFFSET(time_base), AV_OPT_TYPE_RATIONAL, {.dbl =
>> 0}, INT_MIN, INT_MAX},
>> {"g", "set the group of picture (GOP) size", OFFSET(gop_size),
>> AV_OPT_TYPE_INT, {.i64 = 12 }, INT_MIN, INT_MAX, V|E},
>> {"ar", "set audio sampling rate (in Hz)", OFFSET(sample_rate),
>> AV_OPT_TYPE_INT, {.i64 = DEFAULT }, 0, INT_MAX, A|D|E},
>>
>>
> This looks very good, because it also fixes the ffm muxer to never create
> files with extradata_size setting, but without extradata.
>
Isn't it API break? Maybe it is better to have a look on asv1 or ffm muxer
to fix problem itself, not to cover it.
I can have a look on this today's evening.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
