Fixes for libcurl FTP wrapper #528

Merged
merged 1 commit into from Apr 7, 2012

Projects

None yet

2 participants

@jcd
jcd commented Apr 7, 2012

Automatically add protocol if not present to improve protocol detection.
Also set upload curl option on FTP file uploads

http://forum.dlang.org/post/neqneotikeekqplzqtwu@forum.dlang.org

@andralex
D Programming Language member

Could use startsWith with a custom predicate to avoid converting the url to lowercase if performance is important here. (Not a breaker for acceptance.)

I myself think this is more readable than a custom predicate. So I would like to keep it like this until performance becomes a problem.

@andralex andralex merged commit d6d9fe0 into dlang:master Apr 7, 2012
@schveiguy schveiguy referenced this pull request in schveiguy/phobos Apr 15, 2012
Merged

std.process ready for review? #1

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