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
Add fallback impl for FileSystem createTemporaryDirectory #2626
Conversation
I think the EWS failures are unrelated (haven't looked at the wincairo failure yet), is this ready for review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Source/WTF/wtf/FileSystem.cpp
Outdated
newTempDirTemplate += "XXXXXXXX"; | ||
|
||
Vector<char> newTempDir(newTempDirTemplate.c_str(), newTempDirTemplate.size()); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I don't think we need this new line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've made the changes and I think it's ready
This change contains multiple commits which are not squashed together, blocking PR #2626. Please squash the commits to land. |
We can only have 1 commit per PR. You currently have 2 commits so our infrastructure is unable to merge the PR. |
https://bugs.webkit.org/show_bug.cgi?id=232499 Reviewed by Chris Dumez. * Source/WTF/wtf/FileSystem.cpp: (WTF::FileSystemImpl::createTemporaryDirectory): * Source/WTF/wtf/FileSystem.h: Canonical link: https://commits.webkit.org/252799@main
39dc894
to
00e1e0d
Compare
Committed 252799@main (00e1e0d): https://commits.webkit.org/252799@main Reviewed commits have been landed. Closing PR #2626 and removing active labels. |
00e1e0d