Re: Testing edge cases and undocumented behavior

2011-08-25 Thread Vincent Povirk
A test that passes on Windows and fails on Wine is not sufficient to motivate a change to Wine. I think we normally consider the following factors: * Is the change correct? If not, we should consider leaving Wine alone, but I can imagine there being other compelling reasons, in rare circumstances,

Testing edge cases and undocumented behavior

2011-08-25 Thread Scott Ritchie
There was a bit of a philosophical discussion on #winehackers about the merits of creating tests for functions that might be testing undefined or unimportant behavior. Windows behaves one way, we behave another, the tests measure this delta, but it's unknown if this will actually improve a real wo

FYI - build-dep can no longer be done for git

2011-08-25 Thread Susan Cragin
There have been a couple of replacement packages. $ sudo apt-get build-dep wine1.3 The following packages have unmet dependencies: ia32-libs : Depends: lib32v4l-0 but it is not going to be installed lib32v4l-dev : Depends: libv4l-0 (= 0.8.3-2) but 0.8.5-3ubuntu1 is to be installed

Re: Tasks request

2011-08-25 Thread Michael Mc Donnell
On Thu, Aug 25, 2011 at 3:31 AM, Nowres Rafid wrote: > I want to help the wine project, any one can tell me what tasks I can start > with.Thanks Are there any parts of win32 that you have ever used? Those might be good to look into if you want to code. A quick grep on the code for FIXME or stub c

Re: (resend)usp10/test: test ScriptXtoX on an RTL set with differring cChars and cGlyphs

2011-08-25 Thread Aric Stewart
On 8/25/11 9:50 AM, Nicolas Le Cam wrote: 2011/8/25 Aric Stewart: Ah i see what you are seeing. I will see if that helps. However it is still strange that the tests are all working for me and when i submit tests but not when it is submitted as a patch. -aric On 8/25/11 7:47 AM, Hans Leidekke

Re: (resend)usp10/test: test ScriptXtoX on an RTL set with differring cChars and cGlyphs

2011-08-25 Thread Nicolas Le Cam
2011/8/25 Aric Stewart : > Ah i see what you are seeing. > > I will see if that helps.  However it is still strange that the tests are > all working for me and when i submit tests but not when it is submitted as a > patch. > > -aric > > On 8/25/11 7:47 AM, Hans Leidekker wrote: >> >> On Thu, 2011-0

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

2011-08-25 Thread sa.anderson009
The error message you have got because you are trying to run a corrupted or missing d3dx9.dll file. Windows displays the above message when system unable to run d3dx9.dll file. To fix the issue, you can restore it. Here is the link to download d3dx9.dll file. http://www.d3dx9.net After download t

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed. (try 3)

2011-08-25 Thread 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=13713 Your paranoid android

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed. (try 2)

2011-08-25 Thread 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=13711 Your paranoid android

Re: (resend)usp10/test: test ScriptXtoX on an RTL set with differring cChars and cGlyphs

2011-08-25 Thread Aric Stewart
Ah i see what you are seeing. I will see if that helps. However it is still strange that the tests are all working for me and when i submit tests but not when it is submitted as a patch. -aric On 8/25/11 7:47 AM, Hans Leidekker wrote: On Thu, 2011-08-25 at 07:38 -0500, Aric Stewart wrote:

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed.

2011-08-25 Thread Jacek Caban
On 08/25/11 14:40, Octavian Voicu wrote: On Thu, Aug 25, 2011 at 3:31 PM, Jacek Caban wrote: As long as there is a failure (like in your patch), I'm fine with return. Just let's not pretend it's somehow valid run with skip() call. Also testing hres is cleaner than unk, IMO. Yeah, the skip will

Re: (resend)usp10/test: test ScriptXtoX on an RTL set with differring cChars and cGlyphs

2011-08-25 Thread Hans Leidekker
On Thu, 2011-08-25 at 07:38 -0500, Aric Stewart wrote: > Yes, it depends on if it is a RTL or LTR string. That is correct. What I meant is that it does not always match the right hand operand in the test condition.

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed.

2011-08-25 Thread Octavian Voicu
On Thu, Aug 25, 2011 at 3:31 PM, Jacek Caban wrote: > As long as there is a failure (like in your patch), I'm fine with return. > Just let's not pretend it's somehow valid run with skip() call. Also testing > hres is cleaner than unk, IMO. Yeah, the skip will only come after a failure because unk

Re: (resend)usp10/test: test ScriptXtoX on an RTL set with differring cChars and cGlyphs

2011-08-25 Thread Aric Stewart
Yes, it depends on if it is a RTL or LTR string. That is correct. -aric On 8/25/11 7:30 AM, Hans Leidekker wrote: On Thu, 2011-08-25 at 07:15 -0500, Aric Stewart wrote: "should return piX=0 not 0"? seems like 0 == 0 to me. They are both int values... That test doesn't look right: win

Re: [PATCH 2/8] msi: make WHEREVIEW able to deal directly with multiple tables

2011-08-25 Thread Alexandre Julliard
Bernhard Loos writes: > +/* below is the query interface to a table */ > + > typedef struct tagMSIROWENTRY > { > -UINT value; > +UINT values[0]; Zero-size arrays are not portable. -- Alexandre Julliard julli...@winehq.org

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed.

2011-08-25 Thread Jacek Caban
On 08/25/11 14:09, Octavian Voicu wrote: On Thu, Aug 25, 2011 at 2:59 PM, Jacek Caban wrote: Those are failures on Wine, so tests are fine, it's Wine that needs fixing. Yes, the first test usually fails on Wine. Nonetheless, if CoCreateInstance fails for whatever reason (like it happens on tha

Re: (resend)usp10/test: test ScriptXtoX on an RTL set with differring cChars and cGlyphs

2011-08-25 Thread Hans Leidekker
On Thu, 2011-08-25 at 07:15 -0500, Aric Stewart wrote: > "should return piX=0 not 0"? seems like 0 == 0 to me. They are both > int values... That test doesn't look right: winetest_ok(piX == offsets[iCP+1], "ScriptCPtoX trailing: iCP=%d should return piX=%d not %d\n", iCP, offsets[iCP

Re: Tasks request

2011-08-25 Thread Per Johansson
25 aug 2011 kl. 14:12 skrev Frédéric Delanoy: > On Thu, Aug 25, 2011 at 03:31, Nowres Rafid wrote: >> Hello, >> >> I want to help the wine project, any one can tell me what tasks I can start >> with.Thanks >> >> Nowres rafid > > There are many things that can be done to help Wine: bug hunting

Re: (resend)usp10/test: test ScriptXtoX on an RTL set with differring cChars and cGlyphs

2011-08-25 Thread Aric Stewart
I don't get this. I do the test locally, they all pass. I submit it directly to testbot and they all pass. I submit the patch and i get these nonsense failures. "should return piX=0 not 0"? seems like 0 == 0 to me. They are both int values... Is this just testbot being strange? -aric On 8

Re: Tasks request

2011-08-25 Thread Frédéric Delanoy
On Thu, Aug 25, 2011 at 03:31, Nowres Rafid wrote: > Hello, > > I want to help the wine project, any one can tell me what tasks I can start > with.Thanks > > Nowres rafid There are many things that can be done to help Wine: bug hunting, fixes, triage (http://bugs.winehq.org/), test applications f

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed.

2011-08-25 Thread Octavian Voicu
On Thu, Aug 25, 2011 at 2:59 PM, Jacek Caban wrote: > Those are failures on Wine, so tests are fine, it's Wine that needs fixing. Yes, the first test usually fails on Wine. Nonetheless, if CoCreateInstance fails for whatever reason (like it happens on that NT4 machine, for example), running the r

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed.

2011-08-25 Thread Jacek Caban
On 08/25/11 13:55, Octavian Voicu wrote: On Thu, Aug 25, 2011 at 2:38 PM, Jacek Caban wrote: Where do you get such a broken installation from? I don't see such failures on test.winehq.org. Hi Jacek, It's not only me, see [1]. It fails every time on WOW64 giving something like this [2]. It's a

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed.

2011-08-25 Thread Octavian Voicu
On Thu, Aug 25, 2011 at 2:38 PM, Jacek Caban wrote: > Where do you get such a broken installation from? I don't see such failures > on test.winehq.org. Hi Jacek, It's not only me, see [1]. It fails every time on WOW64 giving something like this [2]. It's a different crash from the win64 one (whi

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed.

2011-08-25 Thread Jacek Caban
Hi Octavian, On 08/25/11 13:26, Octavian Voicu wrote: Prevents a crash on WOW64, where CLSID_InternetExplorer\LocalServer32 points to nonexistent iexplore.exe in Program Files (x86) folder. Where do you get such a broken installation from? I don't see such failures on test.winehq.org. Jacek

Re: ieframe/tests: Skip rest of tests if CoCreateInstance failed.

2011-08-25 Thread 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=13705 Your paranoid android

Re: [1/2] msxml3/mxwriter: Fix parameter validation for startElement() for MXXMLWriter60

2011-08-25 Thread 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=13699 Your paranoid android

Re: [2/2] msxml3/mxwriter: Fix parameter validation for endElement()

2011-08-25 Thread 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=13700 Your paranoid android

avifil32: Remove the stream format string from the resources?

2011-08-25 Thread Francois Gouget
In the wvifil32 resources we have this string: IDS_AVISTREAMFORMAT "%s %s #%d" I see no way in which this can be meaningfully translated and many ways in which translators can mess it up. It was introduced in this commit: commit 4c04e1fd8773fd7b8884a8c5fead4609869bf4ab Author: Mi

winefile: Remove the title format from the resources?

2011-08-25 Thread Francois Gouget
In the resources, and hence in the PO files we have this string: IDS_TITLEFMT"%s - %s" I see no way in which this can be meaningfully translated and many ways in which translators can mess it up. Yet it was moved there by this commit: commit 1573382dd6baa48bb825c26faef96a

winecoreaudio: Fix AudioRenderClient Get/ReleaseBuffer protocol.

2011-08-25 Thread Joerg-Cyril . Hoehle
Hi, what happened to this patch? There are already 2 bug reports about crashes in ReleaseBuffer that may be cured by this, even though the underlying problem is #28093 (GetCurrentPadding would not return "buffer full" on native, so the SW would not attempt to Release 0 bytes which causes winec

Re: [PATCH 3/6] server: sharing 0 is not allowed for named pipes

2011-08-25 Thread Nikolay Sivov
On 8/25/2011 08:49, Bernhard Loos wrote: --- dlls/ntdll/tests/pipe.c |2 +- server/named_pipe.c |6 ++ 2 files changed, 7 insertions(+), 1 deletions(-) @@ -964,6 +964,12 @@ DECL_HANDLER(create_named_pipe) struct unicode_str name; struct directory *root = NULL; +

Re: [1/7] cmd: Fix FOR tab handling (try 3)

2011-08-25 Thread Frédéric Delanoy
On Thu, Aug 25, 2011 at 01:06, Octavian Voicu wrote: > 2011/8/25 Frédéric Delanoy : >> +  const WCHAR inW[] = {'i','n'}; >> +  const WCHAR doW[] = {'d','o'}; > ... > > Hello, > > After these patches get in, it might be a good idea to make all these > strings static. Even if they are const, the com