Michael Niedermayer:
> On Tue, Mar 02, 2021 at 07:53:07PM +0100, Paul B Mahol wrote:
>> Signed-off-by: Paul B Mahol <one...@gmail.com>
>> ---
>> libavcodec/mpeg4videodec.c | 6 ++++--
>> 1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c
>> index fd985f0430..43c2cf2dc3 100644
>> --- a/libavcodec/mpeg4videodec.c
>> +++ b/libavcodec/mpeg4videodec.c
>> @@ -3551,9 +3551,11 @@ static av_cold int decode_init(AVCodecContext *avctx)
>> return 0;
>> }
>>
>> +#define OFFSET(x) offsetof(MpegEncContext, x)
>> +#define VD AV_OPT_FLAG_VIDEO_PARAM | AV_OPT_FLAG_DECODING_PARAM
>> static const AVOption mpeg4_options[] = {
>> - {"quarter_sample", "1/4 subpel MC", offsetof(MpegEncContext,
>> quarter_sample), AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, 0},
>> - {"divx_packed", "divx style packed b frames", offsetof(MpegEncContext,
>> divx_packed), AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, 0},
>> + {"quarter_sample", "1/4 subpel MC", OFFSET(quarter_sample),
>> AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, VD},
>> + {"divx_packed", "divx style packed b frames", OFFSET(divx_packed),
>> AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, VD},
>> {NULL}
>> };
>
> this looks odd, divx_packed is set by the decoder not by the user
>
You added it in f51e5015ad76e5fae57e34712f6a6e4f0f8b2204 and the commit
message says that it is to export said information to the user. Which is
indeed really odd.
- Andreas
_______________________________________________
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".
