[virt-tools-list] [PATCH 1/4] virt-viewer-window: Remove useless tests for priv->window != NULL

Marc-André Lureau marcandre.lureau at gmail.com
Mon Mar 5 17:30:17 UTC 2012


On Mon, Mar 5, 2012 at 4:22 PM, Hans de Goede <hdegoede at redhat.com> wrote:
> priv->window gets set on init and never unset, so there is no need to check
> for it.

It's remaining code from the browser plugin support, which has been
broken for a long time.. if you continue in this direction, you'll
probably end up removing all related code.. instead, I think we should
try to keep in mind that a window may not always be available etc.. At
least, that's what we tried to do I think..

-- 
Marc-André Lureau




More information about the virt-tools-list mailing list