Re: [[PATCHv2] 4/7] media: Add variant info API/XML

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

 



On Thu, Nov 28, 2013 at 8:24 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> On Thu, Nov 28, 2013 at 01:07:14AM +0000, Zeeshan Ali (Khattak) wrote:
>> +    g_return_val_if_fail(OSINFO_IS_MEDIA(media), NULL);
>> +    os = g_weak_ref_get(&media->priv->os);
>> +    os_variants = osinfo_os_get_variant_list(os);
>> +    g_object_unref (os);
>> +
>> +    ids = osinfo_entity_get_param_value_list(OSINFO_ENTITY(media),
>> +                                             OSINFO_MEDIA_PROP_VARIANT);
>> +    filter = osinfo_filter_new();
>> +    media_variants = osinfo_os_variantlist_new();
>> +    for (node = ids; node != NULL; node = node->next) {
>> +        osinfo_filter_clear_constraints(filter);
>> +        osinfo_filter_add_constraint(filter,
>> +                                     OSINFO_ENTITY_PROP_ID,
>> +                                     (const char *) node->data);
>> +        osinfo_list_add_filtered(OSINFO_LIST(media_variants),
>> +                                 OSINFO_LIST(os_variants),
>> +                                 filter);
>> +    }
>> +    g_object_unref (os_variants);
>
> Spacing is off here (space before '(' )

I missed this but I've corrected this for both cases above, locally.

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux