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

Draft: Generalize dirEntries to take a DirEntry predicate as parameter #8586

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Draft: Generalize dirEntries to take a DirEntry predicate as parameter #8586

Changes from 2 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
62b9c0c
Generalize dirEntries to take a DirEntry predicate as parameter
nordlow Oct 5, 2022
221176b
Swap template parameters pred and useDIP1000
nordlow Oct 5, 2022
4624f2e
Remove const ref from pred parameter and activate template restrictions
nordlow Oct 5, 2022
00ea32d
Pass entry by ref in pred in use __traits(compiles) instead of is()
nordlow Oct 6, 2022
5741571
Correct logic in mayStepIn
nordlow Oct 6, 2022
eea0b72
Add failing test
nordlow Oct 6, 2022
8ba61f4
Use pred in posix version of mayStepIn
nordlow Oct 12, 2022
d681179
Try to use pred in DirIteratorImpl
nordlow Oct 12, 2022
77822bf
Use final switch in DirIteratorImpl.popFront
nordlow Oct 12, 2022
308a9dc
Make test cover all span modes
nordlow Oct 12, 2022
debde5d
Merge branch 'dlang:master' into dirEntries-pred
nordlow Oct 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions std/file.d
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4783,6 +4783,8 @@ if (__traits(compiles, { DirEntry entry; bool _ = pred(entry); }))

bool mayStepIn()
{
if (!pred(_cur))
return false;
return _followSymlink ? _cur.isDir : attrIsDir(_cur.linkAttributes);
}
}
Expand Down Expand Up @@ -5198,6 +5200,12 @@ if (__traits(compiles, { DirEntry entry; bool _ = pred(entry); }))
sort(result);

assert(equal(files, result));

import std.algorithm : endsWith;
auto result2 = dirEntries!((scope ref DirEntry entry) => entry.name.endsWith("Hello World"))(dir, SpanMode.shallow).map!((return a) => a.name.normalize()).array();
import std.stdio;
writeln(result2);
LightBender marked this conversation as resolved.
Show resolved Hide resolved
assert(result2.length == 1);
}

// https://issues.dlang.org/show_bug.cgi?id=21250
Expand Down