Dirlist #3

Closed
wants to merge 4 commits into
from

Conversation

Projects
None yet
1 participant
@andre-d

andre-d commented May 25, 2012

I don't really expect you to pull this, but here is the (working) dir traversing version.

@andre-d

This comment has been minimized.

Show comment Hide comment
@andre-d

andre-d May 25, 2012

Owner

Whops, I never free() that

Whops, I never free() that

@andre-d

This comment has been minimized.

Show comment Hide comment
@andre-d

andre-d May 25, 2012

I am unsure if doing it C style with strtok would save many lines...maybe....Either way, proof of concept, doesn't have to be amazing code :P

andre-d commented May 25, 2012

I am unsure if doing it C style with strtok would save many lines...maybe....Either way, proof of concept, doesn't have to be amazing code :P

@andre-d

This comment has been minimized.

Show comment Hide comment
@andre-d

andre-d May 25, 2012

Actually, since we know the original path and the new path should be the exact same length....you should be able to save a bunch of lines by just copying the old path and writing over the characters where the case is different (rather than constructing a new path).

edit: And then there would be less string concatenation/etc

andre-d commented May 25, 2012

Actually, since we know the original path and the new path should be the exact same length....you should be able to save a bunch of lines by just copying the old path and writing over the characters where the case is different (rather than constructing a new path).

edit: And then there would be less string concatenation/etc

@andre-d andre-d closed this May 25, 2012

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