January 2014 - Wine-commits

Michael Stefaniuc : dmime/tests: Fix a crash due to a missing audio driver.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: 03d5b1248bcb75cc84b1bebe4e4a2b8a4cca8869 URL: http://source.winehq.org/git/wine.git/?a=commit;h=03d5b1248bcb75cc84b1bebe4… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Fri Jan 10 23:07:15 2014 +0100 dmime/tests: Fix a crash due to a missing audio driver. --- dlls/dmime/tests/dmime.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dlls/dmime/tests/dmime.c b/dlls/dmime/tests/dmime.c index 3d5a426..669e99e 100644 --- a/dlls/dmime/tests/dmime.c +++ b/dlls/dmime/tests/dmime.c @@ -57,6 +57,10 @@ static void test_COM_audiopath(void) } hr = IDirectMusicPerformance8_InitAudio(performance, NULL, NULL, NULL, DMUS_APATH_SHARED_STEREOPLUSREVERB, 64, DMUS_AUDIOF_ALL, NULL); + if (hr == DSERR_NODRIVER) { + skip("No audio driver\n"); + return; + } ok(hr == S_OK, "DirectMusicPerformance_InitAudio failed: %08x\n", hr); hr = IDirectMusicPerformance8_GetDefaultAudioPath(performance, &dmap); ok(hr == S_OK, "DirectMusicPerformance_GetDefaultAudioPath failed: %08x\n", hr);

1 0

Michael Stefaniuc : dmloader/tests: Fix a crash due to a missing audio driver.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: 3fef1be56f70075076d9ddb767b47d1a059152e4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3fef1be56f70075076d9ddb76… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Fri Jan 10 23:07:26 2014 +0100 dmloader/tests: Fix a crash due to a missing audio driver. --- dlls/dmloader/tests/loader.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dlls/dmloader/tests/loader.c b/dlls/dmloader/tests/loader.c index ea5edca..8759033 100644 --- a/dlls/dmloader/tests/loader.c +++ b/dlls/dmloader/tests/loader.c @@ -73,6 +73,10 @@ static void test_simple_playing(void) hr = IDirectMusicPerformance8_InitAudio(perf, &music, &dsound, NULL, DMUS_APATH_DYNAMIC_STEREO, 64, DMUS_AUDIOF_ALL, NULL); + if (hr == DSERR_NODRIVER) { + skip("No audio driver.\n"); + return; + } ok(hr == S_OK, "InitAudio failed: %08x\n", hr); ok(music != NULL, "Didn't get IDirectMusic pointer\n"); ok(dsound != NULL, "Didn't get IDirectSound pointer\n");

1 0

Andrey Gusev : Assorted typos fixes.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: be367393c95bd391d7ad3309d3e085d9f4be2237 URL: http://source.winehq.org/git/wine.git/?a=commit;h=be367393c95bd391d7ad3309d… Author: Andrey Gusev <andrey.goosev(a)gmail.com> Date: Sat Jan 11 17:02:02 2014 +0200 Assorted typos fixes. --- dlls/acledit/main.c | 2 +- dlls/advapi32/security.c | 4 ++-- dlls/atl100/atl.c | 2 +- dlls/avifil32/api.c | 2 +- dlls/avifil32/avifile.c | 2 +- dlls/comctl32/dpa.c | 2 +- dlls/comctl32/imagelist.c | 2 +- dlls/comctl32/listview.c | 6 +++--- dlls/comctl32/tests/trackbar.c | 2 +- dlls/comctl32/treeview.c | 2 +- dlls/comdlg32/filedlg31.c | 2 +- dlls/comdlg32/fontdlg.c | 2 +- dlls/comdlg32/printdlg.c | 8 ++++---- dlls/crypt32/tests/store.c | 2 +- 14 files changed, 20 insertions(+), 20 deletions(-) diff --git a/dlls/acledit/main.c b/dlls/acledit/main.c index d1a9ac7..49de8b5 100644 --- a/dlls/acledit/main.c +++ b/dlls/acledit/main.c @@ -55,7 +55,7 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) LONG WINAPI FMExtensionProcW(HWND hWnd, WORD wEvent, LONG lParam) { - FIXME("%p, %d, 0x%x) stub\n", hWnd, wEvent, lParam); + FIXME("(%p, %d, 0x%x) stub\n", hWnd, wEvent, lParam); return 0; } diff --git a/dlls/advapi32/security.c b/dlls/advapi32/security.c index 280ba77..ac0a941 100644 --- a/dlls/advapi32/security.c +++ b/dlls/advapi32/security.c @@ -5302,7 +5302,7 @@ BOOL WINAPI CreateProcessAsUserW( LPSTARTUPINFOW lpStartupInfo, LPPROCESS_INFORMATION lpProcessInformation ) { - FIXME("%p %s %s %p %p %d 0x%08x %p %s %p %p - semi- stub\n", hToken, + FIXME("%p %s %s %p %p %d 0x%08x %p %s %p %p - semi-stub\n", hToken, debugstr_w(lpApplicationName), debugstr_w(lpCommandLine), lpProcessAttributes, lpThreadAttributes, bInheritHandles, dwCreationFlags, lpEnvironment, debugstr_w(lpCurrentDirectory), lpStartupInfo, lpProcessInformation); @@ -5801,7 +5801,7 @@ DWORD WINAPI TreeResetNamedSecurityInfoW( LPWSTR pObjectName, BOOL KeepExplicit, FN_PROGRESS fnProgress, PROG_INVOKE_SETTING ProgressInvokeSetting, PVOID Args) { - FIXME("(%s, %i, %i, %p, %p, %p, %p, %i, %p, %i, %p Stub\n", + FIXME("(%s, %i, %i, %p, %p, %p, %p, %i, %p, %i, %p) stub\n", debugstr_w(pObjectName), ObjectType, SecurityInfo, pOwner, pGroup, pDacl, pSacl, KeepExplicit, fnProgress, ProgressInvokeSetting, Args); diff --git a/dlls/atl100/atl.c b/dlls/atl100/atl.c index 9156afb..058fa22 100644 --- a/dlls/atl100/atl.c +++ b/dlls/atl100/atl.c @@ -391,7 +391,7 @@ HRESULT WINAPI AtlLoadTypeLib(HINSTANCE inst, LPCOLESTR lpszIndex, */ HRESULT WINAPI AtlWinModuleInit(_ATL_WIN_MODULE *winmod) { - TRACE("(%p\n", winmod); + TRACE("(%p)\n", winmod); if(winmod->cbSize != sizeof(*winmod)) return E_INVALIDARG; diff --git a/dlls/avifil32/api.c b/dlls/avifil32/api.c index 6c07a2f..20985cd 100644 --- a/dlls/avifil32/api.c +++ b/dlls/avifil32/api.c @@ -1515,7 +1515,7 @@ HRESULT WINAPI AVISaveVA(LPCSTR szFile, CLSID *pclsidHandler, HRESULT hr; int len; - TRACE("%s,%p,%p,%d,%p,%p)\n", debugstr_a(szFile), pclsidHandler, + TRACE("(%s,%p,%p,%d,%p,%p)\n", debugstr_a(szFile), pclsidHandler, lpfnCallback, nStream, ppavi, plpOptions); if (szFile == NULL || ppavi == NULL || plpOptions == NULL) diff --git a/dlls/avifil32/avifile.c b/dlls/avifil32/avifile.c index 7c14805..6295c7b 100644 --- a/dlls/avifil32/avifile.c +++ b/dlls/avifil32/avifile.c @@ -1292,7 +1292,7 @@ static HRESULT WINAPI IAVIStream_fnWriteData(IAVIStream *iface, DWORD fcc, void if (fcc == ckidSTREAMHANDLERDATA) { if (This->lpHandlerData != NULL) { - FIXME(": handler data already set -- overwirte?\n"); + FIXME(": handler data already set -- overwrite?\n"); return AVIERR_UNSUPPORTED; } diff --git a/dlls/comctl32/dpa.c b/dlls/comctl32/dpa.c index 11dd14f..0b96df6 100644 --- a/dlls/comctl32/dpa.c +++ b/dlls/comctl32/dpa.c @@ -269,7 +269,7 @@ BOOL WINAPI DPA_Merge (HDPA hdpa1, HDPA hdpa2, DWORD dwFlags, INT nResult, i; INT nIndex; - TRACE("%p %p %08x %p %p %08lx)\n", + TRACE("(%p %p %08x %p %p %08lx)\n", hdpa1, hdpa2, dwFlags, pfnCompare, pfnMerge, lParam); if (IsBadWritePtr (hdpa1, sizeof(*hdpa1))) diff --git a/dlls/comctl32/imagelist.c b/dlls/comctl32/imagelist.c index 6461df6..ec90a9c 100644 --- a/dlls/comctl32/imagelist.c +++ b/dlls/comctl32/imagelist.c @@ -2889,7 +2889,7 @@ ImageList_SetImageCount (HIMAGELIST himl, UINT iImageCount) himl->hbmImage = hbmNewBitmap; } else - ERR("Could not create new image bitmap !\n"); + ERR("Could not create new image bitmap!\n"); if (himl->hbmMask) { diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c index 5ce7de2..bc9a04c 100644 --- a/dlls/comctl32/listview.c +++ b/dlls/comctl32/listview.c @@ -8247,7 +8247,7 @@ static BOOL LISTVIEW_SetColumnWidth(LISTVIEW_INFO *infoPtr, INT nColumn, INT cx) INT max_cx = 0; HDITEMW hdi; - TRACE("(nColumn=%d, cx=%d\n", nColumn, cx); + TRACE("(nColumn=%d, cx=%d)\n", nColumn, cx); /* set column width only if in report or list mode */ if (infoPtr->uView != LV_VIEW_DETAILS && infoPtr->uView != LV_VIEW_LIST) return FALSE; @@ -8631,7 +8631,7 @@ static HIMAGELIST LISTVIEW_SetImageList(LISTVIEW_INFO *infoPtr, INT nType, HIMAG INT oldHeight = infoPtr->nItemHeight; HIMAGELIST himlOld = 0; - TRACE("(nType=%d, himl=%p\n", nType, himl); + TRACE("(nType=%d, himl=%p)\n", nType, himl); switch (nType) { @@ -8786,7 +8786,7 @@ static BOOL LISTVIEW_SetItemPosition(LISTVIEW_INFO *infoPtr, INT nItem, const PO { POINT Origin, Pt; - TRACE("(nItem=%d, pt=%s\n", nItem, wine_dbgstr_point(pt)); + TRACE("(nItem=%d, pt=%s)\n", nItem, wine_dbgstr_point(pt)); if (!pt || nItem < 0 || nItem >= infoPtr->nItemCount || !(infoPtr->uView == LV_VIEW_ICON || infoPtr->uView == LV_VIEW_SMALLICON)) return FALSE; diff --git a/dlls/comctl32/tests/trackbar.c b/dlls/comctl32/tests/trackbar.c index 3ee7fda..32c9575 100644 --- a/dlls/comctl32/tests/trackbar.c +++ b/dlls/comctl32/tests/trackbar.c @@ -777,7 +777,7 @@ static void test_tic_settings(HWND hWndTrackbar){ ok(r == TRUE, "Expected TRUE, got %d\n", r); r = SendMessageA(hWndTrackbar, TBM_SETTIC, 0, 20); - ok(r == FALSE, "Expected False, got %d\n", r); + ok(r == FALSE, "Expected FALSE, got %d\n", r); /* test TBM_SETTICFREQ */ SendMessageA(hWndTrackbar, TBM_SETRANGE, TRUE, MAKELONG(0, 10)); diff --git a/dlls/comctl32/treeview.c b/dlls/comctl32/treeview.c index a885543..b2eed3c 100644 --- a/dlls/comctl32/treeview.c +++ b/dlls/comctl32/treeview.c @@ -3428,7 +3428,7 @@ TREEVIEW_Expand(TREEVIEW_INFO *infoPtr, TREEVIEW_ITEM *item, TREEVIEW_ITEM *nextItem, *tmpItem; BOOL sendsNotifications; - TRACE("(%p, %p, partial=%d, %d\n", infoPtr, item, partial, user); + TRACE("(%p, %p, partial=%d, %d)\n", infoPtr, item, partial, user); if (!TREEVIEW_HasChildren(infoPtr, item)) return FALSE; diff --git a/dlls/comdlg32/filedlg31.c b/dlls/comdlg32/filedlg31.c index 5f4c676..0409915 100644 --- a/dlls/comdlg32/filedlg31.c +++ b/dlls/comdlg32/filedlg31.c @@ -89,7 +89,7 @@ static BOOL FD31_Init(void) if (hFolder == 0 || hFolder2 == 0 || hFloppy == 0 || hHDisk == 0 || hCDRom == 0 || hNet == 0) { - ERR("Error loading icons !\n"); + ERR("Error loading icons!\n"); return FALSE; } initialized = TRUE; diff --git a/dlls/comdlg32/fontdlg.c b/dlls/comdlg32/fontdlg.c index 68f0467..9e73ec9 100644 --- a/dlls/comdlg32/fontdlg.c +++ b/dlls/comdlg32/fontdlg.c @@ -632,7 +632,7 @@ static LRESULT CFn_WMInitDialog(HWND hDlg, LPARAM lParam, LPCHOOSEFONTW lpcf) if (lpcf->lStructSize != sizeof(CHOOSEFONTW)) { - ERR("structure size failure !!!\n"); + ERR("structure size failure!!!\n"); EndDialog (hDlg, 0); return FALSE; } diff --git a/dlls/comdlg32/printdlg.c b/dlls/comdlg32/printdlg.c index 292f764..d52b47d 100644 --- a/dlls/comdlg32/printdlg.c +++ b/dlls/comdlg32/printdlg.c @@ -1495,7 +1495,7 @@ static LRESULT PRINTDLG_WMInitDialog(HWND hDlg, PrintStructures->hNoCollateIcon == 0 || PrintStructures->hPortraitIcon == 0 || PrintStructures->hLandscapeIcon == 0) { - ERR("no icon in resourcefile\n"); + ERR("no icon in resource file\n"); COMDLG32_SetCommDlgExtendedError(CDERR_LOADRESFAILURE); EndDialog(hDlg, FALSE); } @@ -1603,7 +1603,7 @@ static LRESULT PRINTDLG_WMInitDialogW(HWND hDlg, PrintStructures->hNoCollateIcon == 0 || PrintStructures->hPortraitIcon == 0 || PrintStructures->hLandscapeIcon == 0) { - ERR("no icon in resourcefile\n"); + ERR("no icon in resource file\n"); COMDLG32_SetCommDlgExtendedError(CDERR_LOADRESFAILURE); EndDialog(hDlg, FALSE); } @@ -2263,7 +2263,7 @@ BOOL WINAPI PrintDlgA(LPPRINTDLGA lppd) } if(lppd->lStructSize != sizeof(PRINTDLGA)) { - WARN("structure size failure !!!\n"); + WARN("structure size failure!!!\n"); COMDLG32_SetCommDlgExtendedError(CDERR_STRUCTSIZE); return FALSE; } @@ -2414,7 +2414,7 @@ BOOL WINAPI PrintDlgW(LPPRINTDLGW lppd) } if(lppd->lStructSize != sizeof(PRINTDLGW)) { - WARN("structure size failure !!!\n"); + WARN("structure size failure!!!\n"); COMDLG32_SetCommDlgExtendedError(CDERR_STRUCTSIZE); return FALSE; } diff --git a/dlls/crypt32/tests/store.c b/dlls/crypt32/tests/store.c index 1fa3ffa..a416a39 100644 --- a/dlls/crypt32/tests/store.c +++ b/dlls/crypt32/tests/store.c @@ -130,7 +130,7 @@ static void _test_store_is_empty(unsigned line, HCERTSTORE store) const CERT_CONTEXT *cert; cert = CertEnumCertificatesInStore(store, NULL); - ok_(__FILE__,line)(!cert && GetLastError() == CRYPT_E_NOT_FOUND, "store is not epmty\n"); + ok_(__FILE__,line)(!cert && GetLastError() == CRYPT_E_NOT_FOUND, "store is not empty\n"); } static void testMemStore(void)

1 0

André Hentschel : oleaut32/tests: Mark W2k results as broken.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: 0485e528f7c3fcf73f5dbf5ad5d6d1444de43192 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0485e528f7c3fcf73f5dbf5ad… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sun Jan 12 23:28:14 2014 +0100 oleaut32/tests: Mark W2k results as broken. --- dlls/oleaut32/tests/safearray.c | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/dlls/oleaut32/tests/safearray.c b/dlls/oleaut32/tests/safearray.c index 3e311e3..bb346f6 100644 --- a/dlls/oleaut32/tests/safearray.c +++ b/dlls/oleaut32/tests/safearray.c @@ -1467,11 +1467,17 @@ static void test_SafeArrayCopyData(void) sa = SafeArrayCreateVector(VT_UI1, 0, 2); sacopy = SafeArrayCreateVector(VT_UI1, 0, 2); - ok(sa->fFeatures == (FADF_HAVEVARTYPE|FADF_CREATEVECTOR), "got 0x%08x\n", sa->fFeatures); - ok(sacopy->fFeatures == (FADF_HAVEVARTYPE|FADF_CREATEVECTOR), "got 0x%08x\n", sacopy->fFeatures); + ok(sa->fFeatures == (FADF_HAVEVARTYPE|FADF_CREATEVECTOR) || + broken(sa->fFeatures == FADF_CREATEVECTOR /* W2k */), + "got 0x%08x\n", sa->fFeatures); + ok(sacopy->fFeatures == (FADF_HAVEVARTYPE|FADF_CREATEVECTOR) || + broken(sacopy->fFeatures == FADF_CREATEVECTOR /* W2k */), + "got 0x%08x\n", sacopy->fFeatures); hres = SafeArrayCopyData(sa, sacopy); ok(hres == S_OK, "got 0x%08x\n", hres); - ok(sacopy->fFeatures == (FADF_HAVEVARTYPE|FADF_CREATEVECTOR), "got 0x%04x\n", sacopy->fFeatures); + ok(sacopy->fFeatures == (FADF_HAVEVARTYPE|FADF_CREATEVECTOR) || + broken(sacopy->fFeatures == FADF_CREATEVECTOR /* W2k */), + "got 0x%04x\n", sacopy->fFeatures); SafeArrayDestroy(sacopy); sacopy = SafeArrayCreate(VT_UI1, NUM_DIMENSIONS, sab); @@ -1771,10 +1777,14 @@ static void test_SafeArrayCopy(void) /* copy from a vector */ sa = SafeArrayCreateVector(VT_UI1, 0, 2); - ok(sa->fFeatures == (FADF_HAVEVARTYPE|FADF_CREATEVECTOR), "got 0x%08x\n", hres); + ok(sa->fFeatures == (FADF_HAVEVARTYPE|FADF_CREATEVECTOR) || + broken(!sa->fFeatures /* W2k */), + "got 0x%08x\n", hres); hres = SafeArrayCopy(sa, &sa2); ok(hres == S_OK, "got 0x%08x\n", hres); - ok(sa2->fFeatures == FADF_HAVEVARTYPE, "got 0x%04x\n", sa2->fFeatures); + ok(sa2->fFeatures == FADF_HAVEVARTYPE || + broken(!sa2->fFeatures /* W2k */), "got 0x%04x\n", + sa2->fFeatures); SafeArrayDestroy(sa2); SafeArrayDestroy(sa);

1 0

Huw Davies : mshtml: Release a leaked class factory.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: 6da6f1861bb2f299c59c1a733ccfe5571b2594b3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6da6f1861bb2f299c59c1a733… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Jan 13 13:32:27 2014 +0000 mshtml: Release a leaked class factory. --- dlls/mshtml/npplugin.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/mshtml/npplugin.c b/dlls/mshtml/npplugin.c index f5eb754..0ce72ff 100644 --- a/dlls/mshtml/npplugin.c +++ b/dlls/mshtml/npplugin.c @@ -635,6 +635,7 @@ static IUnknown *create_activex_object(HTMLInnerWindow *window, nsIDOMElement *n } hres = IClassFactory_CreateInstance(cf, NULL, &IID_IUnknown, (void**)&obj); + IClassFactory_Release(cf); if(FAILED(hres)) return NULL;

1 0

Dmitry Timoshkov : taskschd/tests: Add some tests for ITaskService:: Connect.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: 55ad9da91f04001351b805675d12e2ffb72cef44 URL: http://source.winehq.org/git/wine.git/?a=commit;h=55ad9da91f04001351b805675… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jan 13 19:07:04 2014 +0900 taskschd/tests: Add some tests for ITaskService::Connect. --- configure | 1 + configure.ac | 1 + dlls/taskschd/tests/Makefile.in | 5 ++ dlls/taskschd/tests/scheduler.c | 118 +++++++++++++++++++++++++++++++++++++++ 4 files changed, 125 insertions(+) diff --git a/configure b/configure index 573bb96..7255985 100755 --- a/configure +++ b/configure @@ -17068,6 +17068,7 @@ wine_fn_config_dll system.drv16 enable_win16 wine_fn_config_dll t2embed enable_t2embed wine_fn_config_dll tapi32 enable_tapi32 implib wine_fn_config_dll taskschd enable_taskschd clean +wine_fn_config_test dlls/taskschd/tests taskschd_test wine_fn_config_dll toolhelp.dll16 enable_win16 wine_fn_config_dll traffic enable_traffic wine_fn_config_dll twain.dll16 enable_win16 diff --git a/configure.ac b/configure.ac index a4f0063..dcabb55 100644 --- a/configure.ac +++ b/configure.ac @@ -3121,6 +3121,7 @@ WINE_CONFIG_DLL(system.drv16,enable_win16) WINE_CONFIG_DLL(t2embed) WINE_CONFIG_DLL(tapi32,,[implib]) WINE_CONFIG_DLL(taskschd,,[clean]) +WINE_CONFIG_TEST(dlls/taskschd/tests) WINE_CONFIG_DLL(toolhelp.dll16,enable_win16) WINE_CONFIG_DLL(traffic) WINE_CONFIG_DLL(twain.dll16,enable_win16) diff --git a/dlls/taskschd/tests/Makefile.in b/dlls/taskschd/tests/Makefile.in new file mode 100644 index 0000000..63424da --- /dev/null +++ b/dlls/taskschd/tests/Makefile.in @@ -0,0 +1,5 @@ +TESTDLL = taskschd.dll +IMPORTS = oleaut32 ole32 + +C_SRCS = \ + scheduler.c diff --git a/dlls/taskschd/tests/scheduler.c b/dlls/taskschd/tests/scheduler.c new file mode 100644 index 0000000..9b99258 --- /dev/null +++ b/dlls/taskschd/tests/scheduler.c @@ -0,0 +1,118 @@ +/* + * Copyright 2014 Dmitry Timoshkov + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <stdarg.h> + +#define COBJMACROS + +#include "windef.h" +#include "winbase.h" +#include "initguid.h" +#include "objbase.h" +#include "taskschd.h" + +#include <wine/test.h> + +static void test_Connect(void) +{ + static const WCHAR deadbeefW[] = { 'd','e','a','d','b','e','e','f',0 }; + WCHAR comp_name[MAX_COMPUTERNAME_LENGTH + 1]; + DWORD len; + HRESULT hr; + BSTR bstr; + VARIANT v_null, v_comp; + VARIANT_BOOL vbool; + ITaskService *service; + + hr = CoCreateInstance(&CLSID_TaskScheduler, NULL, CLSCTX_INPROC_SERVER, &IID_ITaskService, (void **)&service); + if (hr != S_OK) + { + win_skip("CoCreateInstance(CLSID_TaskScheduler) error %#x\n", hr); + return; + } + + vbool = 0xdead; + hr = ITaskService_get_Connected(service, &vbool); +todo_wine + ok(hr == S_OK, "get_Connected error %#x\n", hr); +todo_wine + ok(vbool == VARIANT_FALSE, "expected VARIANT_FALSE, got %d\n", vbool); + + hr = ITaskService_get_TargetServer(service, &bstr); +todo_wine + ok(hr == HRESULT_FROM_WIN32(ERROR_ONLY_IF_CONNECTED), "expected ERROR_ONLY_IF_CONNECTED, got %#x\n", hr); + + /* Win7 doesn't support UNC \\ prefix, but according to a user + * comment on MSDN Win8 supports both ways. + */ + len = sizeof(comp_name)/sizeof(comp_name[0]); + GetComputerNameW(comp_name, &len); + + V_VT(&v_null) = VT_NULL; + + V_VT(&v_comp) = VT_BSTR; + V_BSTR(&v_comp) = SysAllocString(comp_name); + + hr = ITaskService_Connect(service, v_comp, v_null, v_null, v_null); +todo_wine + ok(hr == S_OK || hr == E_ACCESSDENIED /* not an administrator */, "Connect error %#x\n", hr); + + SysFreeString(V_BSTR(&v_comp)); + V_BSTR(&v_comp) = SysAllocString(deadbeefW); + + hr = ITaskService_Connect(service, v_comp, v_null, v_null, v_null); +todo_wine + ok(hr == HRESULT_FROM_WIN32(ERROR_BAD_NETPATH), "expected ERROR_BAD_NETPATH, got %#x\n", hr); + + vbool = 0xdead; + hr = ITaskService_get_Connected(service, &vbool); +todo_wine + ok(hr == S_OK, "get_Connected error %#x\n", hr); +todo_wine + ok(vbool == VARIANT_TRUE, "expected VARIANT_TRUE, got %d\n", vbool); + + hr = ITaskService_Connect(service, v_null, v_null, v_null, v_null); +todo_wine + ok(hr == S_OK, "Connect error %#x\n", hr); + + vbool = 0xdead; + hr = ITaskService_get_Connected(service, &vbool); +todo_wine + ok(hr == S_OK, "get_Connected error %#x\n", hr); +todo_wine + ok(vbool == VARIANT_TRUE, "expected VARIANT_TRUE, got %d\n", vbool); + + hr = ITaskService_get_TargetServer(service, &bstr); +todo_wine + ok(hr == S_OK, "get_TargetServer error %#x\n", hr); + if (hr == S_OK) + { + ok(!lstrcmpW(comp_name, bstr), "compname %s != server name %s\n", wine_dbgstr_w(comp_name), wine_dbgstr_w(bstr)); + SysFreeString(bstr); + } + SysFreeString(V_BSTR(&v_comp)); +} + +START_TEST(scheduler) +{ + OleInitialize(NULL); + + test_Connect(); + + OleUninitialize(); +}

1 0

Dmitry Timoshkov : secur32: Slightly simplify the GetUserNameExW implementation.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: bc59d7e8303a45d7356f3972d1d8e9478552fde8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bc59d7e8303a45d7356f3972d… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jan 13 19:07:31 2014 +0900 secur32: Slightly simplify the GetUserNameExW implementation. --- dlls/secur32/secur32.c | 35 ++++++++++++++--------------------- 1 file changed, 14 insertions(+), 21 deletions(-) diff --git a/dlls/secur32/secur32.c b/dlls/secur32/secur32.c index acb09cf..867fa2c 100644 --- a/dlls/secur32/secur32.c +++ b/dlls/secur32/secur32.c @@ -1096,16 +1096,16 @@ BOOLEAN WINAPI GetUserNameExA( BOOLEAN WINAPI GetUserNameExW( EXTENDED_NAME_FORMAT NameFormat, LPWSTR lpNameBuffer, PULONG nSize) { - BOOLEAN status; - WCHAR samname[UNLEN + 1 + MAX_COMPUTERNAME_LENGTH + 1]; - LPWSTR out; - DWORD len; TRACE("(%d %p %p)\n", NameFormat, lpNameBuffer, nSize); switch (NameFormat) { case NameSamCompatible: { + WCHAR samname[UNLEN + 1 + MAX_COMPUTERNAME_LENGTH + 1]; + LPWSTR out; + DWORD len; + /* This assumes the current user is always a local account */ len = MAX_COMPUTERNAME_LENGTH + 1; if (GetComputerNameW(samname, &len)) @@ -1115,25 +1115,20 @@ BOOLEAN WINAPI GetUserNameExW( len = UNLEN + 1; if (GetUserNameW(out, &len)) { - status = (lstrlenW(samname) < *nSize); - if (status) + if (lstrlenW(samname) < *nSize) { lstrcpyW(lpNameBuffer, samname); *nSize = lstrlenW(samname); + return TRUE; } - else - { - SetLastError(ERROR_MORE_DATA); - *nSize = lstrlenW(samname) + 1; - } + + SetLastError(ERROR_MORE_DATA); + *nSize = lstrlenW(samname) + 1; } - else - status = FALSE; } - else - status = FALSE; + return FALSE; } - break; + case NameUnknown: case NameFullyQualifiedDN: case NameDisplay: @@ -1144,14 +1139,12 @@ BOOLEAN WINAPI GetUserNameExW( case NameServicePrincipal: case NameDnsDomain: SetLastError(ERROR_NONE_MAPPED); - status = FALSE; - break; + return FALSE; + default: SetLastError(ERROR_INVALID_PARAMETER); - status = FALSE; + return FALSE; } - - return status; } BOOLEAN WINAPI TranslateNameA(

1 0

Stefan Dösinger : wined3d: Use the map binding to optimize sysmem->sysmem blits.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: 28bfc2058903793d87097941cc12f00f14e0c3ee URL: http://source.winehq.org/git/wine.git/?a=commit;h=28bfc2058903793d87097941c… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon Jan 13 15:21:00 2014 +0100 wined3d: Use the map binding to optimize sysmem->sysmem blits. --- dlls/wined3d/surface.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 54891ac..7023bd7 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -6216,8 +6216,8 @@ HRESULT CDECL wined3d_surface_blt(struct wined3d_surface *dst_surface, const REC { /* In principle this would apply to depth blits as well, but we don't * implement those in the CPU blitter at the moment. */ - if ((dst_surface->flags & SFLAG_INSYSMEM) - && (!src_surface || (src_surface->flags & SFLAG_INSYSMEM))) + if ((dst_surface->flags & dst_surface->map_binding) + && (!src_surface || (src_surface->flags & src_surface->map_binding))) { if (scale) TRACE("Not doing sysmem blit because of scaling.\n");

1 0

Stefan Dösinger : wined3d: Make the callers of load_location responsible for allocating sysmem.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: 676a7f2140aaefa50358001dbec89cc9a8d591e8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=676a7f2140aaefa50358001db… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon Jan 13 15:21:01 2014 +0100 wined3d: Make the callers of load_location responsible for allocating sysmem. --- dlls/wined3d/surface.c | 12 ++++++++---- dlls/wined3d/texture.c | 1 + dlls/wined3d/wined3d_private.h | 1 + 3 files changed, 10 insertions(+), 4 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 7023bd7..5ac2241 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -589,7 +589,7 @@ static void surface_create_pbo(struct wined3d_surface *surface, const struct win context_release(context); } -static void surface_prepare_system_memory(struct wined3d_surface *surface) +void surface_prepare_system_memory(struct wined3d_surface *surface) { const struct wined3d_gl_info *gl_info = &surface->resource.device->adapter->gl_info; @@ -767,6 +767,7 @@ static void surface_realize_palette(struct wined3d_surface *surface) if (!(surface->flags & surface->map_binding)) { TRACE("Palette changed with surface that does not have an up to date system memory copy.\n"); + surface_prepare_system_memory(surface); surface_load_location(surface, surface->map_binding); } surface_invalidate_location(surface, ~surface->map_binding); @@ -802,10 +803,10 @@ static BYTE *surface_map(struct wined3d_surface *surface, const RECT *rect, DWOR TRACE("surface %p, rect %s, flags %#x.\n", surface, wine_dbgstr_rect(rect), flags); + surface_prepare_system_memory(surface); if (flags & WINED3D_MAP_DISCARD) { TRACE("WINED3D_MAP_DISCARD flag passed, marking SYSMEM as up to date.\n"); - surface_prepare_system_memory(surface); surface_validate_location(surface, surface->map_binding); } else @@ -1391,6 +1392,7 @@ static void surface_unload(struct wined3d_resource *resource) } else { + surface_prepare_system_memory(surface); surface_load_location(surface, surface->map_binding); } surface_invalidate_location(surface, ~surface->map_binding); @@ -2321,6 +2323,7 @@ void surface_load(struct wined3d_surface *surface, BOOL srgb) /* To perform the color key conversion we need a sysmem copy of * the surface. Make sure we have it. */ + surface_prepare_system_memory(surface); surface_load_location(surface, surface->map_binding); surface_invalidate_location(surface, ~surface->map_binding); /* Switching color keying on / off may change the internal format. */ @@ -4907,8 +4910,6 @@ static DWORD resource_access_from_location(DWORD location) static void surface_load_sysmem(struct wined3d_surface *surface, const struct wined3d_gl_info *gl_info, DWORD dst_location) { - surface_prepare_system_memory(surface); - if (surface->flags & (SFLAG_INRB_MULTISAMPLE | SFLAG_INRB_RESOLVED)) surface_load_location(surface, SFLAG_INTEXTURE); @@ -5023,6 +5024,7 @@ static HRESULT surface_load_texture(struct wined3d_surface *surface, { /* Performance warning... */ FIXME("Downloading RGB surface %p to reload it as sRGB.\n", surface); + surface_prepare_system_memory(surface); surface_load_location(surface, surface->map_binding); } } @@ -5032,6 +5034,7 @@ static HRESULT surface_load_texture(struct wined3d_surface *surface, { /* Performance warning... */ FIXME("Downloading sRGB surface %p to reload it as RGB.\n", surface); + surface_prepare_system_memory(surface); surface_load_location(surface, surface->map_binding); } } @@ -5040,6 +5043,7 @@ static HRESULT surface_load_texture(struct wined3d_surface *surface, { WARN("Trying to load a texture from sysmem, but SFLAG_INSYSMEM is not set.\n"); /* Lets hope we get it from somewhere... */ + surface_prepare_system_memory(surface); surface_load_location(surface, SFLAG_INSYSMEM); } diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index f06db6d..090290f 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -735,6 +735,7 @@ static void texture2d_sub_resource_add_dirty_region(struct wined3d_resource *sub { struct wined3d_surface *surface = surface_from_resource(sub_resource); + surface_prepare_system_memory(surface); surface_load_location(surface, surface->map_binding); surface_invalidate_location(surface, ~surface->map_binding); } diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index a57e11b..ce1711b 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2273,6 +2273,7 @@ HRESULT surface_upload_from_surface(struct wined3d_surface *dst_surface, const P void surface_validate_location(struct wined3d_surface *surface, DWORD location) DECLSPEC_HIDDEN; HRESULT CDECL wined3d_surface_create(struct wined3d_texture *container, const struct wined3d_resource_desc *desc, DWORD flags, struct wined3d_surface **surface) DECLSPEC_HIDDEN; +void surface_prepare_system_memory(struct wined3d_surface *surface) DECLSPEC_HIDDEN; void get_drawable_size_swapchain(const struct wined3d_context *context, UINT *width, UINT *height) DECLSPEC_HIDDEN; void get_drawable_size_backbuffer(const struct wined3d_context *context, UINT *width, UINT *height) DECLSPEC_HIDDEN;

1 0

Stefan Dösinger : wined3d: Start to split up surface_prepare_system_memory.
by Alexandre Julliard 13 Jan '14

13 Jan '14

Module: wine Branch: master Commit: e2c0bc63cb95b83386b192e6e130abbc9a0c8a8f URL: http://source.winehq.org/git/wine.git/?a=commit;h=e2c0bc63cb95b83386b192e6e… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon Jan 13 15:21:02 2014 +0100 wined3d: Start to split up surface_prepare_system_memory. --- dlls/wined3d/surface.c | 35 ++++++++++++++++++++++++++--------- dlls/wined3d/texture.c | 2 +- dlls/wined3d/wined3d_private.h | 2 +- 3 files changed, 28 insertions(+), 11 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 5ac2241..a3c3ebf 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -589,7 +589,7 @@ static void surface_create_pbo(struct wined3d_surface *surface, const struct win context_release(context); } -void surface_prepare_system_memory(struct wined3d_surface *surface) +static void surface_prepare_system_memory(struct wined3d_surface *surface) { const struct wined3d_gl_info *gl_info = &surface->resource.device->adapter->gl_info; @@ -597,8 +597,7 @@ void surface_prepare_system_memory(struct wined3d_surface *surface) if (!(surface->flags & SFLAG_PBO) && surface_need_pbo(surface, gl_info)) surface_create_pbo(surface, gl_info); - else if (!(surface->resource.allocatedMemory || surface->flags & SFLAG_PBO - || surface->user_memory)) + else if (!(surface->resource.allocatedMemory || surface->flags & SFLAG_PBO)) { /* Whatever surface we have, make sure that there is memory allocated * for the downloaded copy, or a PBO to map. */ @@ -611,6 +610,24 @@ void surface_prepare_system_memory(struct wined3d_surface *surface) } } +void surface_prepare_map_memory(struct wined3d_surface *surface) +{ + switch (surface->map_binding) + { + case SFLAG_INUSERMEM: + if (!surface->user_memory) + ERR("Map binding is set to SFLAG_INUSERMEM but surface->user_memory is NULL.\n"); + break; + + case SFLAG_INSYSMEM: + surface_prepare_system_memory(surface); + break; + + default: + ERR("Unexpected map binding %s.\n", debug_surflocation(surface->map_binding)); + } +} + static void surface_evict_sysmem(struct wined3d_surface *surface) { if (surface->resource.map_count || (surface->flags & SFLAG_DONOTFREE) @@ -767,7 +784,7 @@ static void surface_realize_palette(struct wined3d_surface *surface) if (!(surface->flags & surface->map_binding)) { TRACE("Palette changed with surface that does not have an up to date system memory copy.\n"); - surface_prepare_system_memory(surface); + surface_prepare_map_memory(surface); surface_load_location(surface, surface->map_binding); } surface_invalidate_location(surface, ~surface->map_binding); @@ -803,7 +820,7 @@ static BYTE *surface_map(struct wined3d_surface *surface, const RECT *rect, DWOR TRACE("surface %p, rect %s, flags %#x.\n", surface, wine_dbgstr_rect(rect), flags); - surface_prepare_system_memory(surface); + surface_prepare_map_memory(surface); if (flags & WINED3D_MAP_DISCARD) { TRACE("WINED3D_MAP_DISCARD flag passed, marking SYSMEM as up to date.\n"); @@ -1392,7 +1409,7 @@ static void surface_unload(struct wined3d_resource *resource) } else { - surface_prepare_system_memory(surface); + surface_prepare_map_memory(surface); surface_load_location(surface, surface->map_binding); } surface_invalidate_location(surface, ~surface->map_binding); @@ -2323,7 +2340,7 @@ void surface_load(struct wined3d_surface *surface, BOOL srgb) /* To perform the color key conversion we need a sysmem copy of * the surface. Make sure we have it. */ - surface_prepare_system_memory(surface); + surface_prepare_map_memory(surface); surface_load_location(surface, surface->map_binding); surface_invalidate_location(surface, ~surface->map_binding); /* Switching color keying on / off may change the internal format. */ @@ -5024,7 +5041,7 @@ static HRESULT surface_load_texture(struct wined3d_surface *surface, { /* Performance warning... */ FIXME("Downloading RGB surface %p to reload it as sRGB.\n", surface); - surface_prepare_system_memory(surface); + surface_prepare_map_memory(surface); surface_load_location(surface, surface->map_binding); } } @@ -5034,7 +5051,7 @@ static HRESULT surface_load_texture(struct wined3d_surface *surface, { /* Performance warning... */ FIXME("Downloading sRGB surface %p to reload it as RGB.\n", surface); - surface_prepare_system_memory(surface); + surface_prepare_map_memory(surface); surface_load_location(surface, surface->map_binding); } } diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index 090290f..897af19 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -735,7 +735,7 @@ static void texture2d_sub_resource_add_dirty_region(struct wined3d_resource *sub { struct wined3d_surface *surface = surface_from_resource(sub_resource); - surface_prepare_system_memory(surface); + surface_prepare_map_memory(surface); surface_load_location(surface, surface->map_binding); surface_invalidate_location(surface, ~surface->map_binding); } diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index ce1711b..985f86f 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2273,7 +2273,7 @@ HRESULT surface_upload_from_surface(struct wined3d_surface *dst_surface, const P void surface_validate_location(struct wined3d_surface *surface, DWORD location) DECLSPEC_HIDDEN; HRESULT CDECL wined3d_surface_create(struct wined3d_texture *container, const struct wined3d_resource_desc *desc, DWORD flags, struct wined3d_surface **surface) DECLSPEC_HIDDEN; -void surface_prepare_system_memory(struct wined3d_surface *surface) DECLSPEC_HIDDEN; +void surface_prepare_map_memory(struct wined3d_surface *surface) DECLSPEC_HIDDEN; void get_drawable_size_swapchain(const struct wined3d_context *context, UINT *width, UINT *height) DECLSPEC_HIDDEN; void get_drawable_size_backbuffer(const struct wined3d_context *context, UINT *width, UINT *height) DECLSPEC_HIDDEN;

1 0