assert!(p, "…") → if !p { panic!("…"); }

VOICEVOX · Sep 30, 2023 · b27f1b0 · b27f1b0
1 parent 2561fda
commit b27f1b0
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 12 deletions.
diff --git a/crates/voicevox_core_c_api/src/drop_check.rs b/crates/voicevox_core_c_api/src/drop_check.rs
@@ -48,12 +48,13 @@ impl CStringDropChecker {
 
         let ptr = s.as_ptr();
         let duplicated = !owned_str_addrs.insert(ptr as usize);
-        assert!(
-            !duplicated,
-            "別の{ptr:p}が管理下にあります。原因としては以前の文字列{ptr:p}が誤った形で解放された\
-             ことが考えられます。このライブラリで生成したオブジェクトの解放は、このライブラリ\
-             が提供するAPIで行われなくてはなりません",
-        );
+        if duplicated {
+            panic!(
+                "別の{ptr:p}が管理下にあります。原因としては以前の文字列{ptr:p}が誤った形で解放\
+                 されたことが考えられます。このライブラリで生成したオブジェクトの解放は、この\
+                 ライブラリが提供するAPIで行われなくてはなりません",
+            );
+        }
         s
     }
 

diff --git a/crates/voicevox_core_c_api/src/slice_owner.rs b/crates/voicevox_core_c_api/src/slice_owner.rs
@@ -47,12 +47,13 @@ impl<T> SliceOwner<T> {
         let len = slice.len();
 
         let duplicated = slices.insert(ptr as usize, slice.into()).is_some();
-        assert!(
-            !duplicated,
-            "別の{ptr:p}が管理下にあります。原因としては以前の配列{ptr:p}が誤った形で解放された\
-             ことが考えられます。このライブラリで生成したオブジェクトの解放は、このライブラリ\
-             が提供するAPIで行われなくてはなりません",
-        );
+        if duplicated {
+            panic!(
+                "別の{ptr:p}が管理下にあります。原因としては以前の配列{ptr:p}が誤った形で解放\
+                 されたことが考えられます。このライブラリで生成したオブジェクトの解放は、この\
+                 ライブラリが提供するAPIで行われなくてはなりません",
+            );
+        }
 
         out_ptr.as_ptr().write_unaligned(ptr);
         out_len.as_ptr().write_unaligned(len);