Re: [PATCH] util: Use new array management macros

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

 



On 07/01/14 23:16, Eric Blake wrote:
On 01/07/2014 08:03 AM, Osier Yang wrote:
Like commit 94a26c7e from Eric Blake, the old fuzzy code should
be replaced by the new array management macros now.

And the type of scsi->count should be changed into "size_t", and
thus virSCSIDeviceListCount should return size_t instead, similar
for vir{PCI,USB}DeviceListCount.
---
  src/util/virpci.c  |  2 +-
  src/util/virpci.h  |  2 +-
  src/util/virscsi.c | 35 ++++++++++-------------------------
  src/util/virscsi.h |  2 +-
  src/util/virusb.c  |  2 +-
  src/util/virusb.h  |  2 +-
  6 files changed, 15 insertions(+), 30 deletions(-)
Not quite.

@@ -387,24 +382,14 @@ virSCSIDeviceListSteal(virSCSIDeviceListPtr list,
      size_t i;
for (i = 0; i < list->count; i++) {
-        if (list->devs[i]->adapter != dev->adapter ||
-            list->devs[i]->bus != dev->bus ||
-            list->devs[i]->target != dev->target ||
-            list->devs[i]->unit != dev->unit)
In the old code, if 3 of the 4 items match, but the fourth does not,
then we skip that item.

-            continue;
-
-        ret = list->devs[i];
-
-        if (i != list->count--)
-            memmove(&list->devs[i],
-                    &list->devs[i+1],
-                    sizeof(*list->devs) * (list->count - i));
-
-        if (VIR_REALLOC_N(list->devs, list->count) < 0) {
-            ; /* not fatal */
+        if (list->devs[i]->adapter == dev->adapter ||
+            list->devs[i]->bus == dev->bus ||
+            list->devs[i]->target == dev->target ||
+            list->devs[i]->unit == dev->unit) {
In the new code as written, only 1 of the 4 items has to match.  You
applied deMorgan's theorem incorrectly; this must be:
I was too quick, fixed and pushed. Thanks.

Regards.
Osier

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




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