[Spice-devel] [PATCH v2] replay: better documentation for -C and -S options

[Pavel Grunt]

Acked-by: Pavel Grunt <pgrunt at redhat.com>

On Fri, 2016-01-15 at 15:09 +0000, Frediano Ziglio wrote:
> Document all possible values
> 
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  server/tests/replay.c | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
> 
> diff --git a/server/tests/replay.c b/server/tests/replay.c
> index f0cbee5..f0ad6ed 100644
> --- a/server/tests/replay.c
> +++ b/server/tests/replay.c
> @@ -298,8 +298,29 @@ int main(int argc, char **argv)
>          { NULL }
>      };
>  
> +    static const char description[] =
> +        "Compression values:\n"
> +        "\t1=off 2=auto_glz 3=auto_lz 4=quic 5=glz 6=lz 7=lz4\n"
> +        "\n"
> +        "Streaming values:\n"
> +        "\t1=off 2=all 3=filter";
> +
> +    /* these asserts are here to check that the documentation we
> state above is still correct */
> +    G_STATIC_ASSERT(SPICE_STREAM_VIDEO_OFF == 1);
> +    G_STATIC_ASSERT(SPICE_STREAM_VIDEO_ALL == 2);
> +    G_STATIC_ASSERT(SPICE_STREAM_VIDEO_FILTER == 3);
> +    G_STATIC_ASSERT(SPICE_IMAGE_COMPRESSION_INVALID == 0);
> +    G_STATIC_ASSERT(SPICE_IMAGE_COMPRESSION_OFF == 1);
> +    G_STATIC_ASSERT(SPICE_IMAGE_COMPRESSION_AUTO_GLZ == 2);
> +    G_STATIC_ASSERT(SPICE_IMAGE_COMPRESSION_AUTO_LZ == 3);
> +    G_STATIC_ASSERT(SPICE_IMAGE_COMPRESSION_QUIC == 4);
> +    G_STATIC_ASSERT(SPICE_IMAGE_COMPRESSION_GLZ == 5);
> +    G_STATIC_ASSERT(SPICE_IMAGE_COMPRESSION_LZ == 6);
> +    G_STATIC_ASSERT(SPICE_IMAGE_COMPRESSION_LZ4 == 7);
> +
>      context = g_option_context_new("- replay spice server
> recording");
>      g_option_context_add_main_entries(context, entries, NULL);
> +    g_option_context_set_description(context, description);
>      if (!g_option_context_parse(context, &argc, &argv, &error)) {
>          g_printerr("Option parsing failed: %s\n", error->message);
>          exit(1);
> @@ -311,6 +332,11 @@ int main(int argc, char **argv)
>      g_option_context_free(context);
>      context = NULL;
>  
> +    if (compression <= SPICE_IMAGE_COMPRESSION_INVALID
> +        || compression >= SPICE_IMAGE_COMPRESSION_ENUM_END) {
> +        g_printerr("invalid compression value\n");
> +        exit(1);
> +    }
>      if (streaming < 0 || streaming == SPICE_STREAM_VIDEO_INVALID) {
>          g_printerr("invalid streaming value\n");
>          exit(1);