[virt-tools-list] [virt-viewer v2 4/4] Hide all windows on disconnection

Christophe Fergeau cfergeau at redhat.com
Wed Oct 30 12:52:29 UTC 2013


On Wed, Oct 30, 2013 at 07:53:13AM -0400, Marc-André Lureau wrote:
> Why is app->priv->main_window special cased? It should be already window 0.

Ah ok, I'll change it then, I didn't really look a lot at main_window vs
windows so didn't notice.

> 
> tbh, I think g_hash_table_foreach() would be simpler (like done with the rest of the code).

I'll change it for consistency reasons, but imo it makes things less
readable.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20131030/4fff7d6b/attachment.sig>


More information about the virt-tools-list mailing list