shared_token_bucket: resolve FIXME

The specialization of 'struct rovers' for 'capped_release::no' does not support 'release()' member function. If it was called, it would call abort(). This change resolves FIXME that demanded this member function to not be compiled. Usage of '= delete' was introduced to break the compilation if the user tries to use release(). Signed-off-by: Patryk Wrobel <[email protected]> Closes scylladb#2308
ptrsmrn · Jul 17, 2024 · a0067d0 · a0067d0
1 parent 9e3550e
commit a0067d0
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/include/seastar/util/shared_token_bucket.hh b/include/seastar/util/shared_token_bucket.hh
@@ -83,9 +83,7 @@ struct rovers<T, capped_release::no> {
         return wrapping_difference(tail.load(std::memory_order_relaxed) + limit, head.load(std::memory_order_relaxed));
     }
 
-    void release(T) {
-        std::abort(); // FIXME shouldn't even be compiled
-    }
+    void release(T) = delete;
 };
 
 template <typename T, typename Period, capped_release Capped, typename Clock = std::chrono::steady_clock>