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

Use File.list and File.walk within a try with resource #5718

Merged
merged 5 commits into from Nov 24, 2020
Merged

Use File.list and File.walk within a try with resource #5718

merged 5 commits into from Nov 24, 2020

Commits on Nov 24, 2020

  1. Use File.list and File.walk within a try with resource 

    The API contract of File.list and File.walk requires them to be closed after use.
    @gregw
    gregw committed Nov 24, 2020
    Configuration menu
    Copy the full SHA
    7627b42 View commit details
    Browse the repository at this point in the history

  2. Fix from review 

    Left out filter
    @gregw
    gregw committed Nov 24, 2020
    Configuration menu
    Copy the full SHA
    dd196e7 View commit details
    Browse the repository at this point in the history

  3. Fix from review 

    Factored out deleteFile with better debug
    @gregw
    gregw committed Nov 24, 2020
    Configuration menu
    Copy the full SHA
    5a63e60 View commit details
    Browse the repository at this point in the history

  4. Fix from review 

    Can delete files whilst walking
    @gregw
    gregw committed Nov 24, 2020
    Configuration menu
    Copy the full SHA
    a83462d View commit details
    Browse the repository at this point in the history

  5. Fix from review 

    Restored sweepFile
fixed minor code suggestions
    @gregw
    gregw committed Nov 24, 2020
    Configuration menu
    Copy the full SHA
    154c679 View commit details
    Browse the repository at this point in the history