RE: DLL loading prolem when injecting into another process

2008-08-01 Thread Hongbo Ni
Rolf Kalbermatter wrote: > I do not think that the solution is to "correct" LoadLibrary to actually do > a search for linked libraries in the parent library's directory too. > > However I fear the proper fix might go deep into Wine server and probably is > above most people's head except for Alex

Re: DLL loading prolem when injecting into another process

2008-08-01 Thread Steven Edwards
On Sat, Aug 2, 2008 at 2:10 AM, Steven Edwards <[EMAIL PROTECTED]> wrote: > Its also documented in Inside Windows by Jeffery Richter if you happen > to have a copy of that laying around. Sorry make that Advanced Windows. I gave my copy to a friend so I don't recall much about the chapter, except t

Re: DLL loading prolem when injecting into another process

2008-08-01 Thread Steven Edwards
On Fri, Aug 1, 2008 at 6:25 PM, Rolf Kalbermatter <[EMAIL PROTECTED]> wrote: > Ok I think I understand. You are using a technique I have once read about in > Microsoft System Journal back in about 1994. And this is certainly tricky. Its also documented in Inside Windows by Jeffery Richter if you h

RE: DLL loading prolem when injecting into another process

2008-08-01 Thread Rolf Kalbermatter
Hongbo Ni wrote: > Back to DLL injecting, It works since Windows 95. My program > has been working for many years, but not on Wine. > > Here is how to inject a DLL into another process: > . Ok I think I understand. You are using a technique I have once read about in Microsoft System Journ

Re: dinput: the DIK_ keycode is not the same as the scancode.

2008-08-01 Thread Vitaliy Margolen
James Mckenzie wrote: > Vitaliy Margolen <[EMAIL PROTECTED]> at Jul 31, 2008 10:10 PM (PNT) wrote > about Re: dinput: the DIK_ keycode is not the same as the scancode. >>> -dik_code = hook->scanCode & 0xff; >>> +dik_code = This->ScanCodemap[hook->scanCode]; >> Scan code can be larger then

Re: crypt32(3/3): More fully implement CryptSIPRetrieveSubjectGuid

2008-08-01 Thread James Hawkins
On Fri, Aug 1, 2008 at 2:23 PM, Juan Lang <[EMAIL PROTECTED]> wrote: >> Please put something in the title to let us know that this is a resumission >> like (try 2). Some folks resend and resend their patches when Alexandre >> does not 'immediately' react to them. > > I did, in the body. They're

Re: crypt32(3/3): More fully implement CryptSIPRetrieveSubjectGuid

2008-08-01 Thread Juan Lang
> Please put something in the title to let us know that this is a resumission > like (try 2). Some folks resend and resend their patches when Alexandre does > not 'immediately' react to them. I did, in the body. They're not different than the original submissions, except for being rediffed, as

Re: crypt32(3/3): More fully implement CryptSIPRetrieveSubjectGuid

2008-08-01 Thread James Mckenzie
Juan: Please put something in the title to let us know that this is a resumission like (try 2). Some folks resend and resend their patches when Alexandre does not 'immediately' react to them. Thank you. James McKenzie -Original Message- >From: Juan Lang <[EMAIL PROTECTED]> >Sent: Au

Re: [2/7] WineD3D: use the drawable size to set up blitting

2008-08-01 Thread Tobias Jakobi
Stefan Dösinger wrote: >> I'm sorry, I cannot reproduce this. Which settings are you using? > I see it now, it only occurs with medium post processing settings. OK, I can't use the high settings for now because of the slowdowns. Medium is the only half-working option for now. > > The patch obvio

RE: New valgrind warnings in wined3d/arb_program_shader.c

2008-08-01 Thread Stefan Dösinger
> gcc is perfectly correct, if you don't want padding space then your > bitfields have to add up to the size of an integer. Figured that out by now :-) I thought it left paddings between my flags, but actually there are 12 unused bits at the end, and one uninitialized, because unused, but still h

Re: New valgrind warnings in wined3d/arb_program_shader.c

2008-08-01 Thread Alexandre Julliard
Stefan Dösinger <[EMAIL PROTECTED]> writes: >> It had a level of indirection mistake, but yes, the >> following patch fixes the new warnings: > I think it's gcc's fault, or the code relying on something that the C spec > doesn't guarantee. It seems to me that gcc doesn't pack the bitfields > prope

RE: DLL loading prolem when injecting into another process

2008-08-01 Thread Hongbo Ni
Rolf Kalbermatter wrote: > My guess is that it will however fail on most Windows versions except maybe > Vista which would still be a reason to try to implement that behaviour in > Wine > too. However you will likely have to do quite > a bit more tests to then figure out the exact conditions un

RE: [1/7] WineD3D: use the window size as drawable size

2008-08-01 Thread Stefan Dösinger
> It would require adding more of them, but that's good, it makes the > locking more explicit. We really have to stop taking a big global lock > and executing thousands of lines of code inside it. As long as you are > only moving the lock inside the individual functions the cost is the > same. In t

RE: New valgrind warnings in wined3d/arb_program_shader.c

2008-08-01 Thread Stefan Dösinger
> It had a level of indirection mistake, but yes, the > following patch fixes the new warnings: I think it's gcc's fault, or the code relying on something that the C spec doesn't guarantee. It seems to me that gcc doesn't pack the bitfields properly as we hoped, and leaves padding bytes in between

Re: [1/7] WineD3D: use the window size as drawable size

2008-08-01 Thread Alexandre Julliard
Stefan Dösinger <[EMAIL PROTECTED]> writes: > To stay on topic though, I oppose moving the lock into every state > handler. There are just the scissorrect and viewport functions that > may have to perform User32 calls(if the patch turns out to be > correct). Moving the lock call into the state han

Re: [Gdiplus] Try harder to get a monospace font

2008-08-01 Thread Vitaly Lipatov
В сообщении от 1 августа 2008 Adam Petaccia написал(a): > + * If Courier New is not found, the replacement font Liberation Mono is > used. */ Why do not use font replacement rules in the registry? I think it is more common way to subst Liberation fonts instead proprietary one. -- Vitaly Lipato

RE: [1/7] WineD3D: use the window size as drawable size

2008-08-01 Thread Stefan Dösinger
> > +LEAVE_GL(); > > target->get_drawable_size(target, &width, &height); > > +ENTER_GL(); > > That's ugly, the lock isn't even entered in that function. Can we > please > start to clean up the ENTER_GL mess to only grab the lock around GL > calls and not over huge amounts

Re: New valgrind warnings in wined3d/arb_program_shader.c

2008-08-01 Thread Dan Kegel
Stefan Dösinger <[EMAIL PROTECTED]> wrote: > Does the attached patch fix the warnings? It had a level of indirection mistake, but yes, the following patch fixes the new warnings: diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index a9fc780..6e1e0ac 100644 --- a/dlls/wined3d/utils.c +++

DIB engine status

2008-08-01 Thread Huw Davies
Hi folks, Here's an update on the DIB engine that I've been working on for the last few months. This work has been made possible by the generosity of Google - so thanks go to them and of course to Dan. You can find the code at http://repo.or.cz/w/wine/dibeng-hd.git What's supported

Re: crypt32(1/3): Use ReadFile rather than a memory-mapped file to check the type of a file

2008-08-01 Thread Alexandre Julliard
"Juan Lang" <[EMAIL PROTECTED]> writes: > From 064365f26630c76546a6c8bedc1e99d47b74563d Mon Sep 17 00:00:00 2001 > From: Juan Lang <[EMAIL PROTECTED]> > Date: Thu, 31 Jul 2008 09:08:09 -0700 > Subject: [PATCH] Use ReadFile rather than a memory-mapped file to check the > type of a file This doesn

Re: [PATCH 1/2] shell32: Add a test for FindExecutableA.

2008-08-01 Thread Alexandre Julliard
"Lei Zhang" <[EMAIL PROTECTED]> writes: > +{ > +char expand[MAX_PATH]; > +strcpy(command, "foobar"); > +strcpy(filename, "foo.msi"); > +file = CreateFileA(filename, GENERIC_WRITE, 0, NULL, CREATE_ALWAYS, > 0, NULL); > +ok(file != INVALID_HANDLE_VALUE, "

Re: d3dx9: Implementation of D3DXGetPixelShaderProfile 1/2 try 3

2008-08-01 Thread Alexandre Julliard
Luis Busquets <[EMAIL PROTECTED]> writes: > + case D3DPS_VERSION(2, 0): > + if ((caps.PS20Caps.NumTemps>=22) && > + (caps.PS20Caps.Caps&D3DPS20CAPS_ARBITRARYSWIZZLE) && > + (caps.PS20Caps.Caps&D3DPS20CAPS_GRADIENTINSTRUCTIONS

RE: DLL loading prolem when injecting into another process

2008-08-01 Thread Rolf Kalbermatter
Hongbo Ni wrote: >> Not sure about what you mean with injecting here but in normal application operation >> when loading a DLL explicitedly from a specific directory (which is not the current >> directory nor the application directory or one of the standard search paths), >> Windows > >

Re: [1/7] WineD3D: use the window size as drawable size

2008-08-01 Thread Alexandre Julliard
Stefan Dösinger <[EMAIL PROTECTED]> writes: > @@ -4681,7 +4681,9 @@ static void viewport_miscpart(DWORD state, > IWineD3DStateBlockImpl *stateblock, W > stateblock->viewport.Width, stateblock->viewport.Height); > } else { > target = (IWineD3DSurfaceImpl *) > sta

Re: dinput: the DIK_ keycode is not the same as the scancode.

2008-08-01 Thread Alexandre Julliard
Aric Stewart <[EMAIL PROTECTED]> writes: > I like Michael's idea of initializing this lookup array on > initialization of the keyboard. Then we only have to worry about it if > the user switches keyboard layouts mid program and since last i looked > wine does not support that anyway we would b