October 2010 - Wine-Devel - list.winehq.org

Re: [PATCH] mscms: Handle bitmap formats BM_xRGBQUADS in from_bmformat( BMFORMAT format ) which is used by TranslateBitmapBits

by Tomasz Michno

Hi, > You are copying quite a bit a infrastructure from profile.c, so it would > be better to add the tests to that file. We can always split them up > later. > Maybe you are right, but if there were more functions from transform.c in my opinion it would be better to have tests/transform.c. Maybe we should move common part to another file and include it in profile.c and transform.c? I sent few minutes ago new version with removed all // comments and new whitespaces and fixed pCreateMultiProfileTransform(&hProfile, 2, &intent, 1,... line.

Oct. 21, 2010

Re: [PATCH] mscms: Handle bitmap formats BM_xRGBQUADS in from_bmformat( BMFORMAT format ) which is used by TranslateBitmapBits

by Hans Leidekker

On Wed, 2010-10-20 at 12:45 +0200, Tomasz Michno wrote: > C_SRCS = \ > - profile.c > + profile.c \ > + transform.c You are copying quite a bit a infrastructure from profile.c, so it would be better to add the tests to that file. We can always split them up later. > +#include "mscms_priv.h" This is not needed. > +void test_TranslateBitmapBits(void){ > + BOOL ret=TRUE; > + COLOR bitmapBits[bitmapSize]; > + COLOR bitmapBitsResult[bitmapSize]; > + #ifdef HAVE_LCMS You can't use that define in tests. > + HPROFILE hProfile; > ... > + transform = pCreateMultiProfileTransform(&hProfile, 2, &intent, 1, > + // test already handled BMFORMATs: This won't work, you say you have 2 profiles but pass only one. Don't use // for comments.

Oct. 21, 2010

Re: ws2_32: Add support and tests for WSARecvMsg and IP_PKTINFO.

by Erich Hoover

On Mon, Oct 18, 2010 at 8:58 PM, Mike Kaplinskiy <mike.kaplinskiy(a)gmail.com> wrote: > On Sun, Oct 17, 2010 at 5:51 PM, Erich Hoover <ehoover(a)mines.edu> wrote: >> ... >> That's why in the original version I had those parameters split out; >> however, I realized that since the length parameter gets modified by >> WSARecvMsg that it is impossible for the WSAMSG structure (or any of >> its parameters) to disappear before WSARecvMsg completes. I have >> re-written the tests (see patch 4 at the end) to show that this >> structure gets modified even when an overlapped request is used and >> results in modification of the structure at a later time, the results >> from the test bot can be found here: >> https://testbot.winehq.org/JobDetails.pl?Key=6250 >> > Fair enough. Would you mind adding a test to see if the flags are > changed as well (and see if the flags returned by > WSAGetOverlappedResult are of that sort as well)? If it doesn't you > can drop lpFlags and just copy it from flags. Otherwise you may want > to rename it to be less windows-y (like local_flags and flags). > I'm not entirely following you here, but if I understand you correctly then the lpFlags is appropriate. The flags in the WSAMSG structure get modified while the WSAGetOverlappedResult flags do not (which works "as is" since the overlapped version of the flags is not a pointer). I've changed the patch around a little so that the pointer version is only used when appropriate (and added tests for this functionality): [3/4] http://www.compholio.com/wine-kane/patches/2010-10-20/0003-ws2_32-Add-suppo… [4/4] http://www.compholio.com/wine-kane/patches/2010-10-20/0004-ws2_32-tests-Add… Erich Hoover ehoover(a)mines.edu

Oct. 20, 2010

Re: current state of full screen on MacOS?

by James McKenzie

On 7/10/09 6:26 AM, Joerg-Cyril.Hoehle(a)t-systems.com wrote: > Hi, > > I'd like to know whether a MacOSX user can expect full screen modes to work, as > I've heard only rumors and too little facts. My experience is this: > Joerg: I have news to announce on this very old thread, but with XQuartz 2.5.3, Quake II runs in a on-screen resizing window and the installer ran in full-screen mode. James McKenzie

Oct. 20, 2010

Re: About: d3dx9: Store transform matrix per-sprite.

by Austin English

On Wed, Oct 20, 2010 at 2:31 PM, Joris Huizer <joris_huizer(a)yahoo.com> wrote: > > --- On Tue, 10/19/10, Austin English <austinenglish(a)gmail.com> wrote: > >> While these emails are very helpful, can you please do a >> reply to the >> original thread? It makes it easier for those of us using >> clients with >> threaded mode to follow. >> >> Thanks! >> >> -- >> -Austin >> > > I can understand that's easier, less messy - I didn't do so up until now as it is quite a number of emails per day (especially if tracking both wine-devel and wine-patch) -- and my contributions/'contributions' are rare; > > Is there a way to do this other than registering to get the emails daily sent to me? I usually browse through the updates via the web (the mail archive and the patch watcher at http://source.winehq.org/patches/) and as far as I know there is no way of replying on either ? > > I think I will register to receive emails from the wine-devel and wine-patch mailing lists unless there's an other way to reply to thread. Registering is the only way I know of. You can setup a filter in your mail client to ignore them / put them in their own folder so they don't flood your inbox. -- -Austin

Oct. 20, 2010

Re: Translation of the AppDB

by Yaron Shahrabani

On Wed, Oct 20, 2010 at 3:57 PM, James Mckenzie <jjmckenzie51(a)earthlink.net>wrote: > Yaron Shahrabani <sh.yaron(a)gmail.com> wrote: > > > >On Wed, Oct 20, 2010 at 6:48 AM, James McKenzie > ><jjmckenzie51(a)earthlink.net>wrote: > > > [snip] > >So you mean you have got to get some input and there's no other way of > >getting input from the users without typing... > >Do you feel like you get enough feedback or that automating parts of the > >reporting system would make it more widely used. > >I'm maintaining lots of translations in the open source world, I never > feel > >like I get enough feedback and I truly think that if the input was at the > >user's fingertips it would help a lot, just my way of looking at it... > > > Correct. There is quite a wide variation of what can and cannot work that > having a bunch of selections would be tedious at best, confusing at worst. > >> > >> There are many 'drop down' menus that it would be nice to input Hebrew > and > >> have English populate them. Would this be a good idea? > >> > >The drop down boxes would be great, the simplest way of interaction is > what > >needed, no ways of typing Hebrew... > > > Again, the existing drop downs could be translated into Hebrew and then > used to select the English equivilent. However, the text boxes will have to > remain due to wide variations in 'What works' and 'What does not work'. Too > many selections can be confusing and makes the web site appear to be slow (I > work with a web application that has 35,000+ selections, it takes about two > minutes for the menu to load on a high speed connection and not all Wine > users have that luxury.) > > >(I was thinking of a small Javascript code that will check for foreign > >scripts and if the script is different than the Latin script, a Red > message > >will appear on top of the box and after clicking the button a message will > >pop up saying that this message contains non-universal characters and ask > >the user if he is sure about sending this report). > > > I would just block it. Again, English, for now, is the world's universal > language. If someone does not know English, there are sites like Babblefish > that cover just about all languages, including (but not limited to) Hebrew, > Farsi, Persian, Arabic, Simplified and Traditional Chinese. Users can avail > themselves of them and copy/paste into the boxes. I can understand quite a > bit of 'translated' English. > So what you are saying is that even if we want to translate the boxes there are many boxes to translate... I'm cool with that, its just an idea :). I want to see what will the Hebrew website bring along, so far I can't measure any changes in interest but this kind of things takes time so I won't jump into conclusions... Kind regards, > > James McKenzie > > >

Oct. 20, 2010

Re: [1/2] msvcrt: Implement _mbsnbcat_s.

by Piotr Caban

Hi, On 10/20/10 08:49, Andrew Nguyen wrote: + * a lead byte and move the pointer back by one for later overwrite. */ + if (get_locale()->locinfo->mb_cur_max > 1 && MSVCRT_isleadbyte(*(ptr - 1))) + size++, ptr--; You may access dst-1. + ok(errno == EINVAL, "Expected errno to be EINVAL, got %d\n", ret); Printing ret here may be misleading. Cheers, Piotr

Oct. 20, 2010

Re: About: d3dx9: Store transform matrix per-sprite.

by Joris Huizer

--- On Tue, 10/19/10, Austin English <austinenglish(a)gmail.com> wrote: > While these emails are very helpful, can you please do a > reply to the > original thread? It makes it easier for those of us using > clients with > threaded mode to follow. > > Thanks! > > -- > -Austin > I can understand that's easier, less messy - I didn't do so up until now as it is quite a number of emails per day (especially if tracking both wine-devel and wine-patch) -- and my contributions/'contributions' are rare; Is there a way to do this other than registering to get the emails daily sent to me? I usually browse through the updates via the web (the mail archive and the patch watcher at http://source.winehq.org/patches/) and as far as I know there is no way of replying on either ? I think I will register to receive emails from the wine-devel and wine-patch mailing lists unless there's an other way to reply to thread. Best regards, Joris

Oct. 20, 2010

Re: [3/4]strmbase: implement OLE registration in AMovieDllRegisterServer2

by Alexandre Julliard

Aric Stewart <aric(a)codeweavers.com> writes: > HRESULT WINAPI AMovieDllRegisterServer2(BOOL bRegister) > { > HRESULT hr; > int i; > IFilterMapper2 *pIFM2 = NULL; > + HMODULE psapi; > + fnGetModuleFileNameExW pFunc; > + WCHAR szFileName[MAX_PATH]; > + > + psapi = LoadLibraryA("psapi.dll"); > + if (psapi) > + { > + pFunc = (fnGetModuleFileNameExW)GetProcAddress(psapi, "GetModuleFileNameExW"); > + if (pFunc) > + { > + if (!pFunc(GetCurrentProcess(), g_hInst, szFileName, MAX_PATH)) > + { > + ERR("Failed to get module file name for registration\n"); > + return E_FAIL; > + } > + } > + else > + { > + ERR("Failed to get get function GetModuleFileNameExW\n"); > + return E_FAIL; > + } > + } > + else > + { > + ERR("Failed to load psapi.dll\n"); > + return E_FAIL; > + } > + FreeLibrary(psapi); There's no reason to use psapi for this. -- Alexandre Julliard julliard(a)winehq.org

Oct. 20, 2010

Re: oleaut32/tests: Better reflect the naming scheme

by Alexandre Julliard

André Hentschel <webmaster(a)dawncrow.de> writes: > @@ -10,7 +10,7 @@ C_SRCS = \ > typelib.c \ > usrmarshal.c \ > varformat.c \ > - vartest.c \ > + variant.c \ > vartype.c Renaming tests should be avoided, it makes it hard to follow the history on test.winehq.org. -- Alexandre Julliard julliard(a)winehq.org

Oct. 20, 2010

Re: Translation of the AppDB

by James Mckenzie

Yaron Shahrabani <sh.yaron(a)gmail.com> wrote: > >On Wed, Oct 20, 2010 at 6:48 AM, James McKenzie ><jjmckenzie51(a)earthlink.net>wrote: > [snip] >So you mean you have got to get some input and there's no other way of >getting input from the users without typing... >Do you feel like you get enough feedback or that automating parts of the >reporting system would make it more widely used. >I'm maintaining lots of translations in the open source world, I never feel >like I get enough feedback and I truly think that if the input was at the >user's fingertips it would help a lot, just my way of looking at it... > Correct. There is quite a wide variation of what can and cannot work that having a bunch of selections would be tedious at best, confusing at worst. >> >> There are many 'drop down' menus that it would be nice to input Hebrew and >> have English populate them. Would this be a good idea? >> >The drop down boxes would be great, the simplest way of interaction is what >needed, no ways of typing Hebrew... > Again, the existing drop downs could be translated into Hebrew and then used to select the English equivilent. However, the text boxes will have to remain due to wide variations in 'What works' and 'What does not work'. Too many selections can be confusing and makes the web site appear to be slow (I work with a web application that has 35,000+ selections, it takes about two minutes for the menu to load on a high speed connection and not all Wine users have that luxury.) >(I was thinking of a small Javascript code that will check for foreign >scripts and if the script is different than the Latin script, a Red message >will appear on top of the box and after clicking the button a message will >pop up saying that this message contains non-universal characters and ask >the user if he is sure about sending this report). > I would just block it. Again, English, for now, is the world's universal language. If someone does not know English, there are sites like Babblefish that cover just about all languages, including (but not limited to) Hebrew, Farsi, Persian, Arabic, Simplified and Traditional Chinese. Users can avail themselves of them and copy/paste into the boxes. I can understand quite a bit of 'translated' English. James McKenzie

Oct. 20, 2010

Re: [1/7] msxml3/tests: internal schema doc storage tests (resend)

by Alexandre Julliard

Adam Martinson <amartinson(a)codeweavers.com> writes: > +#define _create(cls) cls, #cls > + > +#define _obj(cls, ...) &_create(cls ## __VA_ARGS__) You can't do that, varargs macros are not portable. -- Alexandre Julliard julliard(a)winehq.org

Oct. 20, 2010

Re: Translation of the AppDB

by Yaron Shahrabani

On Wed, Oct 20, 2010 at 6:48 AM, James McKenzie <jjmckenzie51(a)earthlink.net>wrote: > On 10/19/10 10:20 AM, Yaron Shahrabani wrote: > > On Tue, Oct 19, 2010 at 5:16 PM, Rosanne DiMesio <dimesio(a)earthlink.net>wrote: > >> On Tue, 19 Oct 2010 12:12:33 +0200 >> Yaron Shahrabani <sh.yaron(a)gmail.com> wrote: >> >> > I see but I had another idea in mind... >> > I was thinking about having the possibility to rate and report but >> without >> > any way to comment, I mean that you cannot comment in foreign languages >> but >> > you can vote and report (any operation that requires clicking is OK, >> typing >> > is not). >> > >> It's not possible to report without typing, and ratings without the typed >> information as to why the user gave the app that rating would be at best >> useless (Why is this silver? What doesn't work?), and at worst misleading. >> "False platinums" in particular are already a big problem: users give an app >> a platinum rating, but list tweaks they applied (so it really should be >> gold) or things that didn't work (so it really should be silver). Admins and >> maintainers need that typed information to assess whether a rating is >> correct. >> > I fully understand, so reporting is way complicated without typing... > Well its just a thought but maybe we should consider simplifying the whole > process, maybe this way more data will be collected. > > > Yaron: > > As a maintainer of several applications, I want to know about > improvements/regressions of all functions in those applications. Thus, I > ask those who use the products I maintain to provide details 'What Works' > and 'What Does Not Work' and these must be, for the present time, in > English. Sorry, but English is the "Universal" language. The boxes, for > now, should stay. > So you mean you have got to get some input and there's no other way of getting input from the users without typing... Do you feel like you get enough feedback or that automating parts of the reporting system would make it more widely used. I'm maintaining lots of translations in the open source world, I never feel like I get enough feedback and I truly think that if the input was at the user's fingertips it would help a lot, just my way of looking at it... > > There are many 'drop down' menus that it would be nice to input Hebrew and > have English populate them. Would this be a good idea? > The drop down boxes would be great, the simplest way of interaction is what needed, no ways of typing Hebrew... (I was thinking of a small Javascript code that will check for foreign scripts and if the script is different than the Latin script, a Red message will appear on top of the box and after clicking the button a message will pop up saying that this message contains non-universal characters and ask the user if he is sure about sending this report). Kind regards, Yaron Shahrabani. > > James McKenzie > > > > >

Oct. 20, 2010

Re: Translation of the AppDB

by James McKenzie

On 10/19/10 10:20 AM, Yaron Shahrabani wrote: > On Tue, Oct 19, 2010 at 5:16 PM, Rosanne DiMesio > <dimesio(a)earthlink.net <mailto:dimesio(a)earthlink.net>> wrote: > > On Tue, 19 Oct 2010 12:12:33 +0200 > Yaron Shahrabani <sh.yaron(a)gmail.com <mailto:sh.yaron(a)gmail.com>> > wrote: > > > I see but I had another idea in mind... > > I was thinking about having the possibility to rate and report > but without > > any way to comment, I mean that you cannot comment in foreign > languages but > > you can vote and report (any operation that requires clicking is > OK, typing > > is not). > > > It's not possible to report without typing, and ratings without > the typed information as to why the user gave the app that rating > would be at best useless (Why is this silver? What doesn't work?), > and at worst misleading. "False platinums" in particular are > already a big problem: users give an app a platinum rating, but > list tweaks they applied (so it really should be gold) or things > that didn't work (so it really should be silver). Admins and > maintainers need that typed information to assess whether a rating > is correct. > > I fully understand, so reporting is way complicated without typing... > Well its just a thought but maybe we should consider simplifying the > whole process, maybe this way more data will be collected. > > Yaron: As a maintainer of several applications, I want to know about improvements/regressions of all functions in those applications. Thus, I ask those who use the products I maintain to provide details 'What Works' and 'What Does Not Work' and these must be, for the present time, in English. Sorry, but English is the "Universal" language. The boxes, for now, should stay. There are many 'drop down' menus that it would be nice to input Hebrew and have English populate them. Would this be a good idea? James McKenzie

Oct. 19, 2010

Re: msxml3/domdoc: Partially implement ::setAttributeNode()

by testbot＠testbot.winehq.org

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=6343 Your paranoid android. === WXPPROSP3 (32 bit domdoc) === domdoc.c:6872: Test failed: got 1 === W2K3R2SESP2 (32 bit domdoc) === domdoc.c:6872: Test failed: got 1 === WVISTAADM (32 bit domdoc) === domdoc.c:6872: Test failed: got 1 === W2K8SE (32 bit domdoc) === domdoc.c:6872: Test failed: got 1 === W7PRO (32 bit domdoc) === domdoc.c:6872: Test failed: got 1 === W7PROX64 (32 bit domdoc) === domdoc.c:6872: Test failed: got 1 === W7PROX64 (64 bit domdoc) === domdoc.c:6872: Test failed: got 1

Oct. 19, 2010

Re: windowscodecs: Fix *_CopyPixels functions to properly handle a NULL rectangle

by James McKenzie

On 10/18/10 9:53 PM, Vitaliy Margolen wrote: > On 10/18/2010 12:51 PM, Krzysztof Nowicki wrote: >> Where I come from copying structures directly is considered bad >> practice and >> it's safer to use memcpy. We had problems before with broken >> compilers that >> would try to do some black magic in such cases. > Then don't use broken compilers. GCC handles struct copying this just > fine. And it's up-to compiler what sort of magic it's uses white or > black. As long as struct gets copied you are all set. > Actually, gcc modifies the code to do a PROPER memcpy. Thus you can copy the contents of a string using a pointer rather than the string if you use variable a = variable b. Using memcpy might lead to interesting results if you do the same thing. >> If the policy for Wine is to copy structures directly then I'll >> respect it. > I'm not the one setting policies, but that's what you should do - > utilize C language constructs instead of going around them. > That's very true and this has been a C language construct for quite some time. James McKenzie

Oct. 19, 2010

Re: Translation of the AppDB

by Yaron Shahrabani

On Tue, Oct 19, 2010 at 5:16 PM, Rosanne DiMesio <dimesio(a)earthlink.net>wrote: > On Tue, 19 Oct 2010 12:12:33 +0200 > Yaron Shahrabani <sh.yaron(a)gmail.com> wrote: > > > I see but I had another idea in mind... > > I was thinking about having the possibility to rate and report but > without > > any way to comment, I mean that you cannot comment in foreign languages > but > > you can vote and report (any operation that requires clicking is OK, > typing > > is not). > > > It's not possible to report without typing, and ratings without the typed > information as to why the user gave the app that rating would be at best > useless (Why is this silver? What doesn't work?), and at worst misleading. > "False platinums" in particular are already a big problem: users give an app > a platinum rating, but list tweaks they applied (so it really should be > gold) or things that didn't work (so it really should be silver). Admins and > maintainers need that typed information to assess whether a rating is > correct. > I fully understand, so reporting is way complicated without typing... Well its just a thought but maybe we should consider simplifying the whole process, maybe this way more data will be collected. Kind regards, Yaron Shahrabani. > > -- > Rosanne DiMesio <dimesio(a)earthlink.net> > > >

Oct. 19, 2010

Re: [3/6] msxml3/tests: internal schema doc storage tests (try 2)

by Adam Martinson

On 10/19/2010 11:32 AM, Alexandre Julliard wrote: > Adam Martinson<amartinson(a)codeweavers.com> writes: > > >> OK, just for future reference or do you want me to resend this set? >> > Please resend, it needs to go through testbot again. > > OK, resent, along with a few new ones.

Oct. 19, 2010

Re: [3/6] msxml3/tests: internal schema doc storage tests (try 2)

by Alexandre Julliard

Adam Martinson <amartinson(a)codeweavers.com> writes: > OK, just for future reference or do you want me to resend this set? Please resend, it needs to go through testbot again. -- Alexandre Julliard julliard(a)winehq.org

Oct. 19, 2010

Re: [3/6] msxml3/tests: internal schema doc storage tests (try 2)

by Adam Martinson

On 10/19/2010 03:17 AM, Alexandre Julliard wrote: > Adam Martinson<amartinson(a)codeweavers.com> writes: > > >> --- >> dlls/msxml3/tests/schema.c | 438 >> +++++++++++++++++++++++++++++++++----------- >> 1 files changed, 334 insertions(+), 104 deletions(-) >> > Please don't resend just a patch marked 3/6, resend a proper series > numbered from 1 so that winetestbot can figure it out. > > OK, just for future reference or do you want me to resend this set?

Oct. 19, 2010

Re: [PATCH] mscms: Handle bitmap formats BM_xRGBQUADS and BM_xBGRQUADS in from_bmformat( BMFORMAT format )

by Hans Leidekker

On Tue, 2010-10-19 at 17:35 +0200, Tomasz Michno wrote: > I thought that I only need to test function which I modified, so I > created tests in that way. Now I'm modifying them to use > TranslateBitmapBits. Should I resend the whole patch or only create > one new with only tests/transform.c? Please resend the whole patch.

Oct. 19, 2010

Re: [PATCH] mscms: Handle bitmap formats BM_xRGBQUADS and BM_xBGRQUADS in from_bmformat( BMFORMAT format )

by Tomasz Michno

Hi, > That's not how it's done in Wine, you will need to go through the > API. In this case that means you need to define a bitmap in the > appropriate format, create a color transform and then call > TranslateBitmapBits. I thought that I only need to test function which I modified, so I created tests in that way. Now I'm modifying them to use TranslateBitmapBits. Should I resend the whole patch or only create one new with only tests/transform.c? I tested wine with my modification with different images (including raw's) and applications and all works well. thanks for your time and help Tomasz Michno

Oct. 19, 2010

Re: [1/2] windowscodecs: Fix *_CopyPixels functions to properly handle a NULL rectangle (try 2).

by Alexandre Julliard

Krzysztof Nowicki <krissn(a)op.pl> writes: > I've decided to keep the optimisation in copy_pixels() that checks if the source > and dest rectangles + strides matches and if yes, does a straight memcpy of the > whole bitmap. Please do this as a separate patch. -- Alexandre Julliard julliard(a)winehq.org

Oct. 19, 2010

Re: Translation of the AppDB

by Rosanne DiMesio

On Tue, 19 Oct 2010 12:12:33 +0200 Yaron Shahrabani <sh.yaron(a)gmail.com> wrote: > I see but I had another idea in mind... > I was thinking about having the possibility to rate and report but without > any way to comment, I mean that you cannot comment in foreign languages but > you can vote and report (any operation that requires clicking is OK, typing > is not). > It's not possible to report without typing, and ratings without the typed information as to why the user gave the app that rating would be at best useless (Why is this silver? What doesn't work?), and at worst misleading. "False platinums" in particular are already a big problem: users give an app a platinum rating, but list tweaks they applied (so it really should be gold) or things that didn't work (so it really should be silver). Admins and maintainers need that typed information to assess whether a rating is correct. -- Rosanne DiMesio <dimesio(a)earthlink.net>

Oct. 19, 2010

Re: Surprise surprise, "Tmax Window OS" was a complete hoax

by James Mckenzie

Scott Ritchie <scott(a)open-vote.org> wrote: > >About a year or so ago a Korean company called Tmax was claiming to have >developed a new operating system that was 100% fully Windows compatible. > They also claimed to have their own office suite, web browser, and >more, all developed with their own special Korean engineering to keep >the license proprietary. > And they would be 'busted' anyway as Korea is one of the signaturs to the World Digital Copyright agreement. Thanks for seeing them for what they were: False. James McKenzie

Oct. 19, 2010

Re: About: d3dx9: Store transform matrix per-sprite.

by Matteo Bruni

2010/10/19 Joris Huizer <joris_huizer(a)yahoo.com>: > Hello, > > In this patch ID3DXSpriteImpl_Flush is adapted; the new loop being like: > > int i, count, start; > /* ... */ > for(start=0;start<This->sprite_count;start+=count,count=0) { > i=start; > while(i<This->sprite_count && > (count==0 || This->sprites[i].texture==This->sprites[i-1].texture)) { > /* filling in array */ > } > /* rest of the outer loop, using array and count */ > } > > It seems count is used uninitialised here on the first iteration ? > > HTH, > Joris > Argh! You're absolutely right. Apparently it managed to work fine for me so I didn't notice that silly mistake... Thank you, I'll send a fix immediately.

Oct. 19, 2010

About: d3dx9: Store transform matrix per-sprite.

by Joris Huizer

Hello, In this patch ID3DXSpriteImpl_Flush is adapted; the new loop being like: int i, count, start; /* ... */ for(start=0;start<This->sprite_count;start+=count,count=0) { i=start; while(i<This->sprite_count && (count==0 || This->sprites[i].texture==This->sprites[i-1].texture)) { /* filling in array */ } /* rest of the outer loop, using array and count */ } It seems count is used uninitialised here on the first iteration ? HTH, Joris

Oct. 19, 2010

Re: [PATCH 3/3] jscript: Add index, input and lastIndex properties to regexp funtions results

by testbot＠testbot.winehq.org

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=6322 Your paranoid android. === W98SE (32 bit) === No test summary line found === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found

Oct. 19, 2010

Re: Reflection interfaces

by Henri Verbeet

2010/10/18 Rico Schüller <kgbricola(a)web.de>: > What's the preferred way to implement the different interfaces? > > 1. Implement a parser which parses the shader (RDEF, STAT), and implement 4 > independent interfaces (each in its own dll). Only reuse the parser. > 2. Like 1, but all 4 implementations have there own parser, which leads to a > lot of duplicated code. But it is the most flexible way. It could easily > parse all possible shader blobs (theoretically they could differ from > version to version, I have no test for that, yet). > 3. Forward all interfaces to d3dcompiler_43.dll and this will contain all > interfaces and the parser. This way the codebase is the smallest possible > one, but it will offer all interfaces, which isn't what native does. Also > all further interfaces would go to d3dcompiler_43.dll. This could get > tricky. > I'm not sure. The problem with 3 is that it will prevent using the native d3dcompiler to work around e.g. the missing HLSL compiler because it doesn't implement ID3D10ShaderReflection etc. Perhaps we don't care in the long term, but at least in the short term that would be a problem. Option 1 would require d3dcompiler to have some private interface or entry point, and you'd still have the same problem as with option 3. Option 2 leads to duplicated code. There may be an option 4 though, which would be to implement the older interfaces on top of ID3D11ShaderReflection. I.e., you'd use the information you get from the ID3D11ShaderReflection interface to construct constant buffers and variables for the other interfaces, instead of parsing the shader bytecode yourself. It would need some investigation to determine how feasible this option is.

Oct. 19, 2010

Re: Translation of the AppDB

by Yaron Shahrabani

On Tue, Oct 19, 2010 at 12:40 AM, Austin English <austinenglish(a)gmail.com>wrote: > On Sun, Oct 17, 2010 at 3:59 AM, Yaron Shahrabani <sh.yaron(a)gmail.com> > wrote: > > Is there any way to localize the AppDB? > > The Israeli community assumes that having a Hebrew system will make more > > users report issues. > > It seems unlikely. There are many different languages used by wine's > users, and English is the most common understood by users/developers. > All bug reports/AppDB/etc. are in English. > I see but I had another idea in mind... I was thinking about having the possibility to rate and report but without any way to comment, I mean that you cannot comment in foreign languages but you can vote and report (any operation that requires clicking is OK, typing is not). > > Having different languages available would probably lead to a lot of > information being lost across the different language sections. > That's why commenting is not an option :) Is this a better suggestion? > > -- > -Austin >

Oct. 19, 2010

Re: [3/6] msxml3/tests: internal schema doc storage tests (try 2)

by Alexandre Julliard

Adam Martinson <amartinson(a)codeweavers.com> writes: > --- > dlls/msxml3/tests/schema.c | 438 > +++++++++++++++++++++++++++++++++----------- > 1 files changed, 334 insertions(+), 104 deletions(-) Please don't resend just a patch marked 3/6, resend a proper series numbered from 1 so that winetestbot can figure it out. -- Alexandre Julliard julliard(a)winehq.org

Oct. 19, 2010

Re: [PATCH] mscms: Handle bitmap formats BM_xRGBQUADS and BM_xBGRQUADS in from_bmformat( BMFORMAT format )

by Hans Leidekker

On Mon, 2010-10-18 at 18:10 +0200, Tomasz Michno wrote: > +/* unfortunately GetProcAddress cannot return proper pointer to > from_bmformat() function, but for this function it's not a problem, > using copy from ../transform.c */ That's not how it's done in Wine, you will need to go through the API. In this case that means you need to define a bitmap in the appropriate format, create a color transform and then call TranslateBitmapBits. Note that Wine doesn't bundle a color profile, so you need to install one, or configure your X server to use one and retrieve it via GetICMProfile.

Oct. 19, 2010

Re: user32: FindWindow() should treat an empty title same way as NULL. (Resend)

by testbot＠testbot.winehq.org

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=6315 Your paranoid android. === W7PROX64 (32 bit win) === win.c:2684: Test failed: GetActiveWindow() = 00000000 win.c:2684: Test failed: GetFocus() = 00000000 win.c:2710: Test failed: GetActiveWindow() = 00000000 win.c:2710: Test failed: GetFocus() = 00000000 win.c:2769: Test failed: GetActiveWindow() = 00000000 win.c:2769: Test failed: GetFocus() = 00000000 === W7PROX64 (64 bit win) === win.c:2684: Test failed: GetActiveWindow() = 0000000000000000 win.c:2684: Test failed: GetFocus() = 0000000000000000 win.c:2710: Test failed: GetActiveWindow() = 0000000000000000 win.c:2710: Test failed: GetFocus() = 0000000000000000 win.c:2769: Test failed: GetActiveWindow() = 0000000000000000 win.c:2769: Test failed: GetFocus() = 0000000000000000

Oct. 19, 2010

Surprise surprise, "Tmax Window OS" was a complete hoax

by Scott Ritchie

About a year or so ago a Korean company called Tmax was claiming to have developed a new operating system that was 100% fully Windows compatible. They also claimed to have their own office suite, web browser, and more, all developed with their own special Korean engineering to keep the license proprietary. This lead some here to justifiably worry that they were using Wine code without permission, as well as dozens of other free software projects. At the time I called it an obvious fraud. Anyway I check backed in on it and, no surprise, Tmax Soft has disavowed all knowledge of the project -- the website is dead, no news about it has appeared since the "launch", and although there are several quicksearches on their main website for the product they return no results. Fortunately, with the exception of a few particularly bad tech journalists, no one seems to have been actually fooled. -Scott Ritchie

Oct. 19, 2010

Re: windowscodecs: Fix *_CopyPixels functions to properly handle a NULL rectangle

by Vitaliy Margolen

On 10/18/2010 12:51 PM, Krzysztof Nowicki wrote: > Where I come from copying structures directly is considered bad practice and > it's safer to use memcpy. We had problems before with broken compilers that > would try to do some black magic in such cases. Then don't use broken compilers. GCC handles struct copying this just fine. And it's up-to compiler what sort of magic it's uses white or black. As long as struct gets copied you are all set. > If the policy for Wine is to copy structures directly then I'll respect it. I'm not the one setting policies, but that's what you should do - utilize C language constructs instead of going around them. Vitaliy.

Oct. 18, 2010

Re: ws2_32: Add support and tests for WSARecvMsg and IP_PKTINFO.

by Mike Kaplinskiy

On Sun, Oct 17, 2010 at 5:51 PM, Erich Hoover <ehoover(a)mines.edu> wrote: > On Sat, Oct 16, 2010 at 1:41 PM, Mike Kaplinskiy > <mike.kaplinskiy(a)gmail.com> wrote: >> On Thu, Oct 14, 2010 at 10:08 PM, Erich Hoover <ehoover(a)mines.edu> wrote: >>> ... >>> DWORD flags; >>> unsigned int n_iovecs; >>> unsigned int first_iovec; >>>+ WSABUF *control; >>> struct iovec iovec[1]; >>> } ws2_async; >> I would prefer control declared above n_iovecs purely for >> organizational purposes (params first, buffers and info about them >> last). Also you can't just store the passed in WSABUF pointer; it may >> be stack allocated and may disappear before the overlapped call >> completes. You could just add control and control_len instead of using >> WSABUF. > > That's why in the original version I had those parameters split out; > however, I realized that since the length parameter gets modified by > WSARecvMsg that it is impossible for the WSAMSG structure (or any of > its parameters) to disappear before WSARecvMsg completes. I have > re-written the tests (see patch 4 at the end) to show that this > structure gets modified even when an overlapped request is used and > results in modification of the structure at a later time, the results > from the test bot can be found here: > https://testbot.winehq.org/JobDetails.pl?Key=6250 > Fair enough. Would you mind adding a test to see if the flags are changed as well (and see if the flags returned by WSAGetOverlappedResult are of that sort as well)? If it doesn't you can drop lpFlags and just copy it from flags. Otherwise you may want to rename it to be less windows-y (like local_flags and flags). >>>+ /* Insufficient room for control headers */ >>>+ errno = EINVAL; >> This should probably be ENOMEM then. On that note, it would be nice to >> test what windows actually does when you don't give it enough space >> (like 1 byte). We should probably rewrite that function at some point >> since I'm not sure what errno it returns anymore. > > This is odd, I must have copy-pasted the wrong error code from the > recvmsg() manual page. I've investigated this issue more and > discovered that the code should actually be EMSGSIZE. This change and > the corresponding tests can be found in my revised patches (at the > end), I've also included support for reporting back the MSG_CTRUNC > flag. > Hmm, you may also want to warn/err on EMSGSIZE coming from recvmsg. >>>+#ifdef IP_PKTINFO >>>+ case WS_IP_PKTINFO: >>>+#endif >> You probably don't need the ifdef. convert_sockopt will just fail. > > I did this to match how IP_HDRINCL was done nearby, should this really > be changed? > We seem to mix both styles. I don't have any preference then. Mike.

Oct. 18, 2010

Re: [PATCH 6/8] jscript: Added VBArray.getItem() implementation

by testbot＠testbot.winehq.org

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=6284 Your paranoid android. === W98SE (32 bit) === No test summary line found === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found

Oct. 18, 2010

Re: [PATCH 8/8] jscript: Added VBArray handling to to_object()

by testbot＠testbot.winehq.org

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=6286 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

Oct. 18, 2010

Fwd: Invitation: Apply to be a Mentoring Organization for GCI

by Austin English

I hadn't see any talk about this, so passing this along from Google: -- -Austin ---------- Forwarded message ---------- From: Carol Smith <carols(a)google.com> Date: Mon, Oct 18, 2010 at 8:10 AM Subject: Invitation: Apply to be a Mentoring Organization for GCI To: Google Summer of Code Mentors List <google-summer-of-code-mentors-list(a)googlegroups.com> Hi everyone, Starting today we are accepting applications from organizations who have participated in a previous Google Summer of Code (you!) to get involved with our contest for 13-18 year olds, Google Code-in [0]. If you haven’t read up about Google Code-in, you can check out our announcement [1] to this list from a couple weeks ago or read our blog post [2] about the contest. If your org has decided it would like to apply to participate in the contest, you can fill out the application [3] on Melange now. The deadline for applications is 29 October at 23:00 UTC. Also, if you want to look over the rules for the contest before applying to participate they can be found on the organization application [3] at the bottom. They will also be posted on the Google Code-in website [0] in the next couple days in other languages. We hope your org will consider participating. [0] - http://code.google.com/gci [1] - http://groups.google.com/group/google-summer-of-code-mentors-list/msg/0985e… [2] - http://google-opensource.blogspot.com/2010/10/google-code-in-schools-out-co… [3] - http://socghop.appspot.com/gci/org_app/take/gci_program/google/gci2010/orga… Cheers, Carol

Oct. 18, 2010

Apps for Wine on ARM

by Detlef Riekenberg

Here is a free compiler to create apps on ARM: http://www.smorgasbordet.com/pellesc/ (Untested, how this app works in Wine) -- By by ... Detlef

Oct. 18, 2010

Re: Translation of the AppDB

by Austin English

On Sun, Oct 17, 2010 at 3:59 AM, Yaron Shahrabani <sh.yaron(a)gmail.com> wrote: > Is there any way to localize the AppDB? > The Israeli community assumes that having a Hebrew system will make more > users report issues. It seems unlikely. There are many different languages used by wine's users, and English is the most common understood by users/developers. All bug reports/AppDB/etc. are in English. Having different languages available would probably lead to a lot of information being lost across the different language sections. -- -Austin

Oct. 18, 2010

Re: windowscodecs: Fix *_CopyPixels functions to properly handle a NULL rectangle

by David Laight

On Mon, Oct 18, 2010 at 08:51:49PM +0200, Krzysztof Nowicki wrote: > W dniu 17.10.2010 18:49, Vitaliy Margolen pisze: > >On 10/17/2010 01:59 AM, Krzysztof Nowicki wrote: > >>Doing a memcpy to a local rectangle seems a morenatural way to do it > > > >Not really. Doing RECT = RECT is the natural way to do it. Don't use > >memcpy to copy one structure to another structure of the same type. > > > >Vitaliy. > > > > Where I come from copying structures directly is considered bad practice > and it's safer to use memcpy. We had problems before with broken > compilers that would try to do some black magic in such cases. So the > policy was to use memcpy for structures because it's safer. A good > compiler will inline such a memcpy anyway. Actually, for a normal application, the compiler knows what memcpy() does and can make assumptions based on the actual structure type involved. This can cause extreme grief if you try to copy from a misaligned pointer into a local variable (strictly, in valid C, you can never get a misaligned pointer - and the compiler will use word accesses). David -- David Laight: david(a)l8s.co.uk

Oct. 18, 2010

Re: [PATCH 7/8] jscript: Added VBArray.toArray() implementation

by testbot＠testbot.winehq.org

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=6285 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

Oct. 18, 2010

Reflection interfaces

by Rico Schüller

Hi, I've a question concerning d3dcompiler, d3d10 and d3dx10 dlls in combination with the shader reflection interfaces. It seems there are 4 different known interfaces: IID_ID3D10ShaderReflection -> d3d10.dll (D3D10ReflectShader()) IID_ID3D10ShaderReflection1 -> d3dx10_xx.dll (D3DX10ReflectShader()) IID_ID3D11ShaderReflection_42 -> d3dcompiler_(40-42).dll (D3DReflect()) IID_ID3D11ShaderReflection -> d3dcompiler_(43).dll (D3DReflect()) The IID_ID3D11ShaderReflection_42 is just a new name for a IID_ID3D11ShaderReflection interface for version 40-42. In d3dcompiler 43 the interface changed and now has a new IID. What's the preferred way to implement the different interfaces? 1. Implement a parser which parses the shader (RDEF, STAT), and implement 4 independent interfaces (each in its own dll). Only reuse the parser. 2. Like 1, but all 4 implementations have there own parser, which leads to a lot of duplicated code. But it is the most flexible way. It could easily parse all possible shader blobs (theoretically they could differ from version to version, I have no test for that, yet). 3. Forward all interfaces to d3dcompiler_43.dll and this will contain all interfaces and the parser. This way the codebase is the smallest possible one, but it will offer all interfaces, which isn't what native does. Also all further interfaces would go to d3dcompiler_43.dll. This could get tricky. Attached is a test program and the output. The programm is compiled with "i686-pc-mingw32-gcc -I./wine/build/include -I./wine/git/include compilertest.c". The test for D3DReflectCode() doesn't work, so please ignore that part. Cheers Rico

Oct. 18, 2010

Re: [PATCH 1/2] urlmon: Added IInternetProtocolEx support to BindProtocol.

by Jacek Caban

On 10/18/10 8:15 PM, Alexandre Julliard wrote: > Jacek Caban<jacek(a)codeweavers.com> writes: > >> --- >> dlls/urlmon/bindprot.c | 343 +++++++++++++++++++++++++++++------------------ >> 1 files changed, 212 insertions(+), 131 deletions(-) > It breaks the tests: > > ../../../tools/runtest -q -P wine -M gameux.dll -T ../../.. -p gameux_test.exe.so gameexplorer.c&& touch gameexplorer.ok > gameexplorer.c:571: Test failed: IGameExplorer::AddGame failed (error 0x80004005) > gameexplorer.c:589: Test failed: IGameExplorer::AddGame failed (error 0x80004005) > gameexplorer.c:640: Test failed: IGameExplorer2::InstallGame failed (error 0x80004005) > gameexplorer.c:643: Test failed: cannot find game with GDF path L"Z:\\home\\julliard\\wine\\wine\\dlls\\gameux\\tests\\gameux_test.exe" in scope 2 > make: *** [gameexplorer.ok] Error 4 I didn't think about running these tests, sorry. It looks like res: protocol handling in CreateUri is not up to the task yet. Jacek

Oct. 18, 2010

Re: windowscodecs: Fix *_CopyPixels functions to properly handle a NULL rectangle

by Krzysztof Nowicki

W dniu 17.10.2010 18:49, Vitaliy Margolen pisze: > On 10/17/2010 01:59 AM, Krzysztof Nowicki wrote: >> Doing a memcpy to a local rectangle seems a morenatural way to do it > > Not really. Doing RECT = RECT is the natural way to do it. Don't use > memcpy to copy one structure to another structure of the same type. > > Vitaliy. > Where I come from copying structures directly is considered bad practice and it's safer to use memcpy. We had problems before with broken compilers that would try to do some black magic in such cases. So the policy was to use memcpy for structures because it's safer. A good compiler will inline such a memcpy anyway. If the policy for Wine is to copy structures directly then I'll respect it. K.

Oct. 18, 2010

Re: [PATCH 5/8] jscript: Added VBArray.dimensions() implementation

by testbot＠testbot.winehq.org

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=6283 Your paranoid android. === W98SE (32 bit) === No test summary line found === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found

Oct. 18, 2010

Re: [PATCH 2/8] jscript: Implement VBArray constructor

by Jacek Caban

Hi Piotr, +(tmp = new VBArray(createArray())).f = Object.prototype.toString; +ok(tmp.f() === "[object Object]", "tmp.f() = " + tmp.f()); We have Function.call implemented now, you can use String.prototype.toString.call(new VBArray ...). + case DISPATCH_METHOD: + if(arg_cnt(dp)<1 || V_VT((arg = get_arg(dp, 0)))!=(VT_ARRAY|VT_VARIANT)) + return throw_type_error(ctx, ei, IDS_NOT_VBARRAY, NULL); + + VariantCopy(retv, arg); Please add error handling. Also a test would be nice, at least something like getVT(createArray()). + case DISPATCH_CONSTRUCT: + if(arg_cnt(dp)<1 || V_VT((arg = get_arg(dp, 0)))!=(VT_ARRAY|VT_VARIANT)) + return throw_type_error(ctx, ei, IDS_NOT_VBARRAY, NULL); + + hres = alloc_vbarray(ctx, NULL,&vbarray); + if(FAILED(hres)) + return hres; + SafeArrayCopy(V_ARRAY(arg),&vbarray->safearray); Error handling. (From 'PATCH 4/8] jscript: Added VBArray.ubound() implementation'): + if(retv) + num_set_val(retv, dim); num_set_val is for DOUBLE and eventually abused for integers out of VT_I4 range. You can set value directly here to avoid conversions. It might be a good idea to introduce something like var_set_int helper given that we could use it in quite a few places. (From '[PATCH 6/8] jscript: Added VBArray.getItem() implementation'): + size = arg_cnt(dp); + if(size< SafeArrayGetDim(vbarray->safearray)) + return throw_range_error(ctx, ei, IDS_SUBSCRIPT_OUT_OF_RANGE, NULL); + + indexes = heap_alloc(sizeof(int)*size); Error handling. (From '[PATCH 8/8] jscript: Added VBArray handling to to_object()'): -exception_test(function() {(new VBArray(createArray())).lbound("aaa");}, "RangeError", -2146828279); -exception_test(function() {(new VBArray(createArray())).lbound(3);}, "RangeError", -2146828279); +exception_test(function() {createArray().lbound("aaa");}, "RangeError", -2146828279); +exception_test(function() {createArray().lbound(3);}, "RangeError", -2146828279); There is no reason to change these tests, esp. that they were better before the change. Please add proper to_object tests, for example by calling Object() function or doing proper call on explicitly converted object. + SafeArrayCopy(sa,&vbarray->safearray); Error handling. Jacek

Oct. 18, 2010

Re: [PATCH 1/2] urlmon: Added IInternetProtocolEx support to BindProtocol.

by Alexandre Julliard

Jacek Caban <jacek(a)codeweavers.com> writes: > --- > dlls/urlmon/bindprot.c | 343 +++++++++++++++++++++++++++++------------------ > 1 files changed, 212 insertions(+), 131 deletions(-) It breaks the tests: ../../../tools/runtest -q -P wine -M gameux.dll -T ../../.. -p gameux_test.exe.so gameexplorer.c && touch gameexplorer.ok gameexplorer.c:571: Test failed: IGameExplorer::AddGame failed (error 0x80004005) gameexplorer.c:589: Test failed: IGameExplorer::AddGame failed (error 0x80004005) gameexplorer.c:640: Test failed: IGameExplorer2::InstallGame failed (error 0x80004005) gameexplorer.c:643: Test failed: cannot find game with GDF path L"Z:\\home\\julliard\\wine\\wine\\dlls\\gameux\\tests\\gameux_test.exe" in scope 2 make: *** [gameexplorer.ok] Error 4 -- Alexandre Julliard julliard(a)winehq.org

Oct. 18, 2010

Re: [PATCH 4/8] jscript: Added VBArray.ubound() implementation

by testbot＠testbot.winehq.org

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=6282 Your paranoid android. === W98SE (32 bit) === No test summary line found === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found

Oct. 18, 2010