Skip to content

Commit 7a49f8d

Browse files
Merge pull request #1 from ckb-cell/bugfix/empty-params
fix: empty params shouldn't represent as null
2 parents 0af5a1d + add107b commit 7a49f8d

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/components/bitcoin_client.rs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,19 @@ impl Clone for BitcoinClient {
3030
}
3131
}
3232

33+
#[derive(Serialize, Clone, Copy)]
34+
struct ZeroElemTuple();
35+
3336
// ### Warning
3437
//
3538
// If parameters contain only one parameter:
3639
// - `serde_json::to_value(($($arg,)+))`
3740
// - `serde_json::to_value(($($arg),+))`
3841
// are different.
42+
//
43+
// Ref: https://github.com/serde-rs/serde/issues/1309
3944
macro_rules! serialize_parameters {
40-
() => ( serde_json::Value::Null );
45+
() => ( serde_json::to_value(ZeroElemTuple())?);
4146
($($arg:ident),+) => ( serde_json::to_value(($($arg,)+))?)
4247
}
4348

0 commit comments

Comments
 (0)