crimson/osd: replace "ceph_abort_msg()" with assert()

these are programming errors, and are easy to detect. also assert() does
not return, so compiler won't complain at seeing a branch does not
return value in a function that returns value.

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/crimson/osd/object_context.h b/src/crimson/osd/object_context.h
index 6f158bbd64ecf8c3bb70ae75fb7a6857dd31776f..8f25420948b0bde8f8c2e40ae3d8eefd2bcdaa7c 100644 (file)
--- a/src/crimson/osd/object_context.h
+++ b/src/crimson/osd/object_context.h
@@ -123,8 +123,7 @@ public:
     case RWState::RWNONE:
       return seastar::make_ready_future<>();
     default:
-      ceph_abort_msg("invalid lock type");
-      return seastar::make_ready_future<>();
+      assert(0 == "invalid lock type");
     }
   }
 
@@ -139,8 +138,7 @@ public:
     case RWState::RWNONE:
       return;
     default:
-      ceph_abort_msg("invalid lock type");
-      return;
+      assert(0 == "invalid lock type");
     }
   }