[FFmpeg-devel] libavcodec/nvenc_hevc.c: Added missing option -temporal_aq

Submitted by Sven C. Dack on Oct. 13, 2016, 5:06 p.m.

Details

Message ID 04fb2bed-6e53-4f04-acd0-95f349cdee01@sky.com
State New
Headers show

Commit Message

Sven C. Dack Oct. 13, 2016, 5:06 p.m.
The option is present in nvenc_h264.c, but was missing from nvenc_hevc.c. Please 
note that there is a bit of a mess going on in the options with regards to 
dashes and underlines. Is there a guideline available for this??

I've settled for "temporal_aq" with nvenc_hevc.c, because of the already 
existing option "spatial_aq". The same options for nvenc_h264.c do use dashes 
instead of underlines ("-spatial-aq" & "-temporal-aq"). Other options are called 
"-rc-lookahead" or "-vbr_2pass" for example, making this somewhat 
user-unfriendly to memorize.

Patch hide | download patch | download mbox

 libavcodec/nvenc_hevc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavcodec/nvenc_hevc.c b/libavcodec/nvenc_hevc.c
index 4edc741..d14e55f 100644
--- a/libavcodec/nvenc_hevc.c
+++ b/libavcodec/nvenc_hevc.c
@@ -91,6 +91,7 @@  static const AVOption options[] = {
     { "no-scenecut",  "When lookahead is enabled, set this to 1 to disable adaptive I-frame insertion at scene cuts",
                                                             OFFSET(no_scenecut),  AV_OPT_TYPE_BOOL,  { .i64 = 0 }, 0, 1, VE },
     { "spatial_aq",   "set to 1 to enable Spatial AQ",      OFFSET(aq),           AV_OPT_TYPE_BOOL,  { .i64 = 0 }, 0, 1, VE },
+    { "temporal_aq",  "set to 1 to enable Temporal AQ",     OFFSET(temporal_aq),  AV_OPT_TYPE_BOOL,  { .i64 = 0 }, 0,  1, VE },
     { "zerolatency",  "Set 1 to indicate zero latency operation (no reordering delay)",
                                                             OFFSET(zerolatency),  AV_OPT_TYPE_BOOL,  { .i64 = 0 }, 0, 1, VE },
     { "nonref_p",     "Set this to 1 to enable automatic insertion of non-reference P-frames",
-- 
2.9.3