Some lines seem to be unreachable code.

[yangby-cryptape]

I have run fuzz tests again and again, the following if condition is never entered:

merkle-mountain-range/src/mmr.rs

Lines 404 to 406 in 8206b58

	if let Some(rhs_peaks_hashes) = proof_iter.next() {
	peaks_hashes.push(rhs_peaks_hashes.clone());
	}

I could NOT prove but I think it is unreachable.

[quake]

it's reachable, I added an unit test: #36

[yangby-cryptape]

Why don't just return CorruptedProof if it only could be reached when "a malicious attacker can create a proof with duplicate items".

In the unit tests that you added, it returns false.

[quake]

Yes, better to return CorruptedProof directly, and I noticed that line 408 checked the iterator empty again, so these lines can be deleted directly, cc @jjyr