[FFmpeg-devel] [PATCH 3/3] libaomenc: Drop unused noise-sensitivity option
Mark Thompson
sw at jkqxz.net
Mon Nov 19 00:09:59 EET 2018
On 18/11/18 18:34, Jan Ekström wrote:
> On Mon, Nov 5, 2018 at 4:53 PM Mark Thompson <sw at jkqxz.net> wrote:
>>
>> ---
>> On 28/10/18 21:02, Carl Eugen Hoyos wrote:
>>> 2018-10-28 21:08 GMT+01:00, Mark Thompson <sw at jkqxz.net>:
>>>> ---
>>>> This was in the 4.0 release, unfortunately (and did nothing there as well).
>>>
>>> If it has never worked, it can be removed imo.
>>
>> Sure, here is a new version.
>>
>> Thanks,
>>
>> - Mark
>>
>>
>> libavcodec/libaomenc.c | 2 --
>> 1 file changed, 2 deletions(-)
>>
>> diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c
>> index c5458766cb..4cad053a48 100644
>> --- a/libavcodec/libaomenc.c
>> +++ b/libavcodec/libaomenc.c
>> @@ -71,7 +71,6 @@ typedef struct AOMEncoderContext {
>> int crf;
>> int static_thresh;
>> int drop_threshold;
>> - int noise_sensitivity;
>> uint64_t sse[4];
>> int have_sse; /**< true if we have pending sse[] */
>> uint64_t frame_number;
>> @@ -980,7 +979,6 @@ static const AVOption options[] = {
>> { "crf", "Select the quality for constant quality mode", offsetof(AOMContext, crf), AV_OPT_TYPE_INT, {.i64 = -1}, -1, 63, VE },
>> { "static-thresh", "A change threshold on blocks below which they will be skipped by the encoder", OFFSET(static_thresh), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, INT_MAX, VE },
>> { "drop-threshold", "Frame drop threshold", offsetof(AOMContext, drop_threshold), AV_OPT_TYPE_INT, {.i64 = 0 }, INT_MIN, INT_MAX, VE },
>> - { "noise-sensitivity", "Noise sensitivity", OFFSET(noise_sensitivity), AV_OPT_TYPE_INT, {.i64 = 0 }, 0, 4, VE},
>> { "tiles", "Tile columns x rows", OFFSET(tile_cols), AV_OPT_TYPE_IMAGE_SIZE, { .str = NULL }, 0, 0, VE },
>> { "tile-columns", "Log2 of number of tile columns to use", OFFSET(tile_cols_log2), AV_OPT_TYPE_INT, {.i64 = -1}, -1, 6, VE},
>> { "tile-rows", "Log2 of number of tile rows to use", OFFSET(tile_rows_log2), AV_OPT_TYPE_INT, {.i64 = -1}, -1, 6, VE},
>> --
>> 2.19.1
>>
>
> Only two references to noise_sensitivity outside of libvpxenc. LGTM.
Applied.
Thanks,
- Mark
More information about the ffmpeg-devel
mailing list