May 2012 - Wine-Commits - list.winehq.org

Frédéric Delanoy : Remove reference to obsolete newsgroup in Contributing page
by Jeremy Newman May 3, 2012

May 3, 2012

Module: website Branch: master Commit: 849a0de3e957447841cacd83ec5a8d412079ad4c URL: http://source.winehq.org/git/website.git/?a=commit;h=849a0de3e957447841cacd… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Thu May 3 01:22:09 2012 +0200 Remove reference to obsolete newsgroup in Contributing page --- templates/en/contributing.template | 38 ++++++++++++++++++------------------ 1 files changed, 19 insertions(+), 19 deletions(-) diff --git a/templates/en/contributing.template b/templates/en/contributing.template index d100afc..84989e7 100644 --- a/templates/en/contributing.template +++ b/templates/en/contributing.template @@ -6,7 +6,7 @@ <p>It's easy to be a bit overwhelmed when you're new to Wine. Yes there may be a lot of things to do but where should you start? Is there anything simple that can get you started? Is there anything you can do that does not require an intimate knowledge of the Windows API and of - Wine's internals? What if you don't know C? Or if you're not a programmer in the first place?</p> + Wine internals? What if you don't know C? Or if you're not a programmer in the first place?</p> <p>So here are a few projects that might provide you with a starting point. Note that this list is not an official statement of what should be done on Wine or anything. It's just a list of things that are important and useful. In any case we welcome your feedback. Similarly, if you @@ -65,7 +65,7 @@ application (they cannot buy all the applications out there) or do not know it well enough to test it properly.</p> <p>You can help even further by becoming an 'application maintainer', that is by - maintaining that application's entry in the <a href="http://appdb.winehq.org/">Application Database</a>. Many + maintaining that application entry in the <a href="http://appdb.winehq.org/">Application Database</a>. Many applications can be made to run (or run better) by using just the right mix of native (Windows) and built-in (Wine) libraries. By testing the application with various library mix and then documenting your results in the @@ -80,7 +80,7 @@ regularly than ten applications once a year.</li> <li>Report all regressions to wine-devel, and cooperate with the developers to diagnose the issue.</li> - <li>Create and maintain an HOWTO Note on the application page summarizing + <li>Create and maintain a HOWTO note on the application page summarizing all actions required to get the application to run properly, e.g. collected from the comments of visitors on the application/versions pages.</li> <li>Coding skills are <strong>not necessary</strong>.</li> @@ -92,23 +92,23 @@ many bug reports that need to be diagnosed and then many bugs to fix.</p> <p>Thus bug diagnosis and support are two closely related activities that are very important to Wine. So we need people willing to answer the questions - and issues that pop up on the newsgroup, wine-users and Bugzilla, and then + and issues that pop up on wine-users and Bugzilla, and then to triage them:</p> <ul> <li>General Wine issues can usually be answered directly</li> <li>For application-specific issues one may want to check with the application owner (if any) to see if this is a known issue (ideally such - issues should be listed in that application's entry)</li> + issues should be listed in that application entry)</li> <li>Other issues require more work to determine whether the problem is reproducible, specific to some environments, and to obtain logs that Wine developers can use to determine where the problem comes from (requires getting enough of an idea of where the problem comes from to determine which debug options to use). Once this is done, it will be much easier for Wine developers to find the exact source of the bug and - to come up with a fix. If the problem is not quickly resolved on the newsgroup - or wine-users then start a bug report in bugzilla. Search through the + to come up with a fix. If the problem is not quickly resolved on + wine-users then start a bug report in bugzilla. Search through the unconfirmed and new bug reports and get them in proper shape for developers. - Subscribe to to wine bugs (<a href="mailto:wine-bugs(a)winehq.org">wine-bugs(a)winehq.org</a>) + Subscribe to <a href="http://www.winehq.org/mailman/listinfo/wine-bugs">wine-bugs</a>. This read-only list has all entries for unassigned bugs sent to it. Confirming unconfirmed bugs and suggesting debug channels to try are just some of the tasks that need to be done.</li> @@ -121,12 +121,12 @@ users how to switch from managed to unmanaged window management, how to disable FreeType support, etc.</li> <li>General knowledge of Wine issues, mostly gleaned by following the - mailing lists and newsgroup.</li> + mailing lists.</li> <li>Knowing how to switch from native to built-in libraries and vice versa.</li> <li>Being able to analyze a log to determine which libraries to switch from built-in to native and vice versa would be a plus.</li> </ul> -<p>For more information, see our <a href="http://wiki.winehq.org/BugzillaTriage">bug triage page</a>.</p> +<p>For more information, see our <a href="http://wiki.winehq.org/BugzillaTriage">bugs triage page</a>.</p> <h2 id="devel">Development</h2> @@ -134,8 +134,8 @@ <h3 id="conf_tests">Conformance tests</h3> -<p>The goal of Wine's conformance testing is to make sure that the behavior - of Wine's APIs conforms to that of the corresponding Windows API. As there are many versions +<p>The goal of Wine conformance testing is to make sure that the behavior + of Wine APIs conforms to that of the corresponding Windows APIs. As there are many versions of Windows from 95 to 7 we need people to write tests as well as volunteers to run them on Windows. The test suite is far from complete but every little bit helps.</p> <ul> @@ -186,7 +186,7 @@ from both sides: put traces in your application and in Wine. All this combined together and makes it much easier to detect where Wine is doing something wrong, what and why. Then, since you already know what Wine is supposed to do, you're also in a better position - to provide a fix. You can also test a programming book's examples in Wine. A kit to get + to provide a fix. You can also test the examples of a programming book in Wine. A kit to get started and create Web pages showing the results is available <a href="http://fgouget.free.fr/wine/booktesting-en.shtml">here</a>.</p> <p>Required skills:</p> @@ -194,7 +194,7 @@ <li>Testing</li> <li>Writing bug reports</li> <li>Debugging to fix them if possible</li> - <li>Programming in C/C++ if you want to try compiling the programs with Wine lib</li> + <li>Programming in C/C++ if you want to try compiling the programs with Winelib</li> </ul> <h3 id ="code_review">Perform a focused code review</h3> @@ -208,7 +208,7 @@ This can take some time so one cannot always do it (drop me a line if you think of some specific bug to look for but you don't have time). But it's a nice way to get acquainted with the code and you can actually find bugs without needing months of experience on - Wine's code.</p> + Wine code.</p> <p>But be warned though that since your changes will be scattered throughout Wine they will be scrutinized by many developers. Furthermore since your changes probably will not be direct obvious bug fixes, there may be (founded) resistance. Make sure your @@ -234,7 +234,7 @@ <h3 id="doc">Documentation writing</h3> <p>Wine is in constant need of documentation updates. Wine has its - "<a href="{$root}/site/documentation">Guides</a>," like the Wine User Guide etc. + "<a href="{$root}/site/documentation">Guides</a>", like the Wine User Guide etc. The SGML source of the Guides can be found in a <a href="http://source.winehq.org/git/docs.git"> separate Git tree</a>.</p> <p>Other important documentation items are the README file @@ -256,11 +256,11 @@ <h3 id="localized">Localization work</h3> -<p>We need help translating Wine's resources into foreign languages. What this +<p>We need help translating Wine resources into foreign languages. What this covers is common dialogs such as the 'File Open' or 'Print' dialogs, informational messages such as the crash dialog, some standard error - messages, but also the builtin tools such as Wine's notepad. When not - translated these can really impair Wine's usability for non-English + messages, but also the builtin tools such as Wine notepad. When not + translated these can really impair Wine usability for non-English speakers.</p> <p>Luckily these are now all translated through standard PO files which makes this task relatively easy. For detailed instructions on how to help, see our

1 0

Michael Mc Donnell : d3dx9: Added D3DXOptimizeFaces test.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: 081b06c012b49ef0f2e4379338786aecf33b81dc URL: http://source.winehq.org/git/wine.git/?a=commit;h=081b06c012b49ef0f2e437933… Author: Michael Mc Donnell <michael(a)mcdonnell.dk> Date: Fri Mar 30 19:02:51 2012 +0200 d3dx9: Added D3DXOptimizeFaces test. --- dlls/d3dx9_36/tests/mesh.c | 155 ++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 155 insertions(+), 0 deletions(-) diff --git a/dlls/d3dx9_36/tests/mesh.c b/dlls/d3dx9_36/tests/mesh.c index 2a05e0d..b8123ab 100644 --- a/dlls/d3dx9_36/tests/mesh.c +++ b/dlls/d3dx9_36/tests/mesh.c @@ -10047,6 +10047,160 @@ cleanup: free_test_context(test_context); } +static void test_optimize_faces(void) +{ + HRESULT hr; + UINT i; + DWORD smallest_face_remap; + /* mesh0 + * + * 0--1 + * | / + * |/ + * 2 + */ + const DWORD indices0[] = {0, 1, 2}; + const UINT num_faces0 = 1; + const UINT num_vertices0 = 3; + const DWORD exp_face_remap0[] = {0}; + /* mesh1 + * + * 0--1 3 + * | / /| + * |/ / | + * 2 5--4 + */ + const DWORD indices1[] = {0, 1, 2, 3, 4, 5}; + const UINT num_faces1 = 2; + const UINT num_vertices1 = 6; + const DWORD exp_face_remap1[] = {1, 0}; + /* mesh2 + * + * 0--1 + * | /| + * |/ | + * 2--3 + */ + const DWORD indices2[] = {0, 1, 2, 1, 3, 2}; + const UINT num_faces2 = 2; + const UINT num_vertices2 = 4; + const DWORD exp_face_remap2[] = {1, 0}; + /* mesh3 + * + * 0--1 + * | /| + * |/ | + * 2--3 + * | /| + * |/ | + * 4--5 + */ + const DWORD indices3[] = {0, 1, 2, 1, 3, 2, 2, 3, 4, 3, 4, 5}; + const UINT num_faces3 = 4; + const UINT num_vertices3 = 6; + const DWORD exp_face_remap3[] = {3, 2, 1, 0}; + /* mesh4 + * + * 0--1 + * | /| + * |/ | + * 2--3 + * | /| + * |/ | + * 4--5 + */ + const WORD indices4[] = {0, 1, 2, 1, 3, 2, 2, 3, 4, 3, 4, 5}; + const UINT num_faces4 = 4; + const UINT num_vertices4 = 6; + const DWORD exp_face_remap4[] = {3, 2, 1, 0}; + /* Test cases are stored in the tc array */ + struct + { + const VOID *indices; + const UINT num_faces; + const UINT num_vertices; + const BOOL indices_are_32bit; + const DWORD *exp_face_remap; + } + tc[] = + { + { + indices0, + num_faces0, + num_vertices0, + TRUE, + exp_face_remap0 + }, + { + indices1, + num_faces1, + num_vertices1, + TRUE, + exp_face_remap1 + }, + { + indices2, + num_faces2, + num_vertices2, + TRUE, + exp_face_remap2 + }, + { + indices3, + num_faces3, + num_vertices3, + TRUE, + exp_face_remap3 + }, + { + indices4, + num_faces4, + num_vertices4, + FALSE, + exp_face_remap4 + }, + }; + + /* Go through all test cases */ + for (i = 0; i < ARRAY_SIZE(tc); i++) + { + DWORD j; + DWORD *face_remap; + face_remap = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, + tc[i].num_faces*sizeof(face_remap)); + + hr = D3DXOptimizeFaces(tc[i].indices, tc[i].num_faces, + tc[i].num_vertices, tc[i].indices_are_32bit, + face_remap); + ok(hr == D3D_OK, "D3DXOptimizeFaces test case %d failed. " + "Got %x\n, expected D3D_OK", i, hr); + + /* Compare face remap with expected face remap */ + for (j = 0; j < tc[i].num_faces; j++) + { + ok(tc[i].exp_face_remap[j] == face_remap[j], + "Test case %d: Got face %d at %d, expected %d\n", i, + face_remap[j], j, tc[i].exp_face_remap[j]); + } + + HeapFree(GetProcessHeap(), 0, face_remap); + } + + /* face_remap must not be NULL */ + hr = D3DXOptimizeFaces(tc[0].indices, tc[0].num_faces, + tc[0].num_vertices, tc[0].indices_are_32bit, + NULL); + ok(hr == D3DERR_INVALIDCALL, "D3DXOptimizeFaces passed NULL face_remap " + "pointer. Got %x\n, expected D3DERR_INVALIDCALL\n", hr); + + /* Number of faces must be smaller than 2^15 */ + hr = D3DXOptimizeFaces(tc[0].indices, 2 << 15, + tc[0].num_vertices, FALSE, + &smallest_face_remap); + ok(hr == D3DERR_INVALIDCALL, "D3DXOptimizeFaces should not accept 2^15 " + "faces when using 16-bit indices. Got %x\n, expected D3DERR_INVALIDCALL\n", hr); +} + START_TEST(mesh) { D3DXBoundProbeTest(); @@ -10072,4 +10226,5 @@ START_TEST(mesh) test_weld_vertices(); test_clone_mesh(); test_valid_mesh(); + test_optimize_faces(); }

1 0

Jacek Caban : wininet: Use stored status code in HTTP_HttpEndRequestW.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: 797a17bb2d8c54538f35e8517075ed8b4043f448 URL: http://source.winehq.org/git/wine.git/?a=commit;h=797a17bb2d8c54538f35e8517… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu May 3 12:20:10 2012 +0200 wininet: Use stored status code in HTTP_HttpEndRequestW. --- dlls/wininet/http.c | 53 +++++++++++++++++++++++--------------------------- 1 files changed, 24 insertions(+), 29 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index b04a51c..d5697ae 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -4964,9 +4964,8 @@ static void AsyncHttpSendRequestProc(WORKREQUEST *workRequest) static DWORD HTTP_HttpEndRequestW(http_request_t *request, DWORD dwFlags, DWORD_PTR dwContext) { - INT responseLen; - DWORD dwCode, dwCodeLength; DWORD dwBufferSize; + INT responseLen; DWORD res = ERROR_SUCCESS; if(!request->netconn) { @@ -4991,11 +4990,6 @@ static DWORD HTTP_HttpEndRequestW(http_request_t *request, DWORD dwFlags, DWORD_ HTTP_ProcessExpires(request); HTTP_ProcessLastModified(request); - dwCodeLength = sizeof(dwCode); - if (HTTP_HttpQueryInfoW(request,HTTP_QUERY_FLAG_NUMBER|HTTP_QUERY_STATUS_CODE, - &dwCode,&dwCodeLength,NULL) != ERROR_SUCCESS) - dwCode = 0; - if ((res = set_content_length(request)) == ERROR_SUCCESS) { if(!request->contentLength) http_release_netconn(request, TRUE); @@ -5003,32 +4997,33 @@ static DWORD HTTP_HttpEndRequestW(http_request_t *request, DWORD dwFlags, DWORD_ if (res == ERROR_SUCCESS && !(request->hdr.dwFlags & INTERNET_FLAG_NO_AUTO_REDIRECT)) { - if (dwCode == HTTP_STATUS_REDIRECT || - dwCode == HTTP_STATUS_MOVED || - dwCode == HTTP_STATUS_REDIRECT_METHOD || - dwCode == HTTP_STATUS_REDIRECT_KEEP_VERB) - { + switch(request->status_code) { + case HTTP_STATUS_REDIRECT: + case HTTP_STATUS_MOVED: + case HTTP_STATUS_REDIRECT_METHOD: + case HTTP_STATUS_REDIRECT_KEEP_VERB: { WCHAR *new_url, szNewLocation[INTERNET_MAX_URL_LENGTH]; dwBufferSize=sizeof(szNewLocation); - if (HTTP_HttpQueryInfoW(request, HTTP_QUERY_LOCATION, szNewLocation, &dwBufferSize, NULL) == ERROR_SUCCESS) + if (HTTP_HttpQueryInfoW(request, HTTP_QUERY_LOCATION, szNewLocation, &dwBufferSize, NULL) != ERROR_SUCCESS) + break; + + if (strcmpW(request->verb, szGET) && strcmpW(request->verb, szHEAD) && + request->status_code != HTTP_STATUS_REDIRECT_KEEP_VERB) { - if (strcmpW(request->verb, szGET) && strcmpW(request->verb, szHEAD) && - dwCode != HTTP_STATUS_REDIRECT_KEEP_VERB) - { - heap_free(request->verb); - request->verb = heap_strdupW(szGET); - } - drain_content(request); - if ((new_url = HTTP_GetRedirectURL( request, szNewLocation ))) - { - INTERNET_SendCallback(&request->hdr, request->hdr.dwContext, INTERNET_STATUS_REDIRECT, - new_url, (strlenW(new_url) + 1) * sizeof(WCHAR)); - res = HTTP_HandleRedirect(request, new_url); - if (res == ERROR_SUCCESS) - res = HTTP_HttpSendRequestW(request, NULL, 0, NULL, 0, 0, TRUE); - heap_free( new_url ); - } + heap_free(request->verb); + request->verb = heap_strdupW(szGET); } + drain_content(request); + if ((new_url = HTTP_GetRedirectURL( request, szNewLocation ))) + { + INTERNET_SendCallback(&request->hdr, request->hdr.dwContext, INTERNET_STATUS_REDIRECT, + new_url, (strlenW(new_url) + 1) * sizeof(WCHAR)); + res = HTTP_HandleRedirect(request, new_url); + if (res == ERROR_SUCCESS) + res = HTTP_HttpSendRequestW(request, NULL, 0, NULL, 0, 0, TRUE); + heap_free( new_url ); + } + } } }

1 0

Jacek Caban : wininet: Use stored status code in HTTP_HttpSendRequestW.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: 4cf3f78542db6571aa55e0cebac75bbf0c2c8e1a URL: http://source.winehq.org/git/wine.git/?a=commit;h=4cf3f78542db6571aa55e0ceb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu May 3 12:19:58 2012 +0200 wininet: Use stored status code in HTTP_HttpSendRequestW. --- dlls/wininet/http.c | 26 +++++++++++--------------- 1 files changed, 11 insertions(+), 15 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index fae4543..b04a51c 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -4797,7 +4797,6 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *request, LPCWSTR lpszHeaders, if (bEndRequest) { DWORD dwBufferSize; - DWORD dwStatusCode; INTERNET_SendCallback(&request->hdr, request->hdr.dwContext, INTERNET_STATUS_RECEIVING_RESPONSE, NULL, 0); @@ -4821,11 +4820,6 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *request, LPCWSTR lpszHeaders, HTTP_ProcessExpires(request); HTTP_ProcessLastModified(request); - dwBufferSize = sizeof(dwStatusCode); - if (HTTP_HttpQueryInfoW(request,HTTP_QUERY_FLAG_NUMBER|HTTP_QUERY_STATUS_CODE, - &dwStatusCode,&dwBufferSize,NULL) != ERROR_SUCCESS) - dwStatusCode = 0; - res = set_content_length(request); if(res != ERROR_SUCCESS) goto lend; @@ -4836,14 +4830,16 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *request, LPCWSTR lpszHeaders, { WCHAR *new_url, szNewLocation[INTERNET_MAX_URL_LENGTH]; dwBufferSize=sizeof(szNewLocation); - if ((dwStatusCode == HTTP_STATUS_REDIRECT || - dwStatusCode == HTTP_STATUS_MOVED || - dwStatusCode == HTTP_STATUS_REDIRECT_KEEP_VERB || - dwStatusCode == HTTP_STATUS_REDIRECT_METHOD) && - HTTP_HttpQueryInfoW(request,HTTP_QUERY_LOCATION,szNewLocation,&dwBufferSize,NULL) == ERROR_SUCCESS) - { + switch(request->status_code) { + case HTTP_STATUS_REDIRECT: + case HTTP_STATUS_MOVED: + case HTTP_STATUS_REDIRECT_KEEP_VERB: + case HTTP_STATUS_REDIRECT_METHOD: + if(HTTP_HttpQueryInfoW(request,HTTP_QUERY_LOCATION,szNewLocation,&dwBufferSize,NULL) != ERROR_SUCCESS) + break; + if (strcmpW(request->verb, szGET) && strcmpW(request->verb, szHEAD) && - dwStatusCode != HTTP_STATUS_REDIRECT_KEEP_VERB) + request->status_code != HTTP_STATUS_REDIRECT_KEEP_VERB) { heap_free(request->verb); request->verb = heap_strdupW(szGET); @@ -4868,7 +4864,7 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *request, LPCWSTR lpszHeaders, { WCHAR szAuthValue[2048]; dwBufferSize=2048; - if (dwStatusCode == HTTP_STATUS_DENIED) + if (request->status_code == HTTP_STATUS_DENIED) { LPHTTPHEADERW Host = HTTP_GetHeader(request, hostW); DWORD dwIndex = 0; @@ -4892,7 +4888,7 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *request, LPCWSTR lpszHeaders, request->authInfo = NULL; } } - if (dwStatusCode == HTTP_STATUS_PROXY_AUTH_REQ) + if (request->status_code == HTTP_STATUS_PROXY_AUTH_REQ) { DWORD dwIndex = 0; while (HTTP_HttpQueryInfoW(request,HTTP_QUERY_PROXY_AUTHENTICATE,szAuthValue,&dwBufferSize,&dwIndex) == ERROR_SUCCESS)

1 0

Michael Stefaniuc : strmbase: Print the debug string and not the pointer to it.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: 0f46f7538aa20e973115f2a84efa91cc9278ed20 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0f46f7538aa20e973115f2a84… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu May 3 11:20:05 2012 +0200 strmbase: Print the debug string and not the pointer to it. --- dlls/strmbase/renderer.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/strmbase/renderer.c b/dlls/strmbase/renderer.c index c2a7e83..1f950f9 100644 --- a/dlls/strmbase/renderer.c +++ b/dlls/strmbase/renderer.c @@ -419,7 +419,7 @@ HRESULT WINAPI BaseRendererImpl_FindPin(IBaseFilter * iface, LPCWSTR Id, IPin ** { BaseRenderer *This = impl_from_IBaseFilter(iface); - TRACE("(%p)->(%p,%p)\n", This, debugstr_w(Id), ppPin); + TRACE("(%p)->(%s,%p)\n", This, debugstr_w(Id), ppPin); if (!Id || !ppPin) return E_POINTER;

1 0

Jacek Caban : jscript: Bettter handling of to_number result in String. indexOf.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: baa07477d2c70debf13fce84980ac9539109ea91 URL: http://source.winehq.org/git/wine.git/?a=commit;h=baa07477d2c70debf13fce849… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu May 3 10:40:45 2012 +0200 jscript: Bettter handling of to_number result in String.indexOf. --- dlls/jscript/string.c | 10 ++++------ dlls/jscript/tests/api.js | 2 ++ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/jscript/string.c b/dlls/jscript/string.c index 9f6dc4f..d74f8cf 100644 --- a/dlls/jscript/string.c +++ b/dlls/jscript/string.c @@ -494,15 +494,13 @@ static HRESULT String_indexOf(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DI if(arg_cnt(dp) >= 2) { VARIANT ival; + double d; hres = to_integer(ctx, get_arg(dp,1), ei, &ival); if(SUCCEEDED(hres)) { - if(V_VT(&ival) == VT_I4) - pos = V_VT(&ival) > 0 ? V_I4(&ival) : 0; - else - pos = V_R8(&ival) > 0.0 ? length : 0; - if(pos > length) - pos = length; + d = num_val(&ival); + if(d > 0.0) + pos = is_int32(d) ? min((int)d, length) : length; } } diff --git a/dlls/jscript/tests/api.js b/dlls/jscript/tests/api.js index 861019f..8d8e819 100644 --- a/dlls/jscript/tests/api.js +++ b/dlls/jscript/tests/api.js @@ -475,6 +475,8 @@ tmp = "abcd".indexOf("bc",0,"test"); ok(tmp === 1, "indexOf = " + tmp); tmp = "abcd".indexOf(); ok(tmp == -1, "indexOf = " + tmp); +tmp = "abcd".indexOf("b", bigInt); +ok(tmp == -1, "indexOf = " + tmp); tmp = "abcd".lastIndexOf("bc",1); ok(tmp === 1, "lastIndexOf = " + tmp);

1 0

Jacek Caban : jscript: Better handling of to_integer in String.substr.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: e26a3018e775715f360ee951164bc9418f71d299 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e26a3018e775715f360ee9511… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu May 3 10:41:36 2012 +0200 jscript: Better handling of to_integer in String.substr. --- dlls/jscript/string.c | 27 +++++++++------------------ 1 files changed, 9 insertions(+), 18 deletions(-) diff --git a/dlls/jscript/string.c b/dlls/jscript/string.c index 3b7e700..9e2fcb2 100644 --- a/dlls/jscript/string.c +++ b/dlls/jscript/string.c @@ -1366,6 +1366,7 @@ static HRESULT String_substr(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DIS INT start=0, len; DWORD length; VARIANT v; + double d; HRESULT hres; TRACE("\n"); @@ -1381,15 +1382,9 @@ static HRESULT String_substr(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DIS return hres; } - if(V_VT(&v) == VT_I4) { - start = V_I4(&v); - if(start < 0) - start = 0; - else if(start >= length) - start = length; - }else { - start = V_R8(&v) < 0.0 ? 0 : length; - } + d = num_val(&v); + if(d >= 0) + start = is_int32(d) ? min(length, d) : length; } if(arg_cnt(dp) >= 2) { @@ -1399,15 +1394,11 @@ static HRESULT String_substr(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DIS return hres; } - if(V_VT(&v) == VT_I4) { - len = V_I4(&v); - if(len < 0) - len = 0; - else if(len > length-start) - len = length-start; - }else { - len = V_R8(&v) < 0.0 ? 0 : length-start; - } + d = num_val(&v); + if(d >= 0.0) + len = is_int32(d) ? min(length-start, d) : length-start; + else + len = 0; }else { len = length-start; }

1 0

Jacek Caban : wininet: Store status code in http_request_t and use it in set_content_length.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: 5240e40b69a28e883961a861ce82d05de59ddf44 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5240e40b69a28e883961a861c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu May 3 12:19:47 2012 +0200 wininet: Store status code in http_request_t and use it in set_content_length. --- dlls/wininet/http.c | 10 ++++++---- dlls/wininet/internet.h | 1 + 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 54db118..fae4543 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -2585,13 +2585,13 @@ static const data_stream_vtbl_t chunked_stream_vtbl = { }; /* set the request content length based on the headers */ -static DWORD set_content_length(http_request_t *request, DWORD status_code) +static DWORD set_content_length(http_request_t *request) { static const WCHAR szChunked[] = {'c','h','u','n','k','e','d',0}; WCHAR encoding[20]; DWORD size; - if(status_code == HTTP_STATUS_NO_CONTENT) { + if(request->status_code == HTTP_STATUS_NO_CONTENT) { request->contentLength = request->netconn_stream.content_length = 0; return ERROR_SUCCESS; } @@ -4826,7 +4826,7 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *request, LPCWSTR lpszHeaders, &dwStatusCode,&dwBufferSize,NULL) != ERROR_SUCCESS) dwStatusCode = 0; - res = set_content_length(request, dwStatusCode); + res = set_content_length(request); if(res != ERROR_SUCCESS) goto lend; if(!request->contentLength) @@ -5000,7 +5000,7 @@ static DWORD HTTP_HttpEndRequestW(http_request_t *request, DWORD dwFlags, DWORD_ &dwCode,&dwCodeLength,NULL) != ERROR_SUCCESS) dwCode = 0; - if ((res = set_content_length( request, dwCode )) == ERROR_SUCCESS) { + if ((res = set_content_length(request)) == ERROR_SUCCESS) { if(!request->contentLength) http_release_netconn(request, TRUE); } @@ -5692,6 +5692,8 @@ static INT HTTP_GetResponseHeaders(http_request_t *request, BOOL clear) goto lend; *status_text++=0; + request->status_code = atoiW(status_code); + TRACE("version [%s] status code [%s] status text [%s]\n", debugstr_w(buffer), debugstr_w(status_code), debugstr_w(status_text) ); diff --git a/dlls/wininet/internet.h b/dlls/wininet/internet.h index 6cf1f38..b515a71 100644 --- a/dlls/wininet/internet.h +++ b/dlls/wininet/internet.h @@ -312,6 +312,7 @@ typedef struct DWORD send_timeout; DWORD receive_timeout; LPWSTR version; + DWORD status_code; LPWSTR statusText; DWORD bytesToWrite; DWORD bytesWritten;

1 0

Jacek Caban : jscript: Better handling of to_integer result in String. charAt.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: 0143201eac5257b806bb72f621ad73668f88fd68 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0143201eac5257b806bb72f62… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu May 3 10:40:13 2012 +0200 jscript: Better handling of to_integer result in String.charAt. --- dlls/jscript/string.c | 10 +++------- dlls/jscript/tests/api.js | 2 ++ 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/dlls/jscript/string.c b/dlls/jscript/string.c index b629d57..bf9312d 100644 --- a/dlls/jscript/string.c +++ b/dlls/jscript/string.c @@ -303,19 +303,15 @@ static HRESULT String_charAt(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DIS if(arg_cnt(dp)) { VARIANT num; + double d; hres = to_integer(ctx, get_arg(dp, 0), ei, &num); if(FAILED(hres)) { SysFreeString(val_str); return hres; } - - if(V_VT(&num) == VT_I4) { - pos = V_I4(&num); - }else { - WARN("pos = %lf\n", V_R8(&num)); - pos = -1; - } + d = num_val(&num); + pos = is_int32(d) ? d : -1; } if(!retv) { diff --git a/dlls/jscript/tests/api.js b/dlls/jscript/tests/api.js index c1ae808..f92cd48 100644 --- a/dlls/jscript/tests/api.js +++ b/dlls/jscript/tests/api.js @@ -308,6 +308,8 @@ tmp = "abc".charAt(0,2); ok(tmp === "a", "'abc',charAt(0.2) = " + tmp); tmp = "abc".charAt(NaN); ok(tmp === "a", "'abc',charAt(NaN) = " + tmp); +tmp = "abc".charAt(bigInt); +ok(tmp === "", "'abc',charAt(bigInt) = " + tmp); tmp = "abc".charCodeAt(0); ok(tmp === 0x61, "'abc'.charCodeAt(0) = " + tmp);

1 0

Jacek Caban : jscript: Better handling of to_integer result in String. charCodeAt.
by Alexandre Julliard May 3, 2012

May 3, 2012

Module: wine Branch: master Commit: 621180e81c260af68a28fcd793a1d4352eccac1c URL: http://source.winehq.org/git/wine.git/?a=commit;h=621180e81c260af68a28fcd79… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu May 3 10:40:27 2012 +0200 jscript: Better handling of to_integer result in String.charCodeAt. --- dlls/jscript/string.c | 10 +++++++--- dlls/jscript/tests/api.js | 2 ++ 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/dlls/jscript/string.c b/dlls/jscript/string.c index bf9312d..9f6dc4f 100644 --- a/dlls/jscript/string.c +++ b/dlls/jscript/string.c @@ -350,6 +350,7 @@ static HRESULT String_charCodeAt(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, if(arg_cnt(dp) > 0) { VARIANT v; + double d; hres = to_integer(ctx, get_arg(dp, 0), ei, &v); if(FAILED(hres)) { @@ -357,13 +358,16 @@ static HRESULT String_charCodeAt(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, return hres; } - if(V_VT(&v) != VT_I4 || V_I4(&v) < 0 || V_I4(&v) >= length) { - if(retv) num_set_nan(&v); + d = num_val(&v); + + if(!is_int32(d) || d < 0 || d >= length) { SysFreeString(val_str); + if(retv) + num_set_nan(retv); return S_OK; } - idx = V_I4(&v); + idx = d; } if(retv) { diff --git a/dlls/jscript/tests/api.js b/dlls/jscript/tests/api.js index f92cd48..861019f 100644 --- a/dlls/jscript/tests/api.js +++ b/dlls/jscript/tests/api.js @@ -329,6 +329,8 @@ tmp = "\052".charCodeAt(0); ok(tmp === 0x2A, "'\052'.charCodeAt(0) = " + tmp); tmp = "\xa2".charCodeAt(0); ok(tmp === 0xA2, "'\xa2'.charCodeAt(0) = " + tmp); +tmp = "abc".charCodeAt(bigInt); +ok(isNaN(tmp), "'abc'.charCodeAt(bigInt) = " + tmp); tmp = "abcd".substring(1,3); ok(tmp === "bc", "'abcd'.substring(1,3) = " + tmp);

1 0