On Wed, 2015-07-22 at 10:44 +0200, Peter Krempa wrote: > > > - present_cpumap = nodeGetPresentCPUBitmap(sysfs_prefix); > > - if (!present_cpumap) > > - goto cleanup; > > - online_cpus_map = nodeGetOnlineCPUBitmap(sysfs_prefix); > > - if (!online_cpus_map) > > - goto cleanup; > > - > > - npresent_cpus = virBitmapSize(present_cpumap); > > This still can be extracted here since the caller doesn't care about > the > npreset count so you are basically passing redundant info. > > ACK to the rest. I'll revert this part and adjust the patch > accordingly > before pushing.
Sure, it was intended as an optimization but then again we already agreed that it doesn't really make sense because virBitmapSize() does so little, plus it indeed looks cleaner if the caller is passing one less parameter. Thanks. -- Andrea Bolognani Software Engineer - Virtualization Team -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list