ledger-tool: Switch subcommand dispatch from if-else to match

[steviez]

Problem

A future change will add more cases to this if-else if-...-else chain. Using a match statement will be easier to follow then a very long if-else if-... chain.

This change was broken out in order to have a higher signal to noise ratio in the subsequent change.

Summary of Changes

Switch if-else over to match, and then run cargo fmt which will adjust white space in a lot of places. The downstream PR that would build on top of this is #34597. Namely, this section here would look like this with the match statement:

    match matches.subcommand() {
        ("bigtable", Some(arg_matches)) => bigtable_process_command(&ledger_path, arg_matches),
        ("blockstore", Some(arg_matches)) => blockstore_process_command(&ledger_path, arg_matches),
        ("program", Some(arg_matches)) => program(&ledger_path, arg_matches),
        // This match case provides legacy support for commands that were previously top level
        // subcommands of the binary, but have been moved under the blockstore subcommand.
        ("analyze-storage", Some(_))
        | ("bounds", Some(_))
        | ("copy", Some(_))
        | ("dead-slots", Some(_))
        | ("duplicate-slots", Some(_))
        | ("json", Some(_))
        | ("latest-optimistic-slots", Some(_))
        | ("list-roots", Some(_))
        | ("parse_full_frozen", Some(_))
        | ("print", Some(_))
        | ("print-file-metadata", Some(_))
        | ("purge", Some(_))
        | ("remove-dead-slot", Some(_))
        | ("repair-roots", Some(_))
        | ("set-dead-slot", Some(_))
        | ("shred-meta", Some(_))
        | ("slot", Some(_)) => blockstore_process_command(&ledger_path, &matches),
        _ => {

This PR is probably only necessary if we desire to push to #34597; so while this PR should go in first, we may want to confirm that we like the changes introduced by #34597 first.

The first commit is the actual change, the rest is a result of new indentation and line wrapping form cargo fmt

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0b49d82) 81.8% compared to head (d2a4d94) 81.8%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #34644   +/-   ##
=======================================
  Coverage    81.8%    81.8%           
=======================================
  Files         824      824           
  Lines      222393   222393           
=======================================
+ Hits       181954   181995   +41     
+ Misses      40439    40398   -41     

[CriesofCarrots]

Sweet, much better. Way to pad those addition/deletion stats, too ;)

[steviez]

Way to pad those addition/deletion stats, too ;)

Haha, it was a match statement before #21575 😉 🙈

[CriesofCarrots]

Haha, it was a match statement before #21575 😉 🙈

Lol! Mea culpa.