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.

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

Mention that std::fs::remove_dir_all fails on files #133298

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Nov 21, 2024

  1. Mention that std::fs::remove_dir_all fails on files

    This is explicitly mentioned for std::fs::remove_file's documentation,
    but not in the aforementioned function.
    
    It is more likely for a slightly lazy programmer to believe that
    removing a file would work and that they do not have to distinguish
    between directories (with contents) and files themself, because of the
    function's recursive nature and how it distinguishes between files and
    directories when removing them.
    n0toose committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    197bba5 View commit details
    Browse the repository at this point in the history