Dan Smith wrote:
HE> + if (STREQC(classname, "VirtualSystemManagementService")) { HE> + s = get_vsms(ref, &inst, _BROKER); HE> + } HE> + else if (STREQC(classname, "ResourcePoolConfigurationService")) { HE> + s = rpcs_instance(ref, &inst, _BROKER); HE> + } HE> + else if (STREQC(classname, "VirtualSystemMigrationService")) { HE> + s = get_migration_service(ref, &inst, _BROKER); HE> + } HE> + if (s.rc != CMPI_RC_OK) HE> + return s;
This doesn't fit the style of the rest of the code. The "else if" statements should be on the same line as the closing brace, and there should be a blank line before the second and unrelated if.
Fixed in the updated version. -- Regards Heidi Eckhart Software Engineer IBM Linux Technology Center - Open Hypervisor