March 2010 - Wine-devel - list.winehq.org

Change Journal and VSS analog in Wine
by Dmitry Konnov 03 Mar '10

03 Mar '10

Hi All, I'm doing backup software. Can somebody advise on weather Wine has functionality of ChangeJournal NTFS and Shadow Copy mechanism. I understand that ChangeJournal NTFS is NTFS file system feature and Shadow Copy perhaps heavily dependent on Windows system. But I wonder weather these features can be simulated on Mac under Wine/Crossover. Please advise. -- Faithfully yours, Dmitry Konnov mailto:konnov_d(a)mail.ru

2 1

Re: [PATCH 1/2] kernel32/tests: Add a test case for calling TlsFree on a freed index.
by Nikolay Sivov 02 Mar '10

02 Mar '10

On 3/2/2010 23:04, Reece Dunn wrote: > Hi, > > TlsFree should return FALSE if there wasn't a TLS slot of the > specified index allocated, as well as returning > ERROR_INVALID_PARAMETER. > > - Reece > > - ok(TlsFree(tlsIndex)!=0,"TlsFree failed\n"); > + ok(TlsFree(tlsIndex)!=0,"TlsFree failed: %08x\n", GetLastError()); > + /* Test freeing an already freed TLS index */ > + ok(TlsFree(tlsIndex)==0,"TlsFree succeeded\n"); > + ok(GetLastError()==ERROR_INVALID_PARAMETER, > + "GetLastError: expected ERROR_INVALID_PARAMETER, got %08x\n", GetLastError()); > You need reset last error to something before test.

1 0

Re: oleaut32: Add a VT_INT_PTR case for VARIANT_UserSize
by Alexandre Julliard 02 Mar '10

02 Mar '10

Nikolay Sivov <nsivov(a)codeweavers.com> writes: > @@ -239,6 +239,8 @@ static unsigned int get_type_size(ULONG *pFlags, VARTYPE vt) > case VT_DISPATCH: > case VT_RECORD: > return 0; > + case VT_INT_PTR: > + return sizeof(INT_PTR); This won't do the right thing on 64-bit. -- Alexandre Julliard julliard(a)winehq.org

3 6

Re: Improve Cygwin compatibility
by John Klehm 02 Mar '10

02 Mar '10

On Mon, Mar 1, 2010 at 2:39 PM, Alexandre Goujon <ale.goujon(a)gmail.com> wrote: > /* > * __ A bit off topic __ > * > * Replying "You need to give your real name in addition to the patch." > * > * I hope this time all's ok. > * You now know who I am but I think it shouldn't be mandatory. > * I'm not here to become famous. > * "For a better world" and 'cause I love what I do. > * > */ > > Whatever, this patch should (finally) makes ./configure script work as expected. > > My understanding is that the real names are required so Wine can legally prove where all it's code came. This way it's provable that no code is from someone that mightve been influenced by MS code. Regards, --John Klehm

1 0

Re: [PATCH 2/5] wined3d: refactoring - add a real gl vendor detection
by Henri Verbeet 02 Mar '10

02 Mar '10

On 1 March 2010 19:27, PAUL ANDREW PANON <ppanon(a)shaw.ca> wrote: > + gl_vendor = wined3d_guess_gl_vendor(gl_info, gl_string, gl_renderer); > card_vendor = wined3d_guess_card_vendor(gl_string, gl_renderer); > - TRACE_(d3d_caps)("found GL_VENDOR (%s)->(0x%04x)\n", debugstr_a(gl_string), card_vendor); > + TRACE_(d3d_caps)("found GL_VENDOR (%s)->(0x%04x/0x%04x)\n", debugstr_a(gl_string), gl_vendor, card_vendor); Sorry for not noticing this earlier, but you can't do that there. The Apple detection uses the list of supported extensions, but that list isn't initialized until further below. If there's no code that depends on the gl_vendor before, you can probably just move it down a bit to the same place that calls wined3d_guess_card(), possibly moving wined3d_guess_card_vendor() as well.

1 0

Re: [PATCH] ole32: fix composite moniker behaviour.
by Alexandre Julliard 02 Mar '10

02 Mar '10

Reece Dunn <msclrhd(a)googlemail.com> writes: > @@ -365,8 +365,7 @@ CompositeMonikerImpl_BindToObject(IMoniker* iface, IBindCtx* pbc, > /* if the requested class was loaded before ! we don't need to reload it */ > res = IRunningObjectTable_GetObject(prot,iface,(IUnknown**)ppvResult); > > - if (res==S_OK) > - return res; > + return res==S_OK ? S_OK : E_INVALIDARG; This doesn't look right. Just because the test wants E_INVALIDARG doesn't mean it's the correct error for all possible failures. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] msvcrt: Add more exports as stubs (alphabetic sorted) [try 2]
by Alexandre Julliard 02 Mar '10

02 Mar '10

Detlef Riekenberg <wine.dev(a)web.de> writes: > stubs, which are used in the cpp tests are now commented out In general the others should be commented out too, for the same reason. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] d3dxof: Turn some TRACEs into WARNs in case of parsing error
by Paul Vriens 02 Mar '10

02 Mar '10

On 03/02/2010 08:37 AM, Christian Costa wrote: > > --- > > dlls/d3dxof/d3dxof.c | 8 ++++---- I guess you only meant to sent this piece in this email? > dlls/wined3d/shader.c | 22 +++++++++++++++++++++- > 2 files changed, 25 insertions(+), 5 deletions(-) -- Cheers, Paul.

2 1

Re: [PATCH] wined3d: Display correctly TEXT comments by printing lines separately
by Henri Verbeet 02 Mar '10

02 Mar '10

On 1 March 2010 09:39, Christian Costa <titan.costa(a)wanadoo.fr> wrote: > Still not quite right, I'm afraid. For example, the following doesn't work: > + while (i < size) > + { > + /* Find end of line */ > + while ((str[i] != 0) && (str[i] != 0x0a)) > + i++; How does something like the attached patch work (untested)?

2 3

Re: shdocvw: return success code in ITaskBarList::HrInit
by Henri Verbeet 01 Mar '10

01 Mar '10

On 1 March 2010 21:20, Andrey Turkin <andrey.turkin(a)gmail.com> wrote: > - FIXME("iface %p stub!\n", iface); > - > - return E_NOTIMPL; > + return S_OK; I think you'll want to at least keep a TRACE.

1 0