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

cp: move copy_directory() to its own module #3901

Merged
merged 3 commits into from
Oct 20, 2022
Merged

cp: move copy_directory() to its own module #3901

merged 3 commits into from
Oct 20, 2022

Commits on Oct 20, 2022

  1. cp: refactor Options::preserve_hard_links() 

    Refactor common code into a helper method
`Options::preserve_hard_links()`. This also eliminates the need for
mutability in a local variable in two places.
    @jfinkels
    jfinkels committed Oct 20, 2022
    Configuration menu
    Copy the full SHA
    11d3e0f View commit details
    Browse the repository at this point in the history

  2. cp: move copy_directory to new module copydir.rs 

    Move the `copy_directory()` helper function to a new module
`copydir.rs`. This commit only changes the organization of the code,
not its behavior.
    @jfinkels
    jfinkels committed Oct 20, 2022
    Configuration menu
    Copy the full SHA
    d8146d1 View commit details
    Browse the repository at this point in the history

  3. cp: break up copy_directory() into helper funcs 

    Add some additional structs and helper functions to make the code in
`copydir.rs` easier to read and maintain. This commit changes only the
organization of the code, not its function.
    @jfinkels
    jfinkels committed Oct 20, 2022
    Configuration menu
    Copy the full SHA
    aeba601 View commit details
    Browse the repository at this point in the history