librbd: fix false-positive compiler warning

Signed-off-by: Jason Dillaman <dillaman@redhat.com>

diff --git a/src/cls/rbd/cls_rbd_types.h b/src/cls/rbd/cls_rbd_types.h
index 5d224a851d96a8d9fbcd7bf3414daeaba4cd8c84..b77212d6435ce473471d36e4695b9bfbe9c88bbc 100644 (file)
--- a/src/cls/rbd/cls_rbd_types.h
+++ b/src/cls/rbd/cls_rbd_types.h
@@ -226,12 +226,12 @@ WRITE_CLASS_ENCODER(GroupSpec);
 
 enum SnapshotNamespaceType {
   SNAPSHOT_NAMESPACE_TYPE_USER = 0,
-  SNAPSHOT_NAMESPACE_TYPE_GROUP = 1,
-  SNAPSHOT_NAMESPACE_TYPE_UNKNOWN = -1,
+  SNAPSHOT_NAMESPACE_TYPE_GROUP = 1
 };
 
 struct UserSnapshotNamespace {
-  static const SnapshotNamespaceType SNAPSHOT_NAMESPACE_TYPE = SNAPSHOT_NAMESPACE_TYPE_USER;
+  static const SnapshotNamespaceType SNAPSHOT_NAMESPACE_TYPE =
+    SNAPSHOT_NAMESPACE_TYPE_USER;
 
   UserSnapshotNamespace() {}
 
@@ -253,18 +253,19 @@ struct UserSnapshotNamespace {
 std::ostream& operator<<(std::ostream& os, const UserSnapshotNamespace& ns);
 
 struct GroupSnapshotNamespace {
-  static const SnapshotNamespaceType SNAPSHOT_NAMESPACE_TYPE = SNAPSHOT_NAMESPACE_TYPE_GROUP;
+  static const SnapshotNamespaceType SNAPSHOT_NAMESPACE_TYPE =
+    SNAPSHOT_NAMESPACE_TYPE_GROUP;
 
   GroupSnapshotNamespace() {}
 
   GroupSnapshotNamespace(int64_t _group_pool,
                         const string &_group_id,
-                        const string &_group_snapshot_id) :group_pool(_group_pool),
-                                                     group_id(_group_id),
-                                                     group_snapshot_id(_group_snapshot_id) {}
+                        const string &_group_snapshot_id)
+    : group_id(_group_id), group_pool(_group_pool),
+      group_snapshot_id(_group_snapshot_id) {}
 
-  int64_t group_pool = 0;
   string group_id;
+  int64_t group_pool = 0;
   string group_snapshot_id;
 
   void encode(bufferlist& bl) const;
@@ -294,7 +295,8 @@ struct GroupSnapshotNamespace {
 std::ostream& operator<<(std::ostream& os, const GroupSnapshotNamespace& ns);
 
 struct UnknownSnapshotNamespace {
-  static const SnapshotNamespaceType SNAPSHOT_NAMESPACE_TYPE = SNAPSHOT_NAMESPACE_TYPE_UNKNOWN;
+  static const SnapshotNamespaceType SNAPSHOT_NAMESPACE_TYPE =
+    static_cast<SnapshotNamespaceType>(-1);
 
   UnknownSnapshotNamespace() {}
 

diff --git a/src/include/rbd/librbd.h b/src/include/rbd/librbd.h
index 59f61383d80e1692c89bada3f01cd57365104645..aa0490a05d86954fd8e88c24d7759d479b7d0d10 100644 (file)
--- a/src/include/rbd/librbd.h
+++ b/src/include/rbd/librbd.h
@@ -181,8 +181,7 @@ typedef struct {
 
 typedef enum {
   SNAP_NAMESPACE_TYPE_USER = 0,
-  SNAP_NAMESPACE_TYPE_GROUP = 1,
-  SNAP_NAMESPACE_TYPE_UNKNOWN = -1,
+  SNAP_NAMESPACE_TYPE_GROUP = 1
 } rbd_snap_namespace_type_t;
 
 typedef struct {

diff --git a/src/pybind/rbd/rbd.pyx b/src/pybind/rbd/rbd.pyx
index 6ff4ec14bd9707af38d71420edd738bbd8583dce..5c5af1e826e1a3a0093c884aa6e0c816ac0b9ab2 100644 (file)
--- a/src/pybind/rbd/rbd.pyx
+++ b/src/pybind/rbd/rbd.pyx
@@ -147,7 +147,6 @@ cdef extern from "rbd/librbd.h" nogil:
     ctypedef enum rbd_snap_namespace_type_t:
         _SNAP_NAMESPACE_TYPE_USER "SNAP_NAMESPACE_TYPE_USER"
         _SNAP_NAMESPACE_TYPE_GROUP "SNAP_NAMESPACE_TYPE_GROUP"
-        _SNAP_NAMESPACE_TYPE_UNKNOWN "SNAP_NAMESPACE_TYPE_UNKNOWN"
 
     ctypedef enum rbd_lock_mode_t:
         _RBD_LOCK_MODE_EXCLUSIVE "RBD_LOCK_MODE_EXCLUSIVE"
@@ -477,7 +476,6 @@ RBD_IMAGE_OPTION_DATA_POOL = _RBD_IMAGE_OPTION_DATA_POOL
 
 SNAP_NAMESPACE_TYPE_USER = _SNAP_NAMESPACE_TYPE_USER
 SNAP_NAMESPACE_TYPE_GROUP = _SNAP_NAMESPACE_TYPE_GROUP
-SNAP_NAMESPACE_TYPE_UNKNOWN = _SNAP_NAMESPACE_TYPE_UNKNOWN
 
 class Error(Exception):
     pass