Merge pull request #43 from okx/yz/fix_tobigint

fix to_bigint unwrap
okx · May 16, 2023 · 8a1faa9 · 8a1faa9
2 parents 1e683ae + 2d71e03
commit 8a1faa9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/brc20/error.rs b/src/brc20/error.rs
@@ -81,6 +81,11 @@ pub enum BRC20Error {
 
   #[error("transferable owner not match {0}")]
   TransferableOwnerNotMatch(InscriptionId),
+
+  /// an InternalError is an error that happens exceed our expect
+  /// and should not happen under normal circumstances
+  #[error("internal error: {0}")]
+  InternalError(String),
 }
 
 impl<L: LedgerRead> From<BRC20Error> for Error<L> {

diff --git a/src/brc20/num.rs b/src/brc20/num.rs
@@ -71,7 +71,10 @@ impl Num {
       self
         .0
         .to_bigint()
-        .unwrap()
+        .ok_or(BRC20Error::InternalError(format!(
+          "convert {} to bigint failed",
+          self.0
+        )))?
         .to_u128()
         .ok_or(BRC20Error::Overflow {
           op: String::from("to_u128"),