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

Juan Lang : crypt32: Use an empty string as a separator when no separator is desired to avoid special cases for NULL .
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: 7eb61a6e5cced2edf379ce1cddf6a6bf50f0c97e URL: http://source.winehq.org/git/wine.git/?a=commit;h=7eb61a6e5cced2edf379ce1cd… Author: Juan Lang <juan.lang(a)gmail.com> Date: Wed Mar 24 11:51:23 2010 -0700 crypt32: Use an empty string as a separator when no separator is desired to avoid special cases for NULL. --- dlls/crypt32/base64.c | 50 ++++++++++++++++-------------------------------- 1 files changed, 17 insertions(+), 33 deletions(-) diff --git a/dlls/crypt32/base64.c b/dlls/crypt32/base64.c index 2a91372..2996ba3 100644 --- a/dlls/crypt32/base64.c +++ b/dlls/crypt32/base64.c @@ -99,8 +99,7 @@ static LONG encodeBase64A(const BYTE *in_buf, int in_len, LPCSTR sep, TRACE("bytes is %d, pad bytes is %d\n", bytes, pad_bytes); needed = bytes + pad_bytes + 1; - if (sep) - needed += (needed / 64 + 1) * strlen(sep); + needed += (needed / 64 + 1) * strlen(sep); if (needed > *out_len) { @@ -117,7 +116,7 @@ static LONG encodeBase64A(const BYTE *in_buf, int in_len, LPCSTR sep, i = 0; while (div > 0) { - if (sep && i && i % 64 == 0) + if (i && i % 64 == 0) { strcpy(ptr, sep); ptr += strlen(sep); @@ -163,8 +162,7 @@ static LONG encodeBase64A(const BYTE *in_buf, int in_len, LPCSTR sep, *ptr++ = '='; break; } - if (sep) - strcpy(ptr, sep); + strcpy(ptr, sep); return ERROR_SUCCESS; } @@ -180,7 +178,7 @@ static BOOL BinaryToBase64A(const BYTE *pbBinary, if (dwFlags & CRYPT_STRING_NOCR) sep = lf; else if (dwFlags & CRYPT_STRING_NOCRLF) - sep = NULL; + sep = ""; else sep = crlf; switch (dwFlags & 0x0fffffff) @@ -217,11 +215,8 @@ static BOOL BinaryToBase64A(const BYTE *pbBinary, { strcpy(ptr, header); ptr += strlen(ptr); - if (sep) - { - strcpy(ptr, sep); - ptr += strlen(sep); - } + strcpy(ptr, sep); + ptr += strlen(sep); } encodeBase64A(pbBinary, cbBinary, sep, ptr, &size); ptr += size - 1; @@ -229,11 +224,8 @@ static BOOL BinaryToBase64A(const BYTE *pbBinary, { strcpy(ptr, trailer); ptr += strlen(ptr); - if (sep) - { - strcpy(ptr, sep); - ptr += strlen(sep); - } + strcpy(ptr, sep); + ptr += strlen(sep); } *pcchString = charsNeeded - 1; } @@ -302,8 +294,7 @@ static LONG encodeBase64W(const BYTE *in_buf, int in_len, LPCWSTR sep, TRACE("bytes is %d, pad bytes is %d\n", bytes, pad_bytes); needed = bytes + pad_bytes + 1; - if (sep) - needed += (needed / 64 + 1) * strlenW(sep); + needed += (needed / 64 + 1) * strlenW(sep); if (needed > *out_len) { @@ -320,7 +311,7 @@ static LONG encodeBase64W(const BYTE *in_buf, int in_len, LPCWSTR sep, i = 0; while (div > 0) { - if (sep && i && i % 64 == 0) + if (i && i % 64 == 0) { strcpyW(ptr, sep); ptr += strlenW(sep); @@ -366,8 +357,7 @@ static LONG encodeBase64W(const BYTE *in_buf, int in_len, LPCWSTR sep, *ptr++ = '='; break; } - if (sep) - strcpyW(ptr, sep); + strcpyW(ptr, sep); return ERROR_SUCCESS; } @@ -375,7 +365,7 @@ static LONG encodeBase64W(const BYTE *in_buf, int in_len, LPCWSTR sep, static BOOL BinaryToBase64W(const BYTE *pbBinary, DWORD cbBinary, DWORD dwFlags, LPWSTR pszString, DWORD *pcchString) { - static const WCHAR crlf[] = { '\r','\n',0 }, lf[] = { '\n',0 }; + static const WCHAR crlf[] = { '\r','\n',0 }, lf[] = { '\n',0 }, empty = {0}; BOOL ret = TRUE; LPCWSTR header = NULL, trailer = NULL, sep; DWORD charsNeeded; @@ -383,7 +373,7 @@ static BOOL BinaryToBase64W(const BYTE *pbBinary, if (dwFlags & CRYPT_STRING_NOCR) sep = lf; else if (dwFlags & CRYPT_STRING_NOCRLF) - sep = NULL; + sep = empty; else sep = crlf; switch (dwFlags & 0x0fffffff) @@ -420,11 +410,8 @@ static BOOL BinaryToBase64W(const BYTE *pbBinary, { strcpyW(ptr, header); ptr += strlenW(ptr); - if (sep) - { - strcpyW(ptr, sep); - ptr += strlenW(sep); - } + strcpyW(ptr, sep); + ptr += strlenW(sep); } encodeBase64W(pbBinary, cbBinary, sep, ptr, &size); ptr += size - 1; @@ -432,11 +419,8 @@ static BOOL BinaryToBase64W(const BYTE *pbBinary, { strcpyW(ptr, trailer); ptr += strlenW(ptr); - if (sep) - { - strcpyW(ptr, sep); - ptr += strlenW(sep); - } + strcpyW(ptr, sep); + ptr += strlenW(sep); } *pcchString = charsNeeded - 1; }

1 0

Jacek Caban : mshtml: Added IHTMLElement:removeAttribute tests.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: 621694efcdb9966bfb1880f379f120f79fb9bd59 URL: http://source.winehq.org/git/wine.git/?a=commit;h=621694efcdb9966bfb1880f37… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 24 20:01:19 2010 +0100 mshtml: Added IHTMLElement:removeAttribute tests. --- dlls/mshtml/tests/jstest.html | 18 ++++++++++++++++++ 1 files changed, 18 insertions(+), 0 deletions(-) diff --git a/dlls/mshtml/tests/jstest.html b/dlls/mshtml/tests/jstest.html index 72ef427..6192f81 100644 --- a/dlls/mshtml/tests/jstest.html +++ b/dlls/mshtml/tests/jstest.html @@ -5,12 +5,30 @@ function ok(b,m) { return external.ok(b, m); } +function test_removeAttribute(e) { + ok(e.removeAttribute('nonexisting') === false, "removeAttribute('nonexisting') didn't return false"); + + e.title = "title"; + ok(e.removeAttribute('title') === true, "removeAttribute('title') didn't return true"); + ok(e.title === "", "e.title = " + e.title); + ok(("title" in e) === true, "title is not in e"); + + e["myattr"] = "test"; + ok(e.removeAttribute('myattr') === true, "removeAttribute('myattr') didn't return true"); + ok(e["myattr"] === undefined, "e['myattr'] = " + e['myattr']); + ok(("myattr" in e) === false, "myattr is in e"); + +} + function runTest() { obj = new Object(); ok(obj === window.obj, "obj !== window.obj"); ok(typeof(divid) === "object", "typeof(divid) = " + typeof(divid)); + test_removeAttribute(document.getElementById("divid")); + test_removeAttribute(document.body); + external.reportSuccess(); } </script>

1 0

Jörg Höhle : include: Add some multimedia defines not present in win9x.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: b33152573f2b466a0b72c22911c8fbfb4fc462f7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b33152573f2b466a0b72c2291… Author: Jörg Höhle <hoehle(a)users.sourceforge.net> Date: Tue Mar 23 21:52:28 2010 +0100 include: Add some multimedia defines not present in win9x. --- include/mmddk.h | 3 +++ include/mmsystem.h | 2 ++ 2 files changed, 5 insertions(+), 0 deletions(-) diff --git a/include/mmddk.h b/include/mmddk.h index 826aefb..d3593b0 100644 --- a/include/mmddk.h +++ b/include/mmddk.h @@ -376,6 +376,9 @@ typedef JOYDEVMSGPROC *LPJOYDEVMSGPROC; #define MCI_RECT 7 #define MCI_CONSTANT 8 #define MCI_END_CONSTANT 9 +#define MCI_HWND 10 +#define MCI_HPAL 11 +#define MCI_HDC 12 #define MAKEMCIRESOURCE(wRet, wRes) MAKELRESULT((wRet), (wRes)) diff --git a/include/mmsystem.h b/include/mmsystem.h index d147bf3..4631762 100644 --- a/include/mmsystem.h +++ b/include/mmsystem.h @@ -657,6 +657,8 @@ DECL_WINELIB_TYPE_AW(LPMIDIOUTCAPS2) #define MOD_SQSYNTH 3 /* square wave internal synth */ #define MOD_FMSYNTH 4 /* FM internal synth */ #define MOD_MAPPER 5 /* MIDI mapper */ +#define MOD_WAVETABLE 6 /* hardware watetable internal synth */ +#define MOD_SWSYNTH 7 /* software synth */ #define MIDICAPS_VOLUME 0x0001 /* supports volume control */ #define MIDICAPS_LRVOLUME 0x0002 /* separate left-right volume control */

1 0

Austin English : cmd: Add testcases for goto.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: 27717a51278e0cb5e90d590206404c1b36b0309f URL: http://source.winehq.org/git/wine.git/?a=commit;h=27717a51278e0cb5e90d59020… Author: Austin English <austinenglish(a)gmail.com> Date: Tue Mar 23 22:45:49 2010 -0500 cmd: Add testcases for goto. --- programs/cmd/tests/test_builtins.cmd | 11 +++++++++++ programs/cmd/tests/test_builtins.cmd.exp | 4 ++++ 2 files changed, 15 insertions(+), 0 deletions(-) diff --git a/programs/cmd/tests/test_builtins.cmd b/programs/cmd/tests/test_builtins.cmd index bae19b8..d596c18 100644 --- a/programs/cmd/tests/test_builtins.cmd +++ b/programs/cmd/tests/test_builtins.cmd @@ -59,3 +59,14 @@ if /i foo==FOO echo if /i seems to work if /i not foo==FOO echo if /i seems to be broken if /I foo==FOO echo if /I seems to work if /I not foo==FOO echo if /I seems to be broken + +echo -----------Testing GOTO----------- +if a==a goto dest1 +:dest1 +echo goto with no leading space worked +if b==b goto dest2 + :dest2 +echo goto with a leading space worked +if c==c goto dest3 + :dest3 +echo goto with a leading tab worked diff --git a/programs/cmd/tests/test_builtins.cmd.exp b/programs/cmd/tests/test_builtins.cmd.exp index ad50358..306953b 100644 --- a/programs/cmd/tests/test_builtins.cmd.exp +++ b/programs/cmd/tests/test_builtins.cmd.exp @@ -40,3 +40,7 @@ Testing case sensitivity with and without /i option if seems to default to case sensitivity if /i seems to work if /I seems to work +-----------Testing GOTO----------- +goto with no leading space worked +goto with a leading space worked +goto with a leading tab worked

1 0

Vincent Povirk : wine.inf: Add some .NET Framework keys.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: 929bac964aaac575740f042bdb606412e8f88dbe URL: http://source.winehq.org/git/wine.git/?a=commit;h=929bac964aaac575740f042bd… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Wed Mar 24 09:12:31 2010 -0500 wine.inf: Add some .NET Framework keys. --- tools/wine.inf.in | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/tools/wine.inf.in b/tools/wine.inf.in index 7476df7..66b966f 100644 --- a/tools/wine.inf.in +++ b/tools/wine.inf.in @@ -37,6 +37,7 @@ AddReg=\ CurrentVersion,\ Debugger,\ Desktop,\ + Dotnet,\ Metrics,\ DirectX,\ Fonts,\ @@ -61,6 +62,7 @@ AddReg=\ CurrentVersion,\ Debugger,\ Desktop,\ + Dotnet,\ Metrics,\ DirectX,\ Fonts,\ @@ -86,6 +88,7 @@ AddReg=\ CurrentVersion,\ Debugger,\ Desktop,\ + Dotnet,\ Metrics,\ DirectX,\ Fonts,\ @@ -463,6 +466,11 @@ HKCU,%Desktop%,"UserPreferenceMask",3,10,00,00,80 HKCU,%Desktop%,"LowPowerActive",,"0" HKCU,%Desktop%,"MenuShowDelay",2,"400" +[DotNet] +HKLM,Software\Microsoft\.NETFramework\policy\v1.0 +HKLM,Software\Microsoft\.NETFramework\policy\v2.0 +HKLM,Software\Microsoft\.NETFramework\policy\v3.0 + [Metrics] HKCU,%Metrics%,"ScrollWidth",2,"16" HKCU,%Metrics%,"ScrollHeight",2,"16"

1 0

Jacek Caban : mshtml: Moved builtin properties lookup to separate function.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: 1b093221e615065a53ea7652b3ce9d758372e53a URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b093221e615065a53ea7652b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 24 19:59:38 2010 +0100 mshtml: Moved builtin properties lookup to separate function. --- dlls/mshtml/dispex.c | 78 ++++++++++++++++++++++++++++---------------------- 1 files changed, 44 insertions(+), 34 deletions(-) diff --git a/dlls/mshtml/dispex.c b/dlls/mshtml/dispex.c index 4b1bd88..592d999 100644 --- a/dlls/mshtml/dispex.c +++ b/dlls/mshtml/dispex.c @@ -729,6 +729,47 @@ static HRESULT get_builtin_func(dispex_data_t *data, DISPID id, func_info_t **re return DISP_E_UNKNOWNNAME; } +static HRESULT get_builtin_id(DispatchEx *This, BSTR name, DWORD grfdex, DISPID *ret) +{ + dispex_data_t *data; + int min, max, n, c; + + data = get_dispex_data(This); + if(!data) + return E_FAIL; + + min = 0; + max = data->func_cnt-1; + + while(min <= max) { + n = (min+max)/2; + + c = strcmpiW(data->name_table[n]->name, name); + if(!c) { + if((grfdex & fdexNameCaseSensitive) && strcmpW(data->name_table[n]->name, name)) + break; + + *ret = data->name_table[n]->id; + return S_OK; + } + + if(c > 0) + max = n-1; + else + min = n+1; + } + + if(This->data->vtbl && This->data->vtbl->get_dispid) { + HRESULT hres; + + hres = This->data->vtbl->get_dispid(This->outer, name, grfdex, ret); + if(hres != DISP_E_UNKNOWNNAME) + return hres; + } + + return DISP_E_UNKNOWNNAME; +} + #define DISPATCHEX_THIS(iface) DEFINE_THIS(DispatchEx, IDispatchEx, iface) static HRESULT WINAPI DispatchEx_QueryInterface(IDispatchEx *iface, REFIID riid, void **ppv) @@ -815,8 +856,6 @@ static HRESULT WINAPI DispatchEx_GetDispID(IDispatchEx *iface, BSTR bstrName, DW { DispatchEx *This = DISPATCHEX_THIS(iface); dynamic_prop_t *dprop; - dispex_data_t *data; - int min, max, n, c; HRESULT hres; TRACE("(%p)->(%s %x %p)\n", This, debugstr_w(bstrName), grfdex, pid); @@ -824,38 +863,9 @@ static HRESULT WINAPI DispatchEx_GetDispID(IDispatchEx *iface, BSTR bstrName, DW if(grfdex & ~(fdexNameCaseSensitive|fdexNameCaseInsensitive|fdexNameEnsure|fdexNameImplicit|FDEX_VERSION_MASK)) FIXME("Unsupported grfdex %x\n", grfdex); - data = get_dispex_data(This); - if(!data) - return E_FAIL; - - min = 0; - max = data->func_cnt-1; - - while(min <= max) { - n = (min+max)/2; - - c = strcmpiW(data->name_table[n]->name, bstrName); - if(!c) { - if((grfdex & fdexNameCaseSensitive) && strcmpW(data->name_table[n]->name, bstrName)) - break; - - *pid = data->name_table[n]->id; - return S_OK; - } - - if(c > 0) - max = n-1; - else - min = n+1; - } - - if(This->data->vtbl && This->data->vtbl->get_dispid) { - HRESULT hres; - - hres = This->data->vtbl->get_dispid(This->outer, bstrName, grfdex, pid); - if(hres != DISP_E_UNKNOWNNAME) - return hres; - } + hres = get_builtin_id(This, bstrName, grfdex, pid); + if(hres != DISP_E_UNKNOWNNAME) + return hres; hres = get_dynamic_prop(This, bstrName, grfdex, &dprop); if(FAILED(hres))

1 0

Jacek Caban : mshtml: Moved builtin property invocation to separate function.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: 30bed7eab3a352e664b70972613f3e2d21eed2c5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=30bed7eab3a352e664b709726… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 24 20:00:08 2010 +0100 mshtml: Moved builtin property invocation to separate function. --- dlls/mshtml/dispex.c | 44 ++++++++++++++++++++++++++------------------ 1 files changed, 26 insertions(+), 18 deletions(-) diff --git a/dlls/mshtml/dispex.c b/dlls/mshtml/dispex.c index 592d999..c5e6de2 100644 --- a/dlls/mshtml/dispex.c +++ b/dlls/mshtml/dispex.c @@ -770,6 +770,31 @@ static HRESULT get_builtin_id(DispatchEx *This, BSTR name, DWORD grfdex, DISPID return DISP_E_UNKNOWNNAME; } +static HRESULT invoke_builtin_prop(DispatchEx *This, DISPID id, LCID lcid, WORD flags, DISPPARAMS *dp, + VARIANT *res, EXCEPINFO *ei, IServiceProvider *caller) +{ + dispex_data_t *data; + func_info_t *func; + HRESULT hres; + + data = get_dispex_data(This); + if(!data) + return E_FAIL; + + hres = get_builtin_func(data, id, &func); + if(id == DISPID_VALUE && hres == DISP_E_UNKNOWNNAME) + return dispex_value(This, lcid, flags, dp, res, ei, caller); + if(FAILED(hres)) + return hres; + + if(func->func_disp_idx == -1) + hres = typeinfo_invoke(This, func, flags, dp, res, ei); + else + hres = function_invoke(This, func, flags, dp, res, ei); + + return hres; +} + #define DISPATCHEX_THIS(iface) DEFINE_THIS(DispatchEx, IDispatchEx, iface) static HRESULT WINAPI DispatchEx_QueryInterface(IDispatchEx *iface, REFIID riid, void **ppv) @@ -879,8 +904,6 @@ static HRESULT WINAPI DispatchEx_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc VARIANT *pvarRes, EXCEPINFO *pei, IServiceProvider *pspCaller) { DispatchEx *This = DISPATCHEX_THIS(iface); - dispex_data_t *data; - func_info_t *func; HRESULT hres; TRACE("(%p)->(%x %x %x %p %p %p %p)\n", This, id, lcid, wFlags, pdp, pvarRes, pei, pspCaller); @@ -972,22 +995,7 @@ static HRESULT WINAPI DispatchEx_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc } } - data = get_dispex_data(This); - if(!data) - return E_FAIL; - - hres = get_builtin_func(data, id, &func); - if(id == DISPID_VALUE && hres == DISP_E_UNKNOWNNAME) - return dispex_value(This, lcid, wFlags, pdp, pvarRes, pei, pspCaller); - if(FAILED(hres)) - return hres; - - if(func->func_disp_idx == -1) - hres = typeinfo_invoke(This, func, wFlags, pdp, pvarRes, pei); - else - hres = function_invoke(This, func, wFlags, pdp, pvarRes, pei); - - return hres; + return invoke_builtin_prop(This, id, lcid, wFlags, pdp, pvarRes, pei, pspCaller); } static HRESULT WINAPI DispatchEx_DeleteMemberByName(IDispatchEx *iface, BSTR bstrName, DWORD grfdex)

1 0

Jacek Caban : mshtml: Added IHTMLElement::removeAttribute implementation.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: 00293e9b0bcee3768fe8a142bd1b73ae9bb92c52 URL: http://source.winehq.org/git/wine.git/?a=commit;h=00293e9b0bcee3768fe8a142b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Mar 24 20:34:17 2010 +0100 mshtml: Added IHTMLElement::removeAttribute implementation. --- dlls/mshtml/dispex.c | 96 ++++++++++++++++++++++++++++++++++-------- dlls/mshtml/htmlelem.c | 6 ++- dlls/mshtml/mshtml_private.h | 1 + 3 files changed, 83 insertions(+), 20 deletions(-) diff --git a/dlls/mshtml/dispex.c b/dlls/mshtml/dispex.c index c5e6de2..3917b30 100644 --- a/dlls/mshtml/dispex.c +++ b/dlls/mshtml/dispex.c @@ -52,8 +52,11 @@ struct dispex_data_t { typedef struct { VARIANT var; LPWSTR name; + DWORD flags; } dynamic_prop_t; +#define DYNPROP_DELETED 0x01 + typedef struct { DispatchEx dispex; const IUnknownVtbl *lpIUnknownVtbl; @@ -442,7 +445,7 @@ static HRESULT get_dynamic_prop(DispatchEx *This, const WCHAR *name, DWORD flags { const BOOL alloc = flags & fdexNameEnsure; dispex_dynamic_data_t *data; - unsigned i; + dynamic_prop_t *prop; data = get_dynamic_data(This, alloc); if(!data) { @@ -453,9 +456,14 @@ static HRESULT get_dynamic_prop(DispatchEx *This, const WCHAR *name, DWORD flags return DISP_E_UNKNOWNNAME; } - for(i=0; i < data->prop_cnt; i++) { - if(flags & fdexNameCaseInsensitive ? !strcmpiW(data->props[i].name, name) : !strcmpW(data->props[i].name, name)) { - *ret = data->props+i; + for(prop = data->props; prop < data->props+data->prop_cnt; prop++) { + if(flags & fdexNameCaseInsensitive ? !strcmpiW(prop->name, name) : !strcmpW(prop->name, name)) { + if(prop->flags & DYNPROP_DELETED) { + if(!alloc) + return DISP_E_UNKNOWNNAME; + prop->flags &= ~DYNPROP_DELETED; + } + *ret = prop; return S_OK; } } @@ -481,10 +489,16 @@ static HRESULT get_dynamic_prop(DispatchEx *This, const WCHAR *name, DWORD flags data->buf_size <<= 1; } - data->props[data->prop_cnt].name = heap_strdupW(name); - VariantInit(&data->props[data->prop_cnt].var); - *ret = data->props + data->prop_cnt++; + prop = data->props + data->prop_cnt; + prop->name = heap_strdupW(name); + if(!prop->name) + return E_OUTOFMEMORY; + + VariantInit(&prop->var); + prop->flags = 0; + data->prop_cnt++; + *ret = prop; return S_OK; } @@ -795,6 +809,43 @@ static HRESULT invoke_builtin_prop(DispatchEx *This, DISPID id, LCID lcid, WORD return hres; } +HRESULT remove_prop(DispatchEx *This, BSTR name, VARIANT_BOOL *success) +{ + dynamic_prop_t *prop; + DISPID id; + HRESULT hres; + + hres = get_builtin_id(This, name, 0, &id); + if(hres == S_OK) { + DISPID named_id = DISPID_PROPERTYPUT; + VARIANT var; + DISPPARAMS dp = {&var,&named_id,1,1}; + EXCEPINFO ei; + + V_VT(&var) = VT_EMPTY; + memset(&ei, 0, sizeof(ei)); + hres = invoke_builtin_prop(This, id, 0, DISPATCH_PROPERTYPUT, &dp, NULL, &ei, NULL); + if(FAILED(hres)) + return hres; + + *success = VARIANT_TRUE; + return S_OK; + } + + hres = get_dynamic_prop(This, name, 0, &prop); + if(FAILED(hres)) { + if(hres != DISP_E_UNKNOWNNAME) + return hres; + *success = VARIANT_FALSE; + return S_OK; + } + + VariantClear(&prop->var); + prop->flags |= DYNPROP_DELETED; + *success = VARIANT_TRUE; + return S_OK; +} + #define DISPATCHEX_THIS(iface) DEFINE_THIS(DispatchEx, IDispatchEx, iface) static HRESULT WINAPI DispatchEx_QueryInterface(IDispatchEx *iface, REFIID riid, void **ppv) @@ -926,12 +977,12 @@ static HRESULT WINAPI DispatchEx_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc if(is_dynamic_dispid(id)) { DWORD idx = id - DISPID_DYNPROP_0; - VARIANT *var; + dynamic_prop_t *prop; if(!This->dynamic_data || This->dynamic_data->prop_cnt <= idx) return DISP_E_UNKNOWNNAME; - var = &This->dynamic_data->props[idx].var; + prop = This->dynamic_data->props+idx; switch(wFlags) { case DISPATCH_METHOD|DISPATCH_PROPERTYGET: @@ -942,8 +993,8 @@ static HRESULT WINAPI DispatchEx_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc DISPPARAMS dp = {NULL, &named_arg, 0, 1}; IDispatchEx *dispex; - if(V_VT(var) != VT_DISPATCH) { - FIXME("invoke vt %d\n", V_VT(var)); + if(V_VT(&prop->var) != VT_DISPATCH) { + FIXME("invoke %s\n", debugstr_variant(&prop->var)); return E_NOTIMPL; } @@ -962,14 +1013,14 @@ static HRESULT WINAPI DispatchEx_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc V_VT(dp.rgvarg) = VT_DISPATCH; V_DISPATCH(dp.rgvarg) = (IDispatch*)DISPATCHEX(This); - hres = IDispatch_QueryInterface(V_DISPATCH(var), &IID_IDispatchEx, (void**)&dispex); + hres = IDispatch_QueryInterface(V_DISPATCH(&prop->var), &IID_IDispatchEx, (void**)&dispex); TRACE("%s call\n", debugstr_w(This->dynamic_data->props[idx].name)); if(SUCCEEDED(hres)) { hres = IDispatchEx_InvokeEx(dispex, DISPID_VALUE, lcid, wFlags, &dp, pvarRes, pei, pspCaller); IDispatchEx_Release(dispex); }else { ULONG err = 0; - hres = IDispatch_Invoke(V_DISPATCH(var), DISPID_VALUE, &IID_NULL, lcid, wFlags, pdp, pvarRes, pei, &err); + hres = IDispatch_Invoke(V_DISPATCH(&prop->var), DISPID_VALUE, &IID_NULL, lcid, wFlags, pdp, pvarRes, pei, &err); } TRACE("%s ret %08x\n", debugstr_w(This->dynamic_data->props[idx].name), hres); @@ -977,7 +1028,9 @@ static HRESULT WINAPI DispatchEx_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc return hres; } case DISPATCH_PROPERTYGET: - return VariantCopy(pvarRes, var); + if(prop->flags & DYNPROP_DELETED) + return DISP_E_UNKNOWNNAME; + return VariantCopy(pvarRes, &prop->var); case DISPATCH_PROPERTYPUT|DISPATCH_PROPERTYPUTREF: case DISPATCH_PROPERTYPUT: if(pdp->cArgs != 1 || (pdp->cNamedArgs == 1 && *pdp->rgdispidNamedArgs != DISPID_PROPERTYPUT) @@ -987,8 +1040,13 @@ static HRESULT WINAPI DispatchEx_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc } TRACE("put %s\n", debugstr_variant(pdp->rgvarg)); - VariantClear(var); - return VariantCopy(var, pdp->rgvarg); + VariantClear(&prop->var); + hres = VariantCopy(&prop->var, pdp->rgvarg); + if(FAILED(hres)) + return hres; + + prop->flags &= ~DYNPROP_DELETED; + return S_OK; default: FIXME("unhandled wFlags %x\n", wFlags); return E_NOTIMPL; @@ -1073,12 +1131,14 @@ static HRESULT WINAPI DispatchEx_GetNextDispID(IDispatchEx *iface, DWORD grfdex, if(!This->dynamic_data || This->dynamic_data->prop_cnt <= idx) return DISP_E_UNKNOWNNAME; - if(idx+1 == This->dynamic_data->prop_cnt) { + while(++idx < This->dynamic_data->prop_cnt && This->dynamic_data->props[idx].flags & DYNPROP_DELETED); + + if(idx == This->dynamic_data->prop_cnt) { *pid = DISPID_STARTENUM; return S_FALSE; } - *pid = id+1; + *pid = DISPID_DYNPROP_0+idx; return S_OK; } diff --git a/dlls/mshtml/htmlelem.c b/dlls/mshtml/htmlelem.c index e77c1dc..b0cb791 100644 --- a/dlls/mshtml/htmlelem.c +++ b/dlls/mshtml/htmlelem.c @@ -198,8 +198,10 @@ static HRESULT WINAPI HTMLElement_removeAttribute(IHTMLElement *iface, BSTR strA LONG lFlags, VARIANT_BOOL *pfSuccess) { HTMLElement *This = HTMLELEM_THIS(iface); - FIXME("(%p)->()\n", This); - return E_NOTIMPL; + + TRACE("(%p)->(%s %x %p)\n", This, debugstr_w(strAttributeName), lFlags, pfSuccess); + + return remove_prop(&This->node.dispex, strAttributeName, pfSuccess); } static HRESULT WINAPI HTMLElement_put_className(IHTMLElement *iface, BSTR v) diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index 228be35..374b6c0 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -171,6 +171,7 @@ void release_dispex(DispatchEx*); BOOL dispex_query_interface(DispatchEx*,REFIID,void**); HRESULT dispex_get_dprop_ref(DispatchEx*,const WCHAR*,BOOL,VARIANT**); HRESULT get_dispids(tid_t,DWORD*,DISPID**); +HRESULT remove_prop(DispatchEx*,BSTR,VARIANT_BOOL*); typedef struct HTMLWindow HTMLWindow; typedef struct HTMLDocumentNode HTMLDocumentNode;

1 0

Nikolay Sivov : cmd: Ignore spaces before goto label.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: e22f6aa6e3e79de063d242e285b89881d701dfbe URL: http://source.winehq.org/git/wine.git/?a=commit;h=e22f6aa6e3e79de063d242e28… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Tue Mar 23 22:45:25 2010 -0500 cmd: Ignore spaces before goto label. --- programs/cmd/builtins.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c index 15d320c..8049ed8 100644 --- a/programs/cmd/builtins.c +++ b/programs/cmd/builtins.c @@ -1165,7 +1165,7 @@ void WCMD_goto (CMD_LIST **cmdList) { return; } if (context != NULL) { - WCHAR *paramStart = param1; + WCHAR *paramStart = param1, *str; static const WCHAR eofW[] = {':','e','o','f','\0'}; /* Handle special :EOF label */ @@ -1179,7 +1179,9 @@ void WCMD_goto (CMD_LIST **cmdList) { SetFilePointer (context -> h, 0, NULL, FILE_BEGIN); while (WCMD_fgets (string, sizeof(string)/sizeof(WCHAR), context -> h)) { - if ((string[0] == ':') && (lstrcmpiW (&string[1], paramStart) == 0)) return; + str = string; + while (isspaceW(*str)) str++; + if ((*str == ':') && (lstrcmpiW (++str, paramStart) == 0)) return; } WCMD_output (WCMD_LoadMessage(WCMD_NOTARGET)); }

1 0

Nikolay Sivov : shell32/tests: Fix a test crash on Win9x.
by Alexandre Julliard March 24, 2010

March 24, 2010

Module: wine Branch: master Commit: ef7abf10870c10eed7bf6fff86cdd3313f61d384 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ef7abf10870c10eed7bf6fff8… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Mar 24 02:41:26 2010 +0300 shell32/tests: Fix a test crash on Win9x. --- dlls/shell32/tests/shlview.c | 144 ++++++++++++++++++++++++++++++++++++++---- 1 files changed, 131 insertions(+), 13 deletions(-) diff --git a/dlls/shell32/tests/shlview.c b/dlls/shell32/tests/shlview.c index 59716d5..dc60a49 100644 --- a/dlls/shell32/tests/shlview.c +++ b/dlls/shell32/tests/shlview.c @@ -86,13 +86,134 @@ static HWND subclass_listview(HWND hwnd) return listview; } +/* dummy IDataObject implementation */ typedef struct { + const IDataObjectVtbl *lpVtbl; + LONG ref; +} IDataObjectImpl; + +static const IDataObjectVtbl IDataObjectImpl_Vtbl; + +IDataObject* IDataObjectImpl_Construct(void) +{ + IDataObjectImpl *obj; + + obj = HeapAlloc(GetProcessHeap(), 0, sizeof(*obj)); + obj->lpVtbl = &IDataObjectImpl_Vtbl; + obj->ref = 1; + + return (IDataObject*)obj; +} + +static HRESULT WINAPI IDataObjectImpl_QueryInterface(IDataObject *iface, REFIID riid, void **ppvObj) +{ + IDataObjectImpl *This = (IDataObjectImpl *)iface; + + if (IsEqualIID(riid, &IID_IUnknown) || + IsEqualIID(riid, &IID_IDataObject)) + { + *ppvObj = This; + } + + if(*ppvObj) + { + IUnknown_AddRef(iface); + return S_OK; + } + + return E_NOINTERFACE; +} + +static ULONG WINAPI IDataObjectImpl_AddRef(IDataObject * iface) +{ + IDataObjectImpl *This = (IDataObjectImpl *)iface; + return InterlockedIncrement(&This->ref); +} + +static ULONG WINAPI IDataObjectImpl_Release(IDataObject * iface) +{ + IDataObjectImpl *This = (IDataObjectImpl *)iface; + ULONG ref = InterlockedDecrement(&This->ref); + + if (!ref) + { + HeapFree(GetProcessHeap(), 0, This); + return 0; + } + return ref; +} + +static HRESULT WINAPI IDataObjectImpl_GetData(IDataObject *iface, FORMATETC *pformat, STGMEDIUM *pmedium) +{ + return E_NOTIMPL; +} + +static HRESULT WINAPI IDataObjectImpl_GetDataHere(IDataObject *iface, FORMATETC *pformat, STGMEDIUM *pmedium) +{ + return E_NOTIMPL; +} + +static HRESULT WINAPI IDataObjectImpl_QueryGetData(IDataObject *iface, FORMATETC *pformat) +{ + return E_NOTIMPL; +} + +static HRESULT WINAPI IDataObjectImpl_GetCanonicalFormatEtc( + IDataObject *iface, FORMATETC *pformatIn, FORMATETC *pformatOut) +{ + return E_NOTIMPL; +} + +static HRESULT WINAPI IDataObjectImpl_SetData( + IDataObject *iface, FORMATETC *pformat, STGMEDIUM *pmedium, BOOL release) +{ + return E_NOTIMPL; +} + +static HRESULT WINAPI IDataObjectImpl_EnumFormatEtc( + IDataObject *iface, DWORD direction, IEnumFORMATETC **ppenumFormatEtc) +{ + return E_NOTIMPL; +} + +static HRESULT WINAPI IDataObjectImpl_DAdvise( + IDataObject *iface, FORMATETC *pformatetc, DWORD advf, IAdviseSink *pSink, DWORD *pConnection) +{ + return E_NOTIMPL; +} + +static HRESULT WINAPI IDataObjectImpl_DUnadvise(IDataObject *iface, DWORD connection) +{ + return E_NOTIMPL; +} + +static HRESULT WINAPI IDataObjectImpl_EnumDAdvise(IDataObject *iface, IEnumSTATDATA **ppenumAdvise) +{ + return E_NOTIMPL; +} + +static const IDataObjectVtbl IDataObjectImpl_Vtbl = +{ + IDataObjectImpl_QueryInterface, + IDataObjectImpl_AddRef, + IDataObjectImpl_Release, + IDataObjectImpl_GetData, + IDataObjectImpl_GetDataHere, + IDataObjectImpl_QueryGetData, + IDataObjectImpl_GetCanonicalFormatEtc, + IDataObjectImpl_SetData, + IDataObjectImpl_EnumFormatEtc, + IDataObjectImpl_DAdvise, + IDataObjectImpl_DUnadvise, + IDataObjectImpl_EnumDAdvise +}; +/* dummy IShellBrowser implementation */ +typedef struct { const IShellBrowserVtbl *lpVtbl; LONG ref; } IShellBrowserImpl; -/* dummy IShellBrowser implementation */ static const IShellBrowserVtbl IShellBrowserImpl_Vtbl; IShellBrowser* IShellBrowserImpl_Construct(void) @@ -114,22 +235,16 @@ static HRESULT WINAPI IShellBrowserImpl_QueryInterface(IShellBrowser *iface, *ppvObj = NULL; - if(IsEqualIID(riid, &IID_IUnknown)) - { - *ppvObj = This; - } - else if(IsEqualIID(riid, &IID_IOleWindow)) - { - *ppvObj = This; - } - else if(IsEqualIID(riid, &IID_IShellBrowser)) + if(IsEqualIID(riid, &IID_IUnknown) || + IsEqualIID(riid, &IID_IOleWindow) || + IsEqualIID(riid, &IID_IShellBrowser)) { *ppvObj = This; } if(*ppvObj) { - IUnknown_AddRef( (IShellBrowser*) *ppvObj); + IUnknown_AddRef(iface); return S_OK; } @@ -556,6 +671,7 @@ static void test_IShellFolderView(void) IShellFolderView *folderview; IShellFolder *desktop; IShellView *view; + IDataObject *obj; UINT i; HRESULT hr; @@ -575,8 +691,10 @@ static void test_IShellFolderView(void) } /* ::MoveIcons */ - hr = IShellFolderView_MoveIcons(folderview, NULL); - ok(hr == E_NOTIMPL, "got (0x%08x)\n", hr); + obj = IDataObjectImpl_Construct(); + hr = IShellFolderView_MoveIcons(folderview, obj); + ok(hr == E_NOTIMPL || broken(hr == S_OK) /* W98 */, "got (0x%08x)\n", hr); + IDataObject_Release(obj); /* ::SetRedraw without list created */ hr = IShellFolderView_SetRedraw(folderview, TRUE);

1 0