Re: [PATCH 7/7] wined3d: Move A4L4 conversion to the formats table.

2010-04-12 Thread Paul Vriens
On 04/13/2010 12:46 AM, Greg Geldorp wrote: From: Ge van Geldorp From: Roderick Colenbrander I'm not sure how the bot works but it looks like it concatenated some of my patches together and then applied to them. I have no idea to what Wine version it was since it applies fine on Wine git here.

Re: winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Dmitry Timoshkov
wrote: > >Since there is no neither todo_wine statements in the tests, nor test > >failures > >under Wine that means that both the tests and the patch are not OK. > > What has Wine to say when talking about the correctness > of tests? Only native dictates what the test result should be. Of co

RE: [PATCH 7/7] wined3d: Move A4L4 conversion to the formats table.

2010-04-12 Thread Greg Geldorp
> From: Ge van Geldorp > > From: Roderick Colenbrander > > > > I'm not sure how the bot works but it looks like it concatenated some > > of my patches together and then applied to them. I have no idea to > > what Wine version it was since it applies fine on Wine git here. It > > could be that I'm

RE: [PATCH 7/7] wined3d: Move A4L4 conversion to the formats table.

2010-04-12 Thread Greg Geldorp
> From: Roderick Colenbrander > > I'm not sure how the bot works but it looks like it concatenated some > of my patches together and then applied to them. I have no idea to > what Wine version it was since it applies fine on Wine git here. It > could be that I'm doing something wrong but I don't s

Re: [PATCH 7/7] wined3d: Move A4L4 conversion to the formats table.

2010-04-12 Thread Roderick Colenbrander
I'm not sure how the bot works but it looks like it concatenated some of my patches together and then applied to them. I have no idea to what Wine version it was since it applies fine on Wine git here. It could be that I'm doing something wrong but I don't see where. Roderick On Mon, Apr 12, 201

Re: [PATCH] d3dx9_36/tests: Move surface tests into surface.c (try 2)

2010-04-12 Thread winetestbot
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://winetestbot.geldorp.nl/JobDetails.pl?Key=1401 Your paranoid andr

Re: [PATCH 7/7] wined3d: Move A4L4 conversion to the formats table.

2010-04-12 Thread winetestbot
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://winetestbot.geldorp.nl/JobDetails.pl?Key=1399 Your paranoid andr

winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Joerg-Cyril.Hoehle
Dmitry Timoshkov wrote: >Since there is no neither todo_wine statements in the tests, nor test failures >under Wine that means that both the tests and the patch are not OK. What has Wine to say when talking about the correctness of tests? Only native dictates what the test result should be. The

Re: winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Dmitry Timoshkov
wrote: > >I don't see any corresponding changes in that patch that remove todo_wine > >statements > >in the tests. That suggests that either such tests don't exist, or that > >change actually > >fixes nothing. > A third explanation is possible: > It's an example of one bug shadowing another bug

winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Joerg-Cyril.Hoehle
Dmitry Timoshkov wrote: >I don't see any corresponding changes in that patch that remove todo_wine >statements >in the tests. That suggests that either such tests don't exist, or that change >actually >fixes nothing. A third explanation is possible: It's an example of one bug shadowing another b

Re: winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Dmitry Timoshkov
[cc:ing wine-devel] wrote: > this is the second time that AJ commits a patch of mine despite > a comment of yours. I think that Alexandre has commited your patch slightly in a hurry, usually he waits at least several hours unless the patch is obviously correct (which is not the case here IMO).

Re: winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Alexandre Julliard
writes: > Ok. After being bitten at least once by assignment/comparison > mismatch I promised myself to use that style. I'm myself used > to read code as "if A equals 3" rather than "if 3 is the value > of A" but I'm convinced that's just a matter of getting used > to this style that is less er

Re: winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Dmitry Timoshkov
wrote: > >> +FIXME("(%04x) vkey %04X stub\n", dwFlags, lpParms->nVirtKey); > >That change is unwanted. > I can remove it, but why? Is supporting break keys a WONTFIX in Wine? If that's really a problem then the first thing to do is start with some bug reports or test cases. Adding spurious F

winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Joerg-Cyril.Hoehle
Dmitry Timoshkov wrote: >> +FIXME("(%04x) vkey %04X stub\n", dwFlags, lpParms->nVirtKey); >That change is unwanted. I can remove it, but why? Is supporting break keys a WONTFIX in Wine? >Do you have a test case which shows that notofication is not sent is the >failure case? It's already in

Re: winmm: MCI system commands are not eligible for auto-open. (try 2)

2010-04-12 Thread Dmitry Timoshkov
wrote: > +FIXME("(%04x) vkey %04X stub\n", dwFlags, lpParms->nVirtKey); That change is unwanted. > -if (dwFlags & MCI_NOTIFY) > - mciDriverNotify((HWND)lpParms->dwCallback, wDevID, > -(dwRet == 0) ? MCI_NOTIFY_SUCCESSFUL : > MCI_NOTIFY_FAILURE); > - > +

[rfc] realtime priority patch based on rtkit

2010-04-12 Thread Maarten Lankhorst
Hi all, I decided to dig through the sample documentation for rtkit and see if I could find a way to make it work for wine. There's still a few things missing: hardcoded libdbus soname, no checking if dbus is available, -I/usr/include/dbus-1.0 -I/usr/lib/dbus-1.0/include should be added to ntd

Re: Re : d3dx9: Implement D3DXDeclaratorFromFVF with tests

2010-04-12 Thread Henri Verbeet
On 12 April 2010 12:10, paulo lesgaz wrote: > What do you mean? > > I think I kept the file format. > > And what are the other things ? ;) > Sorry, but I really think these are things you should be able to figure out on your own.

Re: [PATCH 09/10] wined3d: Improve FBO support in ClearSurface.

2010-04-12 Thread Henri Verbeet
On 12 April 2010 12:44, Roderick Colenbrander wrote: > -    context = context_acquire(This, (IWineD3DSurface *)target, > CTXUSAGE_CLEAR); > +    if (wined3d_settings.offscreen_rendering_mode == ORM_FBO) > +    { > +        if (!surface_is_offscreen((IWineD3DSurface *)target)) > +        { > +    

Re: [PATCH 10/10] wined3d: Add BLT_OP_COLOR_FILL to blit_supported and use it in BltOverride.

2010-04-12 Thread Henri Verbeet
On 12 April 2010 12:44, Roderick Colenbrander wrote: > +            if (ffp_blit.blit_supported(&myDevice->adapter->gl_info, > BLIT_OP_COLOR_FILL, > +                                        NULL, 0, 0, NULL, > +                                        &dst_rect, This->resource.usage, > This->reso

Re: [PATCH 03/10] wined3d: Move D24X4S4 to the formats table.

2010-04-12 Thread Henri Verbeet
On 12 April 2010 12:44, Roderick Colenbrander wrote: >     {WINED3DFMT_D16_UNORM,              GL_DEPTH_COMPONENT24_ARB,         > GL_DEPTH_COMPONENT24_ARB,               0, >             GL_DEPTH_COMPONENT,         GL_UNSIGNED_SHORT,                0, >             WINED3DFMT_FLAG_POSTPIXELSHADE

Re: wineserver: Fix French manpage

2010-04-12 Thread Frédéric Delanoy
2010/4/12 Nicolas Le Cam : > Le 12 avril 2010 11:02, Frédéric Delanoy a écrit > : >> 2010/4/9 Nicolas Le Cam : >>> Le 9 avril 2010 13:30, Frédéric Delanoy a >>> écrit : 2010/4/9 Nicolas Le Cam : > Hi Frédéric, > >>+processus clients se sont terminés. Ceci évite le coût inhérent

Re: wineserver: Fix French manpage

2010-04-12 Thread Nicolas Le Cam
Le 12 avril 2010 11:02, Frédéric Delanoy a écrit : > 2010/4/9 Nicolas Le Cam : >> Le 9 avril 2010 13:30, Frédéric Delanoy a écrit >> : >>> 2010/4/9 Nicolas Le Cam : Hi Frédéric, >+processus clients se sont terminés. Ceci évite le coût inhérent à l'arrêt sont -> soient >>>

Re: usbhub.sys: add stubbed usbhub.sys

2010-04-12 Thread Alexandre Julliard
Damjan Jovanovic writes: > I've hacked at this enough to get some basic USB I/O working, so now I > have a better idea of what's necessary. > > I like usbhub.sys separate from usbd.sys because: > * The service is called Usbhub in the registry, drivers might depend > on the service, and having the

Re: winetricks failed, expecting "system32" where I have "System32" and so on

2010-04-12 Thread Francois Gouget
On Fri, 9 Apr 2010, Austin English wrote: [...] > Why did you rename system32? Are you linking your ~/.wine/drive_c to a > real windows installation? Windows paths are case insensitive so anything in drive_c should be treated in a case-insensitive way. If winetricks does not do so, then it is bu

Re: [PATCH] d3dx9_36/tests: Move surface tests into surface.c

2010-04-12 Thread Christian Costa
The issues were existing before and thus not related to the patch. The patch is only a file renaming and the test selector changed from "texture" to "surface" to match tested functions. I will see if I can do something to fix the test failures. Christian (Marvin) a écrit : Hi, While running

Re : d3dx9: Implement D3DXDeclaratorFromFVF with tests

2010-04-12 Thread paulo lesgaz
What do you mean? I think I kept the file format. And what are the other things ? ;) A+ David De : Henri Verbeet À : wine-devel@winehq.org Envoyé le : Lun 12 avril 2010, 11 h 52 min 15 s Objet : Re: d3dx9: Implement D3DXDeclaratorFromFVF with tests Formatt

Re: d3dx9: Implement D3DXDeclaratorFromFVF with tests

2010-04-12 Thread Henri Verbeet
Formatting, among others.

Re: [PATCH] d3dx9_36/tests: Move surface tests into surface.c

2010-04-12 Thread winetestbot
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://winetestbot.geldorp.nl/JobDetails.pl?Key=1363 Your paranoid andr

Re: winetricks failed, expecting "system32" where I have "System32" and so on

2010-04-12 Thread Helge Hafting
Austin English wrote: On Fri, Apr 9, 2010 at 8:45 AM, Helge Hafting wrote: I just tried installing gdiplus using winetricks, on a debian testing system. This failed, as winetricks tried to copy stuff into windows/system32, where I happen to have Windows/System32. I first tried making symlinks

Re: [Wine] updated winetricks directx-beta verb

2010-04-12 Thread perryh
Austin English wrote: > I've updated winetricks ... > > Get it at http://winezeug.googlecode.com/svn/trunk/winetricks Were you aware that VERSION is still set to 20100317?

Re: wineserver: Fix French manpage

2010-04-12 Thread Frédéric Delanoy
2010/4/9 Nicolas Le Cam : > Le 9 avril 2010 13:30, Frédéric Delanoy a écrit : >> 2010/4/9 Nicolas Le Cam : >>> Hi Frédéric, >>> +processus clients se sont terminés. Ceci évite le coût inhérent à l'arrêt >>> sont -> soient >>> +\fIwineserver\fR dans le chemin système ou quelques autres empl

Re: tools: Add French translation of wineprefixcreate manpage

2010-04-12 Thread Frédéric Delanoy
2010/4/9 Nicolas Le Cam : > Le 9 avril 2010 13:23, Frédéric Delanoy a écrit : >> Well I thought about putting it in non-infinitive form, but it didn't >> sound/feel right to me. >> >> In fact "N'affiche" would be more like a description, while >> "N'afficher" is more an action/modifier/behaviour "

Re: [Wine] updated winetricks directx-beta verb

2010-04-12 Thread Austin English
On Mon, Apr 12, 2010 at 1:43 AM, wrote: > Austin English wrote: > >> I've updated winetricks ... >> >> Get it at http://winezeug.googlecode.com/svn/trunk/winetricks > > Were you aware that VERSION is still set to 20100317? Yes, Dan updates that when he puts it at http://kegel.com/wine/winetrick