Skip to content

Commit

Permalink
Remove test for edge key length differences (#388)
Browse files Browse the repository at this point in the history
  • Loading branch information
rkuris authored Nov 28, 2023
1 parent ae01a4a commit 80a09f6
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions firewood/src/proof.rs
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,6 @@ pub enum ProofError {
InvalidProof,
#[error("invalid edge keys")]
InvalidEdgeKeys,
#[error("inconsisent edge keys")]
InconsistentEdgeKeys,
#[error("node insertion error")]
NodesInsertionError,
#[error("node not in trie")]
Expand Down Expand Up @@ -223,11 +221,6 @@ impl<N: AsRef<[u8]> + Send> Proof<N> {
return Err(ProofError::InvalidEdgeKeys);
}

// TODO(Hao): different length edge keys should be supported
if first_key.as_ref().len() != last_key.as_ref().len() {
return Err(ProofError::InconsistentEdgeKeys);
}

// Convert the edge proofs to edge trie paths. Then we can
// have the same tree architecture with the original one.
// For the first edge proof, non-existent proof is allowed.
Expand Down

0 comments on commit 80a09f6

Please sign in to comment.