Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make c2rust just forward args to discovered subcommands #579

Merged
merged 7 commits into from
Aug 25, 2022
Merged

Make c2rust just forward args to discovered subcommands #579

merged 7 commits into from
Aug 25, 2022

Commits on Aug 23, 2022

  1. Allow all discovered c2rust subcommands and don't do subcommand arg… 

    … parsing in the `c2rust` binary.
    @kkysen
    kkysen committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    a4219b5 View commit details
    Browse the repository at this point in the history

  2. Deleted old clap code I tried that didn't work and moved the failur… 

    …e explanations to comments.
    @kkysen
    kkysen committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    5956ff5 View commit details
    Browse the repository at this point in the history

  3. Replaced 2 args.next()s with 1 args.nth(1) to correctly propagate… 

    … a `None` on the first `args.next()`.
    @kkysen
    kkysen committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    3cacb96 View commit details
    Browse the repository at this point in the history

  4. Simplified control flow logic for the .strip_prefixs and used `Path… 

    …::to_str` over `UtfPath::to_str` since we just want to skip the file, not error on it.
    @kkysen
    kkysen committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    18e95f5 View commit details
    Browse the repository at this point in the history

  5. Refactored things to use more .filter()s, .and_then()s, and `.map… 

    …()`s to make it cleaner and more concise.
    @kkysen
    kkysen committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    c4e17fd View commit details
    Browse the repository at this point in the history

  6. Replaced the match and continue with an if let.

    @kkysen
    kkysen committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    4ee2582 View commit details
    Browse the repository at this point in the history

Commits on Aug 25, 2022

  1. Merge branch 'master' into kkysen/c2rust-forward

    @kkysen
    kkysen authored Aug 25, 2022
    Configuration menu
    Copy the full SHA
    3f58677 View commit details
    Browse the repository at this point in the history