Skip to content
This repository has been archived by the owner on Aug 27, 2021. It is now read-only.

fixing O(too_much) bug in recursive list_subdirs #248

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

Conversation

biappi
Copy link

@biappi biappi commented May 2, 2015

for some reason this function was scanning subdirectories more than once, resulting in a really sub-ottimal performance.

with this patch, i was able to reduce build of the project i'm working on from:
real 1m1.629s

to:
real 0m1.923s

on some occasion i was also able to fix an:
(ino-env)Minene:ino willy$ time ino build > /dev/null
make: /bin/sh: Argument list too long
make: *** [.build/uno/src/iec_driver.d] Error 1

real 0m32.280s
user 0m16.205s
sys 0m13.660s

error.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant