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.

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

Adjustments to std.file.readLink for issue 6814. #294

Merged
merged 1 commit into from Oct 31, 2011

Conversation

jmdavis
Copy link
Member

@jmdavis jmdavis commented Oct 16, 2011

@jmdavis
Copy link
Member Author

jmdavis commented Oct 19, 2011

I can do that. The amount size of the buffer and the amount to grow it by are fairly arbitrary as it is. I originally picked 2048 with the idea that the odds of it being passed were low, so hopefully the odds of it being passed are indeed low. I'm aware of limitations on file name size, but I'm not aware of limitations on path size (though it wouldn't entirely surprise me if there were one).

@jmdavis
Copy link
Member Author

jmdavis commented Oct 20, 2011

Okay. I change the value to 1.5.

@jmdavis
Copy link
Member Author

jmdavis commented Oct 20, 2011

That's odd. Andrei's comment disappeared, and he's no longer listed as participating in this pull request...

andralex added a commit that referenced this pull request Oct 31, 2011
Adjustments to std.file.readLink for issue 6814.
@andralex andralex merged commit 3d68580 into dlang:master Oct 31, 2011
marler8997 pushed a commit to marler8997/phobos that referenced this pull request Nov 10, 2019
Modify `which` function to match Windows search behavior.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants