starknet_committer,starknet_patricia: don't use impl Storage, use generics

[dorimedini-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[dorimedini-starkware]

• starknet_committer_cli: use Box<dyn Storage> for storage init #9851
• starknet_committer,starknet_patricia: don't use impl Storage, use generics #9850 👈 (View in Graphite)
• starknet_patricia_storage: add short-key storage #9849 : 1 other dependent PR (#9865 )
• main-v0.14.1-committer

This stack of pull requests is managed by Graphite. Learn more about stacking.

[dorimedini-starkware]

Reviewable status: 0 of 10 files reviewed, all discussions resolved (waiting on @nimrod-starkware and @rotem-starkware)

---

a discussion (no related file):
without this change, the next PR (passing Box<dyn Storage>) is not possible

[github-actions]

Benchmark movements: No major performance changes detected.

[nimrod-starkware]

@nimrod-starkware reviewed 10 of 10 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @rotem-starkware)

---

crates/starknet_committer/src/block_committer/commit.rs line 26 at r1 (raw file):

type BlockCommitmentResult<T> = Result<T, BlockCommitmentError>;

pub async fn commit_block<S: Storage + ?Sized>(

Can you please explain why is it needed?

Code quote:

S: Storage + ?Sized

[nimrod-starkware]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @rotem-starkware)