On Tue, Jul 19, 2016 at 03:04:58PM -0300, Eduardo Lima (Etrunko) wrote:
> On 07/19/2016 02:54 PM, Eduardo Lima (Etrunko) wrote:
> > On 07/19/2016 12:57 PM, Christophe Fergeau wrote:
> >>> +
> >>> +self = REMOTE_VIEWER_ISO_LIST_DIALOG(dialog);
> >>> +self->priv->foreign_menu = foreign_menu;
On 07/19/2016 02:54 PM, Eduardo Lima (Etrunko) wrote:
> On 07/19/2016 12:57 PM, Christophe Fergeau wrote:
>>> +
>>> +self = REMOTE_VIEWER_ISO_LIST_DIALOG(dialog);
>>> +self->priv->foreign_menu = foreign_menu;
>>
>> I'd g_object_ref it if you need to have it around (together with
>>
On 07/19/2016 12:57 PM, Christophe Fergeau wrote:
>> +
>> +self = REMOTE_VIEWER_ISO_LIST_DIALOG(dialog);
>> +self->priv->foreign_menu = foreign_menu;
>
> I'd g_object_ref it if you need to have it around (together with
> g_clear_object
> in dispose/finalize).
Okay, fixed.
>> +
>>
On Sun, Jul 17, 2016 at 11:13:10PM -0300, Eduardo Lima (Etrunko) wrote:
> The OvirtForeignMenu pointer is needed by the new ISO list dialog, and
> we make it acessible via property to avoid interdependency between
> objects.
>
> Signed-off-by: Eduardo Lima (Etrunko)
> ---
>
The OvirtForeignMenu pointer is needed by the new ISO list dialog, and
we make it acessible via property to avoid interdependency between
objects.
Signed-off-by: Eduardo Lima (Etrunko)
---
src/remote-viewer-iso-list-dialog.c | 20 +++-