On 07/19/2016 12:43 PM, Christophe Fergeau wrote: > On Sun, Jul 17, 2016 at 11:13:05PM -0300, Eduardo Lima (Etrunko) wrote: >> With this commit, we finish cleaning up ovirt foreign menu code, which >> only deals with data, leaving the UI bits to be handled properly in the >> new ISO list dialog. > > I'm not clear exactly what this is doing from the commit log. This > removes the old toplevel "change cd" menu, replaces it with a > File/Change CD menu entry, which is going to be not doing anything for > now? > Yes, that's it. It is strange, but I avoided removing old code and replacing with lots of new code only to keep the functionality working. Same happens with the dialog code, which is introduced with no functionality at all, and only plugged into oVirt foreign menu later on. >> - ovirt_foreign_menu_start(foreign_menu); >> -} >> >> + ovirt_foreign_menu_updated(self); > > Why do we use _updated() instead of _start()? > _updated() will only make the menu option visible, while I moved the call to _start() when dialog is shown. This whole naming seems wrong btw, because those functions are static to remote-viewer.c and not present in ovirt-foreign-menu.c. -- Eduardo de Barros Lima (Etrunko) Software Engineer - RedHat etrunko@xxxxxxxxxx _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list