[FFmpeg-devel] [PATCH] avformat/hlsenc: add hls encrypt options

[Moritz Barsnick]

On Wed, Apr 05, 2017 at 22:04:01 +0800, Steven Liu wrote:
> +    {"hls_enc",       "AES128 encryption support",               OFFSET(encrypt), AV_OPT_TYPE_INT,    {.i64 = 0},     0, 1, E},
> +    {"hls_enc_key",   "use the specified hex-coded 16byte key to encrypt the segments",  OFFSET(key), AV_OPT_TYPE_STRING, .flags = E},
> +    {"hls_enc_key_url", "url to access the key to decrypt the segments",    OFFSET(key_url), AV_OPT_TYPE_STRING, {.str = NULL},  0, 0, E},
> +    {"hls_enc_iv",     "use the specified hex-coded 16byte initialization vector",  OFFSET(iv), AV_OPT_TYPE_STRING, .flags = E},

"use the specified" makes it sound like a boolean option. I suggest:

 {"hls_enc_key", "hex-coded 16 byte key to encrypt the segments" [...]
 {"hls_enc_iv", "hex-coded 16 byte initialization vector" [...]

And these should be documented of course.

BTW, your use of spaces in the option declaration is quite ... weird.
;-)

Moritz