Unified Diff: src/DefaultWebRequestWinInet.cpp
Issue 10369004:
File system path resolving (Closed)
Patch Set: Comments addressed
Created April 30, 2013, 8:09 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« src/DefaultFileSystem.cpp ('K')
|
« src/DefaultFileSystem.cpp ('k')
|
src/FileSystemJsObject.cpp » ('j')
|
test/BaseJsTest.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: src/DefaultWebRequestWinInet.cpp |
| =================================================================== |
| --- a/src/DefaultWebRequestWinInet.cpp |
| +++ b/src/DefaultWebRequestWinInet.cpp |
| @@ -181,7 +181,7 @@ |
| std::wstring canonizedUrl = Utils::CanonizeUrl(Utils::ToUTF16String(url, url.length())); |
| - std::string headersString; |
| + std::string headersString = ""; |
|
Oleksandr
2013/04/30 08:13:07
This fixes an issue on XP. std::string's length is
Felix Dahlke
2013/04/30 09:14:52
Worth a comment I think. Or is this not the only p
|
| for (int i = 0; i < requestHeaders.size(); i++) |
| { |
| headersString += requestHeaders[i].first + ": "; |
« src/DefaultFileSystem.cpp ('K')
|
« src/DefaultFileSystem.cpp ('k')
|
src/FileSystemJsObject.cpp » ('j')
|
test/BaseJsTest.h » ('J')
