Re: [PATCH v2 3/4] osinfo_loader: replace some xpath queries with direct data access

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

 



Eric Blake <eblake@xxxxxxxxxx> writes:

> On 06/03/2014 12:19 PM, Giuseppe Scrivano wrote:
>> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx>
>> ---
>
>> -    for (i = 0 ; i < nrelated ; i++) {
>> -        gchar *id = (gchar *)xmlGetProp(related[i], BAD_CAST "id");
>> +    for(it = root->children; it; it = it->next) {
>
> Style nit - isn't there supposed to be a space after 'for' ?

thanks to have catched it.  Fixed in my local version.

I was hoping for "make syntax-check" to catch this kind of errors, but
the rule wasn't there.

This should help in future:

  https://www.redhat.com/archives/libosinfo/2014-June/msg00025.html

Regards,
Giuseppe

_______________________________________________
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