From: Ján Tomko <jtomko@redhat.com>
Date: Fri, 3 Jun 2016 16:54:43 +0000 (+0200)
Subject: Remove PROPERTY_* constants
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=c40ed871cc7f90c5ca2acca95a95c73bd3151bfd;p=libvirt.git

Remove PROPERTY_* constants

They are no longer used.
---

diff --git a/src/node_device/node_device_udev.c b/src/node_device/node_device_udev.c
index d9d9887830..8b6cebd092 100644
--- a/src/node_device/node_device_udev.c
+++ b/src/node_device/node_device_udev.c
@@ -206,10 +206,10 @@ static int udevGetUint64SysfsAttr(struct udev_device *udev_device,
 
     if (str && virStrToLong_ull(str, NULL, 0, value) < 0) {
         VIR_ERROR(_("Failed to convert '%s' to unsigned long long"), str);
-        return PROPERTY_ERROR;
+        return -1;
     }
 
-    return str == NULL ? PROPERTY_MISSING : PROPERTY_FOUND;
+    return 0;
 }
 
 
@@ -687,18 +687,12 @@ static int udevProcessDisk(struct udev_device *device,
     virNodeDevCapDataPtr data = &def->caps->data;
     int ret = 0;
 
-    if (udevGetUint64SysfsAttr(device,
-                               "size",
-                               &data->storage.num_blocks) == PROPERTY_ERROR) {
+    if (udevGetUint64SysfsAttr(device, "size", &data->storage.num_blocks) < 0)
         goto out;
-    }
 
-    if (udevGetUint64SysfsAttr(device,
-                               "queue/logical_block_size",
-                               &data->storage.logical_block_size)
-        == PROPERTY_ERROR) {
+    if (udevGetUint64SysfsAttr(device, "queue/logical_block_size",
+                               &data->storage.logical_block_size) < 0)
         goto out;
-    }
 
     data->storage.size = data->storage.num_blocks *
         data->storage.logical_block_size;
@@ -729,17 +723,13 @@ static int udevProcessRemoveableMedia(struct udev_device *device,
                                   &data->storage.media_label) < 0)
             goto out;
 
-        if (udevGetUint64SysfsAttr(device,
-                                   "size",
-                                   &data->storage.num_blocks) == PROPERTY_ERROR) {
+        if (udevGetUint64SysfsAttr(device, "size",
+                                   &data->storage.num_blocks) < 0)
             goto out;
-        }
 
-        if (udevGetUint64SysfsAttr(device,
-                                   "queue/logical_block_size",
-                                   &data->storage.logical_block_size) == PROPERTY_ERROR) {
+        if (udevGetUint64SysfsAttr(device, "queue/logical_block_size",
+                                   &data->storage.logical_block_size) < 0)
             goto out;
-        }
 
         /* XXX This calculation is wrong for the qemu virtual cdrom
          * which reports the size in 512 byte blocks, but the logical
@@ -801,18 +791,13 @@ static int udevProcessSD(struct udev_device *device,
     virNodeDevCapDataPtr data = &def->caps->data;
     int ret = 0;
 
-    if (udevGetUint64SysfsAttr(device,
-                               "size",
-                               &data->storage.num_blocks) == PROPERTY_ERROR) {
+    if (udevGetUint64SysfsAttr(device, "size",
+                               &data->storage.num_blocks) < 0)
         goto out;
-    }
 
-    if (udevGetUint64SysfsAttr(device,
-                               "queue/logical_block_size",
-                               &data->storage.logical_block_size)
-        == PROPERTY_ERROR) {
+    if (udevGetUint64SysfsAttr(device, "queue/logical_block_size",
+                               &data->storage.logical_block_size) < 0)
         goto out;
-    }
 
     data->storage.size = data->storage.num_blocks *
         data->storage.logical_block_size;
diff --git a/src/node_device/node_device_udev.h b/src/node_device/node_device_udev.h
index 47eeb1f3f2..9a07ab77e4 100644
--- a/src/node_device/node_device_udev.h
+++ b/src/node_device/node_device_udev.h
@@ -29,6 +29,3 @@ typedef struct _udevPrivate udevPrivate;
 #define DRV_STATE_UDEV_MONITOR(ds) (((udevPrivate *)((ds)->privateData))->udev_monitor)
 #define DMI_DEVPATH "/sys/devices/virtual/dmi/id"
 #define DMI_DEVPATH_FALLBACK "/sys/class/dmi/id"
-#define PROPERTY_FOUND 0
-#define PROPERTY_MISSING 1
-#define PROPERTY_ERROR -1