Re: dsound: Runtime link to advapi32 RegDeleteTree

2007-09-03 Thread Reece Dunn
, what other windows is there? Wine dsound works fine under windows. I was wondering if you meant Wine running as Windows XP. Thanks for the clarification. > >> It is the least of all evils as far as I could see. It's either > >> this or adding RegDeleteTree to libwine, unless

Re: dsound: Runtime link to advapi32 RegDeleteTree

2007-09-03 Thread Maarten Lankhorst
e dsound dll work in windows xp and older versions of windows >> again. >> > By this, do you mean running the Wine dsound.dll on _native_ Windows? > Yes, what other windows is there? Wine dsound works fine under windows. >> It is the least of all evils as far as

Re: dsound: Runtime link to advapi32 RegDeleteTree

2007-09-03 Thread Reece Dunn
indows > again. By this, do you mean running the Wine dsound.dll on _native_ Windows? > It is the least of all evils as far as I could see. It's either > this or adding RegDeleteTree to libwine, unless someone has a third option. If the intent is to get this working on native, add

Re: RegDeleteTree and older versions of windows?

2007-08-31 Thread Alexandre Julliard
Maarten Lankhorst <[EMAIL PROTECTED]> writes: > I'm open for suggestions though, since I really can't come up with any > good ways to make sure wine dlls will keep working for older versions of > windows. I don't think we want to add more infrastructure for that regsvr.c stuff. It should really b

RegDeleteTree and older versions of windows?

2007-08-31 Thread Maarten Lankhorst
Now that most functions get replaced with RegDeleteTree, I'm wondering if it is possible to link against that function statically for mingw/msvc dlls, so wine dlls can be still be used on win9x and windows nt and win2k. The problem is I don't know what the best solution would be to mak

Re: RegDeleteTree [10]

2007-05-25 Thread Stefan Leichter
Am Friday 25 May 2007 18:41 schrieb James Hawkins: > On 5/25/07, Stefan Leichter <[EMAIL PROTECTED]> wrote: > > Am Friday 25 May 2007 00:48 schrieb Juan Lang: > > > Hi Stefan, > > > > > > +/* Find how many subkeys there are */ > > > > > > This comment is incorrect. > > > > > > You have also rei

Re: RegDeleteTree [10]

2007-05-25 Thread James Hawkins
On 5/25/07, Stefan Leichter <[EMAIL PROTECTED]> wrote: Am Friday 25 May 2007 00:48 schrieb Juan Lang: > Hi Stefan, > > +/* Find how many subkeys there are */ > > This comment is incorrect. > > You have also reintroduced tabs in this patch - please don't do that. > --Juan ChangeLog ---

Re: RegDeleteTree [9]

2007-05-24 Thread Juan Lang
Hi Stefan, +/* Find how many subkeys there are */ This comment is incorrect. You have also reintroduced tabs in this patch - please don't do that. --Juan Pinpoint customers who are looking for what

Re: RegDeleteTree [8]

2007-05-23 Thread Alexandre Julliard
Stefan Leichter <[EMAIL PROTECTED]> writes: > +/* Recursively delete all the subkeys */ > +for (i = 0; i < dwKeyCount; i++) { > +dwSize = dwMaxLen; > +ret = RegEnumKeyExW(hSubKey, 0, lpszName, &dwSize, NULL, > + NULL, NULL, NULL); There's no reason to u

Re: RegDeleteTree [7] [resend]

2007-05-22 Thread Laurent Vromman
Stefan Leichter a écrit : Am Tuesday 22 May 2007 18:05 schrieb Laurent Vromman: There is a few tabs in the file : Here (line 92), first blank character : + &dwValCount, NULL, NULL, NULL, NULL); Here (line 103) + } Laurent Hello Laurent, thank you for t

Re: RegDeleteTree [7] [resend]

2007-05-22 Thread Stefan Leichter
Am Tuesday 22 May 2007 18:05 schrieb Laurent Vromman: > There is a few tabs in the file : > > Here (line 92), first blank character : > + &dwValCount, NULL, NULL, NULL, NULL); > > Here (line 103) > + } > > Laurent > Hello Laurent, thank you for the comment. How did you sp

Re: RegDeleteTree [7] [resend]

2007-05-22 Thread Laurent Vromman
There is a few tabs in the file : Here (line 92), first blank character : + &dwValCount, NULL, NULL, NULL, NULL); Here (line 103) + } Laurent > Hello, > > any comment on the current version of this patch? > > ChangeLog > - > added implementation

Re: RegDeleteTree [6]

2007-05-18 Thread Juan Lang
Hi Stefan, +if (!lpszName) { +ret = ERROR_NOT_ENOUGH_MEMORY; + goto cleanup; +} Now you're mixing tabs and spaces (here and elsewhere.) Please honor the existing formatting. --Juan ___

Re: RegDeleteTree [5]

2007-05-18 Thread Juan Lang
Hi Stefan, +if (ret) return ret; + +ret = RegDeleteTreeW(hSubKey, lpszName); +if (ret) return ret; you've followed James's advice, but now you leak hSubKey (when it isn't hKey) and lpszName (when it isn't szNameBuf.) A goto would be better. --Juan _

Re: RegDeleteTree [4th] [resend]

2007-05-17 Thread James Hawkins
On 5/17/07, Stefan Leichter <[EMAIL PROTECTED]> wrote: Hello, is anything wrong with this patch? +#if (_WIN32_WINNT >= 0x0600) +LONGWINAPI RegDeleteTreeA(HKEY,LPCSTR); +LONGWINAPI RegDeleteTreeW(HKEY,LPCWSTR); +#define RegDeleteTree WINELIB_NAME_AW(RegDeleteTree

Re: RegDeleteTree [3rd]

2007-04-25 Thread Stefan Leichter
Am Wednesday 25 April 2007 23:00 schrieb Robert Shearman: > Stefan Leichter wrote: > > Am Thursday 19 April 2007 12:14 schrieb Alexandre Julliard: > >> Stefan Leichter <[EMAIL PROTECTED]> writes: > >>> + } else { > >>> +if (!ret) > >>> + ret = RegSetValueW(hSubKey, NULL, REG_SZ,

Re: RegDeleteTree [3rd]

2007-04-25 Thread Robert Shearman
Stefan Leichter wrote: Am Thursday 19 April 2007 12:14 schrieb Alexandre Julliard: Stefan Leichter <[EMAIL PROTECTED]> writes: + } else { +if (!ret) + ret = RegSetValueW(hSubKey, NULL, REG_SZ, emptyW, 0); The function is supposed to delete the ke

Re: RegDeleteTree [3rd]

2007-04-22 Thread Paul Vriens
Stefan Leichter wrote: Am Saturday 21 April 2007 20:13 schrieb Paul Vriens: Hi Stefan the results: registry.c:576:test_query_value_ex: type set to: 0x0022fe78 registry.c:1163: Test failed: Expected ERROR_SUCCESS, got 6 registry: 293 tests executed (0 marked as todo, 1 failure), 0 skipped. Tha

Re: RegDeleteTree [3rd]

2007-04-21 Thread Stefan Leichter
Am Saturday 21 April 2007 20:13 schrieb Paul Vriens: >Hi Stefan > >the results: > >registry.c:576:test_query_value_ex: type set to: 0x0022fe78 >registry.c:1163: Test failed: Expected ERROR_SUCCESS, got 6 >registry: 293 tests executed (0 marked as todo, 1 failure), 0 skipped. > >That's because you'r

Re: RegDeleteTree [3rd]

2007-04-21 Thread Paul Vriens
Stefan Leichter wrote: Am Thursday 19 April 2007 12:14 schrieb Alexandre Julliard: Stefan Leichter <[EMAIL PROTECTED]> writes: +/* Recursively delete all the subkeys */ +for (i = 0; i < dwKeyCount && !ret; i++) { +dwSize = dwMaxSubkeyLen; +

Re: RegDeleteTree [3rd]

2007-04-20 Thread Stefan Leichter
Am Friday 20 April 2007 05:54 schrieb Tom Spear: > I actually have a Vista Home Basic install on a laptop that I can > check testcases on, at least for right now. I'm thinking of going out > and getting a licensed copy of xp for it (linux isn't viable for it > because it isnt my laptop and its bei

Re: RegDeleteTree [3rd]

2007-04-20 Thread Paul Vriens
On 4/19/07, Stefan Leichter <[EMAIL PROTECTED]> wrote: Am Thursday 19 April 2007 12:14 schrieb Alexandre Julliard: > Stefan Leichter <[EMAIL PROTECTED]> writes: > > +/* Recursively delete all the subkeys */ > > +for (i = 0; i < dwKeyCount && !ret; i++) { > > +

Re: RegDeleteTree [3rd]

2007-04-19 Thread Tom Spear
I actually have a Vista Home Basic install on a laptop that I can check testcases on, at least for right now. I'm thinking of going out and getting a licensed copy of xp for it (linux isn't viable for it because it isnt my laptop and its being used as the main pc for business transactions, so it

Re: RegDeleteTree [3rd]

2007-04-19 Thread Stefan Leichter
Am Thursday 19 April 2007 12:14 schrieb Alexandre Julliard: > Stefan Leichter <[EMAIL PROTECTED]> writes: > > +/* Recursively delete all the subkeys */ > > +for (i = 0; i < dwKeyCount && !ret; i++) { > > +dwSize = dwMaxSubkeyLen; > > +

Re: RegDeleteTree [3rd]

2007-04-19 Thread Alexandre Julliard
Stefan Leichter <[EMAIL PROTECTED]> writes: > +/* Recursively delete all the subkeys */ > +for (i = 0; i < dwKeyCount && !ret; i++) { > +dwSize = dwMaxSubkeyLen; > +ret = RegEnumKeyExW(hSubKey, i, lpszName, &dwSize, NULL, > +

Re: RegDeleteTree

2007-04-17 Thread Stefan Leichter
Paul Vriens schrieb: Stefan Leichter wrote: ChangeLog -- added implementation of RegDeleteTreeA/W --- ../wine-git/dlls/advapi32/advapi32.spec2006-10-03 10:52:50.0 +0200 +++ dlls/advapi32/adv

Re: RegDeleteTree

2007-04-16 Thread Paul Vriens
Stefan Leichter wrote: ChangeLog -- added implementation of RegDeleteTreeA/W --- ../wine-git/dlls/advapi32/advapi32.spec 2006-10-03 10:52:50.0 +0200 +++ dlls/advapi32/advapi32.spec 2007-04

Re: RegDeleteTree

2007-04-15 Thread Detlef Riekenberg
On So, 2007-04-15 at 10:02 +0200, Stefan Leichter wrote: I did not test the code, but ... > +LONG WINAPI RegDeleteTreeW(HKEY hKey, LPCWSTR lpszSubKey) > +{ > +if (!ret) > + ret = RegSetValueA(hSubKey, NULL, REG_SZ, "", 0); Please do not call an ANSI-Function in an UNI

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-10-11 Thread Paul Vriens
p a little > > --Brad DeMorrow > > > On 9/28/06, Frank Richter <[EMAIL PROTECTED]> wrote: > On 28.09.2006 15:26, Paul Vriens wrote: > > I'll have a few hours tomorrow and will have a look. The > cleanest > > solution f

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-10-10 Thread Vitaliy Margolen
303.html >>> ([ros-svn] [weiden] 17572: implemented RegDeleteTree() (untested!)) >>> mentions this, and nothing more: >>> --- CUT --- >>> implemented RegDeleteTree() (untested!) >>> >>> Updated files: >>> trunk/reactos/lib/advapi32/reg/reg.c >

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-10-10 Thread Segin
James Hawkins wrote: > On 10/10/06, Segin <[EMAIL PROTECTED]> wrote: >> > >> Some Google searching provides that there may or may not be an >> implementation. >> >> http://www.reactos.org/pipermail/ros-svn/2005-August/005303.html >> ([ros-svn] [weid

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-10-10 Thread James Hawkins
On 10/10/06, Segin <[EMAIL PROTECTED]> wrote: > Some Google searching provides that there may or may not be an implementation. http://www.reactos.org/pipermail/ros-svn/2005-August/005303.html ([ros-svn] [weiden] 17572: implemented RegDeleteTree() (untested!)) mentions this, and not

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-10-10 Thread Segin
Morrow > > > On 9/28/06, Frank Richter <[EMAIL PROTECTED]> wrote: >> >> On 28.09.2006 15:26, Paul Vriens wrote: >> > I'll have a few hours tomorrow and will have a look. The cleanest >> > solution for now seems to create RegDeleteTree[A|W] in advapi

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-10-10 Thread Brad DeMorrow
.09.2006 15:26, Paul Vriens wrote:> I'll have a few hours tomorrow and will have a look. The cleanest> solution for now seems to create RegDeleteTree[A|W] in advapi32 and use> that wherever possible (unless we find that it forwards to shlwapi). > Most DLL's already import adva

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-09-28 Thread Paul Vriens
inecfg. > > Vitaliy. > Do you mean winecfg has it's implementation, or winecfg has the solution? I had another look, apparently shlwapi has SHDeleteKey[A|W]. It does the same thing. I'll have a few hours tomorrow and will have a look. The cleanest solution for now seems to crea

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-09-28 Thread Frank Richter
On 28.09.2006 15:26, Paul Vriens wrote: > I'll have a few hours tomorrow and will have a look. The cleanest > solution for now seems to create RegDeleteTree[A|W] in advapi32 and use > that wherever possible (unless we find that it forwards to shlwapi). > Most DLL's already

Re: Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-09-28 Thread Vitaliy Margolen
Paul Vriens wrote: > Hi, > > I was browsing some code and found that we have several places where we > made our own implementation of a recursive delete of registry > keys/value. See winecfg. Vitaliy.

Anybody have a RegDeleteTree[A|W] implementation lying around?

2006-09-28 Thread Paul Vriens
Hi, I was browsing some code and found that we have several places where we made our own implementation of a recursive delete of registry keys/value. Apparently there is an official one in Vista: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/sysinfo/base/regopenkeyex.asp I do