Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

File path filters #18

Merged
merged 6 commits into from Apr 23, 2013

Conversation

Projects
None yet
2 participants
Contributor

ZECTBynmo commented Apr 19, 2013

I added a new option that allows you the user to filter paths that contain certain string keys. It only works for async walking, but it really helps read large directories fast, when you know you can ignore certain things.

The overhead for non-users should be just a variable declaration and an if statement per directory read.

Owner

coolaj86 commented Apr 20, 2013

is there a reason to call this asyncFilters instead of just filters?

Contributor

ZECTBynmo commented Apr 20, 2013

Yes: it only works for asynchronous walking.

Contributor

ZECTBynmo commented Apr 20, 2013

Try that, it should work for both.

@coolaj86 coolaj86 added a commit that referenced this pull request Apr 23, 2013

@coolaj86 coolaj86 Merge pull request #18 from ZECTBynmo/master
File path filters
49e9260

@coolaj86 coolaj86 merged commit 49e9260 into Daplie:master Apr 23, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment