Skip to content

Windows: Fix std.fs.Dir.deleteDir() deleting files - #5538

Merged
andrewrk merged 1 commit into
ziglang:masterfrom
squeek502:win-deletedir
Jun 5, 2020
Merged

Windows: Fix std.fs.Dir.deleteDir() deleting files#5538
andrewrk merged 1 commit into
ziglang:masterfrom
squeek502:win-deletedir

Conversation

@squeek502

@squeek502squeek502 commented Jun 4, 2020•
edited
Loading

Copy link
Copy Markdown
Member

Would previously delete files, now correctly returns error.NotDir. Fixes#5536

@squeek502
Windows: Fix std.fs.Dir.deleteDir() deleting files
b2982c2
Would previously delete files, now correctly returns error.NotDir. Fixesziglang#5536
@andrewrk

andrewrk commented Jun 5, 2020

Copy link
Copy Markdown
Member

Thanks for the fix! Would be nice to add test coverage for this but the fix alone is welcome too

@andrewrk
andrewrk merged commit f7b6957 into ziglang:masterJun 5, 2020
@squeek502

squeek502 commented Jun 5, 2020•
edited
Loading

Copy link
Copy Markdown
MemberAuthor

Would be nice to add test coverage for this

Agreed. All of std.fs (and other fs related stuff) is rather undertested imo; might take a stab at improving coverage overall at some point.

@daurnimatordaurnimator added os-windows Microsoft Windows standard library This issue involves writing Zig code for the standard library. labels Jun 8, 2020
@squeek502squeek502 mentioned this pull request Jun 20, 2020
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

os-windowsMicrosoft Windowsstandard libraryThis issue involves writing Zig code for the standard library.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Windows: std.fs.Dir.deleteDir() will delete files

3 participants

@squeek502@andrewrk@daurnimator