chore: remove hash generic args from the different tree traits

[TzahiTaub]

This change is 

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 18.88%. Comparing base (a5db564) to head (f8fdece).

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   18.88%   18.88%           
=======================================
  Files           7        7           
  Lines          90       90           
  Branches       90       90           
=======================================
  Hits           17       17           
  Misses         73       73           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dorimedini-starkware]

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @amosStarkware and @TzahiTaub)

---

crates/committer/src/patricia_merkle_tree/updated_skeleton_tree.rs line 18 at r1 (raw file):

        &self,
        tree_hash_func: &impl TreeHashFunction<L, H>,
    ) -> Result<impl FilledTree<L>, UpdatedSkeletonTreeError>;

say we define a struct PedersenTreeHash and impl<L: LeafDataTrait> TreeHashFunction<L, PedersenHash> for PedersenTreeHash; why would we need to pass an instance of PedersenTreeHash? what is tree_hash_func for?

Suggestion:

    fn compute_filled_tree<H: HashFunction, TH: TreeHashFunction<L, H>>(
        &self
    ) -> Result<impl FilledTree<L>, UpdatedSkeletonTreeError>;

[TzahiTaub]

Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @amosStarkware and @dorimedini-starkware)

---

crates/committer/src/patricia_merkle_tree/updated_skeleton_tree.rs line 18 at r1 (raw file):

Previously, dorimedini-starkware wrote…

say we define a struct PedersenTreeHash and impl<L: LeafDataTrait> TreeHashFunction<L, PedersenHash> for PedersenTreeHash; why would we need to pass an instance of PedersenTreeHash? what is tree_hash_func for?

No need, got confused.

[dorimedini-starkware]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @amosStarkware)