Re: [PATCH virt-viewer 7/9] Report error in dialog

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Nov 13, 2014 at 06:20:43PM +0100, Marc-André Lureau wrote:
> Use a UI dialog to inform of connection error.


ACK

> ---
>  src/virt-viewer.c | 14 +++++++++++---
>  1 file changed, 11 insertions(+), 3 deletions(-)
> 
> diff --git a/src/virt-viewer.c b/src/virt-viewer.c
> index c1d2765..517aab8 100644
> --- a/src/virt-viewer.c
> +++ b/src/virt-viewer.c
> @@ -839,9 +839,17 @@ virt_viewer_connect(VirtViewerApp *app)
>      }
>  
>      if (!virt_viewer_app_initial_connect(app, &error)) {
> -        if (error != NULL)
> -            g_warning("%s", error->message);
> -        g_clear_error(&error);
> +        if (error != NULL) {
> +            GtkWidget *dialog = gtk_message_dialog_new(NULL,
> +                                                       GTK_DIALOG_DESTROY_WITH_PARENT,
> +                                                       GTK_MESSAGE_ERROR,
> +                                                       GTK_BUTTONS_CLOSE,
> +                                                       "Failed to connect: %s",
> +                                                       error->message);
> +            gtk_dialog_run(GTK_DIALOG(dialog));
> +            gtk_widget_destroy(GTK_WIDGET(dialog));
> +            g_clear_error(&error);
> +        }
>          return -1;
>      }
>  
> -- 
> 1.9.3
> 
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/virt-tools-list

Attachment: pgpzP6T8a6jLD.pgp
Description: PGP signature

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list

[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux