[virt-tools-list] [PATCH virt-viewer] Fix recent --spice-controller regression, add error message

Marc-André Lureau marcandre.lureau at gmail.com
Fri Mar 30 11:37:44 UTC 2012


On Fri, Mar 30, 2012 at 8:38 AM, Hans de Goede <hdegoede at redhat.com> wrote:
> Looks good, ack.
>
> One remark though (just a remark as the behavior is
> from before this test). Wouldn't it be better to error
> out when g_strv_length(args) != 1, rather then showing
> the connection dialog as if no args were given?

You mean if g_strv_length(args) > 1, error out? Yes, definetely,
something like that?

    if (!args || g_strv_length(args) == 0) {
        if (connect_dialog(&uri) != 0)
            goto cleanup;
    } else if (g_strv_length(args) > 1) {
        g_printerr(_("Error: you must specify a single URI\n"));
    } else {
        uri = g_strdup(args[0]);
    }

-- 
Marc-André Lureau




More information about the virt-tools-list mailing list