Re: [Libvirt-cim] [PATCH 5 of 5] RAFP: adopt RASD interface changes

6 Mar 2008

      HE> -        s = get_pool_by_name(_BROKER,
HE> -                             ref,
HE> -                             poolid,
HE> -                             &pool);
HE> +        s = get_pool_by_name(_BROKER, ref, poolid, &pool);

Please don't needlessly reformat lines that are not part of the
function changes of a patch.

HE> +        s = enum_rasds(_BROKER, ref, NULL, type,
HE> +                       info->properties, &tmp_list);

Everywhere else, we make multi-line function calls have one parameter
per line (with a few pattern exceptions for cu_statusf(), etc).

Otherwise I'm happy with this set :)

-- 
Dan Smith
IBM Linux Technology Center
Open Hypervisor Team
email: danms@us.ibm.com