osd: no error checking in data structure (caller should do it)

diff --git a/src/osd/osd_types.h b/src/osd/osd_types.h
index 4a50d9269762e177dffbcb56d3c2ec713763df68..442c8582e1808df78c155d9729fd59277da4f6c7 100644 (file)
--- a/src/osd/osd_types.h
+++ b/src/osd/osd_types.h
@@ -653,26 +653,23 @@ struct pg_pool_t {
     snaps[s].name = n;
     snaps[s].stamp = stamp;
   }
-#define EINVAL 22
-  int add_unmanaged_snap(__u64& snapid) {
+  void add_unmanaged_snap(__u64& snapid) {
     if (removed_snaps.empty()) {
-      if (!snaps.empty()) return -EINVAL; //already has pool snaps
+      assert(snaps.empty());
       removed_snaps.insert(snapid_t(1));
       v.snap_seq = 1;
     }
     snapid = v.snap_seq = v.snap_seq + 1;
-    return 0;
   }
   void remove_snap(snapid_t s) {
     assert(snaps.count(s));
     snaps.erase(s);
     v.snap_seq = v.snap_seq + 1;
   }
-  int remove_unmanaged_snap(snapid_t s) {
-    if (!snaps.empty()) return -EINVAL; //has pool snaps, not unmanaged!
+  void remove_unmanaged_snap(snapid_t s) {
+    assert(snaps.empty());
     removed_snaps.insert(s);
     v.snap_seq = v.snap_seq + 1;
-    return 0;
   }
 
   SnapContext get_snap_context() const {