July 2011 - Wine-Devel - list.winehq.org

Re: richedit: Implement WB_DELIMITER and remove todo_wine's in tests.

by Paul Vriens

On 07/26/2011 03:36 AM, James McKenzie wrote: > if (wbuf[0] == 0x20 || wbuf[0] == 0xf020) > - todo_wine > - ok(result == delimiter_tests[i].isdelimiter, > - "wanted ISDELIMITER_W(0x%x) %d, got %d\n", > - delimiter_tests[i].c, delimiter_tests[i].isdelimiter,result); > + ok(result == delimiter_tests[i].isdelimiter, > + "wanted ISDELIMITER_W(0x%x) %d, got %d\n", > + delimiter_tests[i].c, delimiter_tests[i].isdelimiter,result); > else > ok(result == delimiter_tests[i].isdelimiter, > "wanted ISDELIMITER_W(0x%x) %d, got %d\n", > @@ -7122,10 +7121,9 @@ static void test_EM_FINDWORDBREAK_A(void) > SendMessageW(hwndRichEdit, WM_SETTEXT, 0, (LPARAM)buf); > result = SendMessage(hwndRichEdit, EM_FINDWORDBREAK, WB_ISDELIMITER, 0); > if (buf[0] == 0x20) > - todo_wine > - ok(result == delimiter_tests[i].isdelimiter, > - "wanted ISDELIMITER_A(0x%x) %d, got %d\n", > - delimiter_tests[i].c, delimiter_tests[i].isdelimiter,result); > + ok(result == delimiter_tests[i].isdelimiter, > + "wanted ISDELIMITER_A(0x%x) %d, got %d\n", > + delimiter_tests[i].c, delimiter_tests[i].isdelimiter,result); > else Hi James, I think that the if/else was introduced to accommodate the todo_wine(). I guess you can remove more lines as the 'if' now contains the same code as the 'else'. -- Cheers, Paul.

July 26, 2011

Re: d3d8/tests: make sure to use return values (LLVM/Clang)

by Henri Verbeet

On 26 July 2011 10:57, Jacek Caban <jacek(a)codeweavers.com> wrote: > It's better to test for the exact value in tests like hr == S_OK. It > makes tests stricter. I don't think that making tests more strict than necessary is a good principle. (Not that it matters a lot here anyway, for most of these D3D_OK is the only possible sucessful return value.)

July 26, 2011

Re: GSoC-2011: Implement Missing Mesh Functions in Wine’s D3DX9

by Michael Mc Donnell

On Mon, Jul 25, 2011 at 4:04 PM, Stefan Dösinger <stefandoesinger(a)gmx.at> wrote: > A few more comments, with one day delay: Thanks that was quick! >> +static float max_abs_diff_vec2(D3DXVECTOR2 *v1, D3DXVECTOR2 *v2) >> +{ >> + float diff_x = fabs(v1->x - v2->x); >> + float diff_y = fabs(v1->y - v2->y); >> + >> + return fmax(diff_x, diff_y); >> +} > fabs and fmax return doubles, probably use the float functions fabsf and fmaxf. gcc doesn't bother about the difference, but msvc generates a precision loss warning. Ok. I've changed all occurrences of fmax to fmaxf and fabs to fabsf. > Duplicating the code for 16 and 32 bit indices looks a bit ugly. Maybe you can use inline functions to read and write values from the proper buffer? The other possibility, which the ddraw blitting code uses is to write a sort of template as a macro and then generate both versions, but I don't like that idea. Yes I agree it's ugly. I've added the functions read_ib and write_ib to handle 32 bit indices. Does that look like a good solution to you too? > Wrt the D3DDECLTYPE epsilon, it may be more efficient to scale the epsilon and calculate the diff and comparison as UBYTEs rather than floats. You'll need the diff functions for the other types like *BYTE*, *SHORT* anyway. Also tests would be helpful here, especially how the epsilon is treated in normalized values like D3DCOLOR and UBYTE4N and how it is treated in non-normalized values like UBYTE4. Ok I'll look into that and get back to you later. > Wrt the usage/index fields, what happens when a combination that isn't supported by the fixed function pipeline is used, e.g. NORMAL3 or POSITION5? Those are valid in a vertex declaration and can be used with shaders. It works because the code does not check the UsageIndex field, except for differentiating between DIFFUSE and SPECULAR. I've added test 13 that shows this (tested on Windows 7 and Linux). Btw. looking at the code again I found a small bug introduced during a cleanup. The default in the switch case in get_component_epsilon is to write an ERR. It should also catch and ignore some of the usages that it is not possible to specify an epsilon for. I've put that back. > In the test: >> + int vertex_size_normal = sizeof(struct vertex_normal); >> + int vertex_size_blendweight = sizeof(struct vertex_blendweight); >> ... > You could make those unsigned. Check. > I'm not quite done reading the test yet, I'll send another mail if I find more. Thanks! I appreciate the feedback. > > Think about caching pointreps and adjacency? > Am 24.07.2011 um 11:57 schrieb Michael Mc Donnell: > >> On Sun, Jul 24, 2011 at 12:18 AM, Stefan Dösinger >> <stefandoesinger(a)gmx.at> wrote: >>> I just started reading this, I'll write more once I am done. Just a quick >>> question: What do you mean with "built in function" in the comment above >>> color_to_vector? >> >> I wanted to ask if there was a DirectX function or macro that can >> convert a D3DCOLOR, which is four bytes in the range 0-255, to four >> floats in the range 0-1 or a D3DXVECTOR4? > There's none I know of. I vaguely remember that there was a macro to construct D3DCOLORs, but that's not too helpful here. Ok. I've removed the comment. >> Reading this again I think color_to_vector should probably accept a >> D3DCOLOR instead of "BYTE color[4]"? > Yes, it would be cleaner. Also keep UBYTE4N in mind, which is pretty much the same as D3DCOLOR for your purposes. It just has a different component ordering(bgra in D3DCOLOR vs rgba in UBYTE4N if I'm not mistaken) when you're using it for drawing. As far as I understand WeldVertices it doesn't compare two attributes of different types, bit if you do you have to watch out there. I'll keep it as it is for now and have a look into generalizing it to UBYTE4N.

July 26, 2011

Re: d3d8/tests: make sure to use return values (LLVM/Clang)

by Jacek Caban

Hi Austin, ok(SUCCEEDED(hr), "GetRenderTarget failed, hr %#x.\n", hr); hr = IDirect3DSurface8_GetDesc(surface, &surface_desc); + ok(SUCCEEDED(hr), "GetDesc failed, hr %#x.\n", hr); It's better to test for the exact value in tests like hr == S_OK. It makes tests stricter. I know there are already tests checking SUCCEEDED(), but while you are at this, let's get it right. Also, patches should go to wine-patches ;) Jacek

July 26, 2011

Re: [8/9] winhttp: Implement IWinHttpRequest::SetRequestHeader.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=12827 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

July 26, 2011

Re: [7/9] winhttp: Implement IWinHttpRequest::GetResponseHeader.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=12826 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

July 26, 2011

Re: [6/9] winhttp: Add some parameter checks.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=12824 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

July 26, 2011

Re: [1/9] winhttp: Implement IWinHttpRequest::SetProxy.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=12823 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

July 26, 2011

Re: msvcrt/tests: remove unused assignments (LLVM/Clang)

by Reece Dunn

2011/7/26 Frédéric Delanoy <frederic.delanoy(a)gmail.com>: > On Tue, Jul 26, 2011 at 03:48, Austin English <austinenglish(a)gmail.com> wrote: >> -- >> -Austin > > Already sent similar patches. Not sure AJ likes those. These should test what is returned by fgetc, not ignore its return value. E.g.: - c= fgetc(fp); + ok((c = fgetc(fp)) == '\r', "fgetc after rewind failed, expected '\\r', got '%c'\n", c); - Reece

July 26, 2011

Re: cmd: mkdir should complain if directory already exists (try 2)

by Frédéric Delanoy

On Tue, Jul 26, 2011 at 07:39, Dan Kegel <dank(a)kegel.com> wrote: > Changes since last try: > - fix remaining todo_wine's > - add one more testcase for invalid paths (one that doesn't trigger > help message) > - simplified create_full_path > - avoid generating error message in race case where somebody else > creates an intermediate directory first Maybe a bit nitpickings, but echo mkdir foo slash star errorlevel %ErrorLevel% could be more readable with echo mkdir foo/* errorlevel %ErrorLevel% / and * are not to be escaped, so don't mind expanding them "litteraly" Frédéric

July 26, 2011