July 2006 - Wine-commits - list.winehq.org

Andrew Talbot : winspool.drv/tests: Write-strings warnings fix.
by Alexandre Julliard 13 Jul '06

13 Jul '06

Module: wine Branch: refs/heads/master Commit: cc318be2098f674f641604abcb9e62b6568dc310 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=cc318be2098f674f641604a… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Wed Jul 12 22:31:46 2006 +0100 winspool.drv/tests: Write-strings warnings fix. --- dlls/winspool.drv/tests/info.c | 17 ++++++++++------- 1 files changed, 10 insertions(+), 7 deletions(-) diff --git a/dlls/winspool.drv/tests/info.c b/dlls/winspool.drv/tests/info.c index 2eff80e..0420464 100644 --- a/dlls/winspool.drv/tests/info.c +++ b/dlls/winspool.drv/tests/info.c @@ -176,6 +176,9 @@ static void test_AddMonitor(void) { MONITOR_INFO_2A mi2a; struct monitor_entry * entry = NULL; + static CHAR empty[] = "", + does_not_exist_dll[] = "does_not_exists.dll", + version_dll[] = "version.dll"; DWORD res; entry = find_installed_monitor(); @@ -238,7 +241,7 @@ #if 0 #endif mi2a.pEnvironment = entry->env; - mi2a.pName = ""; + mi2a.pName = empty; SetLastError(MAGIC_DEAD); res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a); /* NT: ERROR_INVALID_PARAMETER, 9x: ERROR_PRIVILEGE_NOT_HELD */ @@ -260,15 +263,14 @@ #endif "ERROR_PRIVILEGE_NOT_HELD)\n", res, GetLastError()); - mi2a.pDLLName = ""; + mi2a.pDLLName = empty; SetLastError(MAGIC_DEAD); res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a); ok( !res && (GetLastError() == ERROR_INVALID_PARAMETER), "returned %ld with %ld (expected '0' with ERROR_INVALID_PARAMETER)\n", res, GetLastError()); - - mi2a.pDLLName = "does_not_exists.dll"; + mi2a.pDLLName = does_not_exist_dll; SetLastError(MAGIC_DEAD); res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a); /* NT: ERROR_MOD_NOT_FOUND, 9x: ERROR_INVALID_PARAMETER */ @@ -278,7 +280,7 @@ #endif "returned %ld with %ld (expected '0' with: ERROR_MOD_NOT_FOUND or " \ "ERROR_INVALID_PARAMETER)\n", res, GetLastError()); - mi2a.pDLLName = "version.dll"; + mi2a.pDLLName = version_dll; SetLastError(MAGIC_DEAD); res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a); /* NT: ERROR_PROC_NOT_FOUND, 9x: ERROR_INVALID_PARAMETER */ @@ -296,7 +298,7 @@ #endif SetLastError(MAGIC_DEAD); res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a); ok(res, "returned %ld with %ld (expected '!= 0')\n", res, GetLastError()); - + /* add a monitor twice */ SetLastError(MAGIC_DEAD); res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a); @@ -975,6 +977,7 @@ static void test_OpenPrinter(void) DWORD res; DWORD size; CHAR buffer[DEFAULT_PRINTER_SIZE]; + static CHAR empty[] = ""; LPSTR ptr; SetLastError(MAGIC_DEAD); @@ -1095,7 +1098,7 @@ static void test_OpenPrinter(void) "ERROR_ACCESS_DENIED)\n", res, GetLastError()); if(res) ClosePrinter(hprinter); - defaults.pDatatype=""; + defaults.pDatatype = empty; hprinter = (HANDLE) MAGIC_DEAD; SetLastError(MAGIC_DEAD);

1 0

Thomas Kho : notepad: Change window title to be like Windows' notepad.exe.
by Alexandre Julliard 13 Jul '06

13 Jul '06

Module: wine Branch: refs/heads/master Commit: 3ec165fd990d1d62bfa5dacb30d2592285c4917f URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=3ec165fd990d1d62bfa5dac… Author: Thomas Kho <tkho(a)ucla.edu> Date: Wed Jul 12 14:15:31 2006 -0700 notepad: Change window title to be like Windows' notepad.exe. --- programs/notepad/En.rc | 2 +- programs/notepad/dialog.c | 29 +++++++++++------------------ 2 files changed, 12 insertions(+), 19 deletions(-) diff --git a/programs/notepad/En.rc b/programs/notepad/En.rc index 3bf6475..af60994 100644 --- a/programs/notepad/En.rc +++ b/programs/notepad/En.rc @@ -108,7 +108,7 @@ STRING_ERROR, STRING_WARNING, "WARNING" STRING_INFO, "Information" -STRING_UNTITLED, "(untitled)" +STRING_UNTITLED, "Untitled" STRING_ALL_FILES, "All files (*.*)" STRING_TEXT_FILES_TXT, "Text files (*.txt)" diff --git a/programs/notepad/dialog.c b/programs/notepad/dialog.c index c487164..1488af0 100644 --- a/programs/notepad/dialog.c +++ b/programs/notepad/dialog.c @@ -54,30 +54,23 @@ VOID ShowLastError(void) /** * Sets the caption of the main window according to Globals.szFileTitle: - * Notepad - (untitled) if no file is open - * Notepad - [filename] if a file is given + * Untitled - Notepad if no file is open + * filename - Notepad if a file is given */ static void UpdateWindowCaption(void) { WCHAR szCaption[MAX_STRING_LEN]; - WCHAR szUntitled[MAX_STRING_LEN]; + WCHAR szNotepad[MAX_STRING_LEN]; + static const WCHAR hyphenW[] = { ' ','-',' ',0 }; - LoadString(Globals.hInstance, STRING_NOTEPAD, szCaption, SIZEOF(szCaption)); - - if (Globals.szFileTitle[0] != '\0') { - static const WCHAR bracket_lW[] = { ' ','-',' ','[',0 }; - static const WCHAR bracket_rW[] = { ']',0 }; - lstrcat(szCaption, bracket_lW); - lstrcat(szCaption, Globals.szFileTitle); - lstrcat(szCaption, bracket_rW); - } + if (Globals.szFileTitle[0] != '\0') + lstrcpy(szCaption, Globals.szFileTitle); else - { - static const WCHAR hyphenW[] = { ' ','-',' ',0 }; - LoadString(Globals.hInstance, STRING_UNTITLED, szUntitled, SIZEOF(szUntitled)); - lstrcat(szCaption, hyphenW); - lstrcat(szCaption, szUntitled); - } + LoadString(Globals.hInstance, STRING_UNTITLED, szCaption, SIZEOF(szCaption)); + + LoadString(Globals.hInstance, STRING_NOTEPAD, szNotepad, SIZEOF(szNotepad)); + lstrcat(szCaption, hyphenW); + lstrcat(szCaption, szNotepad); SetWindowText(Globals.hMainWnd, szCaption); }

1 0

Andrew Talbot : mshtml: Write-strings warnings fix.
by Alexandre Julliard 13 Jul '06

13 Jul '06

Module: wine Branch: refs/heads/master Commit: f7e38ac6ea597dc2045239f6f4bcb3bfcbd53a55 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=f7e38ac6ea597dc2045239f… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Wed Jul 12 21:26:49 2006 +0100 mshtml: Write-strings warnings fix. --- dlls/mshtml/main.c | 16 +++++++++++----- 1 files changed, 11 insertions(+), 5 deletions(-) diff --git a/dlls/mshtml/main.c b/dlls/mshtml/main.c index 58098d0..d4c4bf7 100644 --- a/dlls/mshtml/main.c +++ b/dlls/mshtml/main.c @@ -252,9 +252,14 @@ DEFINE_GUID(CLSID_MHTMLDocument, 0x3050F DEFINE_GUID(CLSID_Scriptlet, 0xAE24FDAE, 0x03C6, 0x11D1, 0x8B,0x76, 0x00,0x80,0xC7,0x44,0xF3,0x89); DEFINE_GUID(CLSID_TridentAPI, 0x429AF92C, 0xA51F, 0x11D2, 0x86,0x1E, 0x00,0xC0,0x4F,0xA3,0x5C,0x89); -#define INF_SET_ID(id) \ - pse[i].pszName = #id; \ - clsids[i++] = &id; +#define INF_SET_ID(id) \ + do \ + { \ + static CHAR name[] = #id; \ + \ + pse[i].pszName = name; \ + clsids[i++] = &id; \ + } while (0) #define INF_SET_CLSID(clsid) INF_SET_ID(CLSID_ ## clsid) @@ -267,11 +272,11 @@ static HRESULT register_server(BOOL do_r STRENTRYA pse[35]; static CLSID const *clsids[35]; int i = 0; - + static const WCHAR wszAdvpack[] = {'a','d','v','p','a','c','k','.','d','l','l',0}; TRACE("(%x)\n", do_register); - + INF_SET_CLSID(AboutProtocol); INF_SET_CLSID(CAnchorBrowsePropertyPage); INF_SET_CLSID(CBackgroundPropertyPage); @@ -331,6 +336,7 @@ static HRESULT register_server(BOOL do_r } #undef INF_SET_CLSID +#undef INF_SET_ID /*********************************************************************** * DllRegisterServer (MSHTML.@)

1 0

Andrew Ziem : user: Implement Ascii version of SPI_GETDESKWALLPAPER.
by Alexandre Julliard 12 Jul '06

12 Jul '06

Module: wine Branch: refs/heads/master Commit: c142bd91156082cf2efb413b347f356b1f287d1a URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=c142bd91156082cf2efb413… Author: Andrew Ziem <ahziem1(a)mailbolt.com> Date: Tue Jul 11 20:22:38 2006 -0600 user: Implement Ascii version of SPI_GETDESKWALLPAPER. --- dlls/user/sysparams.c | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/dlls/user/sysparams.c b/dlls/user/sysparams.c index 5fed316..de0677f 100644 --- a/dlls/user/sysparams.c +++ b/dlls/user/sysparams.c @@ -2496,6 +2496,14 @@ BOOL WINAPI SystemParametersInfoA( UINT break; } + case SPI_GETDESKWALLPAPER: /* 115 */ + { + WCHAR buffer[MAX_PATH]; + ret = (SystemParametersInfoW( SPI_GETDESKWALLPAPER, uiParam, buffer, fuWinIni ) && + WideCharToMultiByte(CP_ACP, 0, buffer, -1, pvParam, uiParam, NULL, NULL)); + break; + } + default: ret = SystemParametersInfoW( uiAction, uiParam, pvParam, fuWinIni ); break;

1 0

Ge van Geldorp : user32: Implement Get/SetClassLongPtrA/W.
by Alexandre Julliard 12 Jul '06

12 Jul '06

Module: wine Branch: refs/heads/master Commit: 54afeb02de187d4187420adf24652df7330f4a21 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=54afeb02de187d4187420ad… Author: Ge van Geldorp <ge(a)gse.nl> Date: Mon Jul 10 08:59:22 2006 +0200 user32: Implement Get/SetClassLongPtrA/W. --- dlls/user/class.c | 214 ++++++++++++++++++++++------------------ include/wine/server_protocol.h | 6 + server/protocol.def | 4 - server/trace.c | 4 - 4 files changed, 127 insertions(+), 101 deletions(-) Diff: http://source.winehq.org/git/?p=wine.git;a=commitdiff;h=54afeb02de187d41874…

1 0

James Hawkins : urlmon: Implement URLDownloadToCacheFileW.
by Alexandre Julliard 12 Jul '06

12 Jul '06

Module: wine Branch: refs/heads/master Commit: e2bf4ff16449d4c90d2a9fb662dc0f506d7bcbbc URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=e2bf4ff16449d4c90d2a9fb… Author: James Hawkins <truiken(a)gmail.com> Date: Tue Jul 11 13:27:59 2006 -0700 urlmon: Implement URLDownloadToCacheFileW. --- dlls/urlmon/umon.c | 43 ++++++++++++++++++++++++++++++++++++++++--- 1 files changed, 40 insertions(+), 3 deletions(-) diff --git a/dlls/urlmon/umon.c b/dlls/urlmon/umon.c index 88304ca..b1ca78e 100644 --- a/dlls/urlmon/umon.c +++ b/dlls/urlmon/umon.c @@ -1611,9 +1611,46 @@ HRESULT WINAPI URLDownloadToCacheFileA(L HRESULT WINAPI URLDownloadToCacheFileW(LPUNKNOWN lpUnkCaller, LPCWSTR szURL, LPWSTR szFileName, DWORD dwBufLength, DWORD dwReserved, LPBINDSTATUSCALLBACK pBSC) { - FIXME("(%p %s %p %ld %ld %p)\n", lpUnkCaller, debugstr_w(szURL), szFileName, - dwBufLength, dwReserved, pBSC); - return E_NOTIMPL; + WCHAR cache_path[MAX_PATH + 1]; + FILETIME expire, modified; + HRESULT hr; + LPWSTR ext; + + static const WCHAR header[] = { + 'H','T','T','P','/','1','.','0',' ','2','0','0',' ', + 'O','K','\\','r','\\','n','\\','r','\\','n',0 + }; + + TRACE("(%p, %s, %p, %ld, %ld, %p)\n", lpUnkCaller, debugstr_w(szURL), + szFileName, dwBufLength, dwReserved, pBSC); + + if (!szURL || !szFileName) + return E_INVALIDARG; + + ext = PathFindExtensionW(szURL); + + if (!CreateUrlCacheEntryW(szURL, 0, ext, cache_path, 0)) + return E_FAIL; + + hr = URLDownloadToFileW(lpUnkCaller, szURL, cache_path, 0, pBSC); + if (FAILED(hr)) + return hr; + + expire.dwHighDateTime = 0; + expire.dwLowDateTime = 0; + modified.dwHighDateTime = 0; + modified.dwLowDateTime = 0; + + if (!CommitUrlCacheEntryW(szURL, cache_path, expire, modified, NORMAL_CACHE_ENTRY, + (LPWSTR)header, sizeof(header), NULL, NULL)) + return E_FAIL; + + if (lstrlenW(cache_path) > dwBufLength) + return E_OUTOFMEMORY; + + lstrcpyW(szFileName, cache_path); + + return S_OK; } /***********************************************************************

1 0

Mike McCormack : ole32: Remove unnecessary assert(This) calls.
by Alexandre Julliard 12 Jul '06

12 Jul '06

Module: wine Branch: refs/heads/master Commit: 1a80cb87cb0378fd5341c7db1a55b994df20b6d1 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=1a80cb87cb0378fd5341c7d… Author: Mike McCormack <mike(a)codeweavers.com> Date: Wed Jul 12 18:24:52 2006 +0900 ole32: Remove unnecessary assert(This) calls. --- dlls/ole32/stg_prop.c | 18 +----------------- 1 files changed, 1 insertions(+), 17 deletions(-) diff --git a/dlls/ole32/stg_prop.c b/dlls/ole32/stg_prop.c index 64a825d..b8242ec 100644 --- a/dlls/ole32/stg_prop.c +++ b/dlls/ole32/stg_prop.c @@ -245,7 +245,6 @@ static PROPVARIANT *PropertyStorage_Find { PROPVARIANT *ret = NULL; - assert(This); dictionary_find(This->propid_to_prop, (void *)propid, (void **)&ret); TRACE("returning %p\n", ret); return ret; @@ -258,7 +257,6 @@ static PROPVARIANT *PropertyStorage_Find PROPVARIANT *ret = NULL; PROPID propid; - assert(This); if (!name) return NULL; if (This->codePage == CP_UNICODE) @@ -289,7 +287,6 @@ static LPWSTR PropertyStorage_FindProper { LPWSTR ret = NULL; - assert(This); dictionary_find(This->propid_to_name, (void *)propid, (void **)&ret); TRACE("returning %p\n", ret); return ret; @@ -468,7 +465,6 @@ static HRESULT PropertyStorage_StoreProp HRESULT hr = S_OK; PROPVARIANT *prop = PropertyStorage_FindProperty(This, propid); - assert(This); assert(propvar); if (propvar->vt & VT_BYREF || propvar->vt & VT_ARRAY) This->format = 1; @@ -1002,7 +998,6 @@ static HRESULT PropertyStorage_ReadDicti DWORD numEntries, i; HRESULT hr = S_OK; - assert(This); assert(This->name_to_propid); assert(This->propid_to_name); @@ -1272,7 +1267,6 @@ static HRESULT PropertyStorage_ReadFromS ULONG count = 0; DWORD dictOffset = 0; - assert(This); This->dirty = FALSE; This->highestProp = 0; hr = IStream_Stat(This->stm, &stat, STATFLAG_NONAME); @@ -1389,6 +1383,7 @@ static HRESULT PropertyStorage_ReadFromS { PROPVARIANT prop; + memset(&prop, 0, sizeof prop); if (SUCCEEDED(PropertyStorage_ReadProperty(This, &prop, buf + idOffset->dwOffset - sizeof(PROPERTYSECTIONHEADER)))) { @@ -1452,7 +1447,6 @@ end: static void PropertyStorage_MakeHeader(PropertyStorage_impl *This, PROPERTYSETHEADER *hdr) { - assert(This); assert(hdr); StorageUtl_WriteWord((BYTE *)&hdr->wByteOrder, 0, PROPSETHDR_BYTEORDER_MAGIC); @@ -1465,7 +1459,6 @@ static void PropertyStorage_MakeHeader(P static void PropertyStorage_MakeFmtIdOffset(PropertyStorage_impl *This, FORMATIDOFFSET *fmtOffset) { - assert(This); assert(fmtOffset); StorageUtl_WriteGUID((BYTE *)fmtOffset, 0, &This->fmtid); StorageUtl_WriteDWord((BYTE *)fmtOffset, offsetof(FORMATIDOFFSET, dwOffset), @@ -1505,7 +1498,6 @@ static BOOL PropertyStorage_DictionaryWr ULONG count; assert(key); - assert(This); assert(closure); StorageUtl_WriteDWord((LPBYTE)&propid, 0, (DWORD)value); c->hr = IStream_Write(This->stm, &propid, sizeof(propid), &count); @@ -1573,7 +1565,6 @@ static HRESULT PropertyStorage_WriteDict DWORD dwTemp; struct DictionaryClosure closure; - assert(This); assert(sectionOffset); /* The dictionary's always the first property written, so seek to its @@ -1628,7 +1619,6 @@ static HRESULT PropertyStorage_WriteProp ULONG count; DWORD dwType, bytesWritten; - assert(This); assert(var); assert(sectionOffset); @@ -1789,7 +1779,6 @@ static HRESULT PropertyStorage_WriteProp { struct PropertyClosure closure; - assert(This); assert(sectionOffset); closure.hr = S_OK; closure.propNum = startingPropNum; @@ -1807,7 +1796,6 @@ static HRESULT PropertyStorage_WriteHead PROPERTYSETHEADER hdr; FORMATIDOFFSET fmtOffset; - assert(This); hr = IStream_Seek(This->stm, seek, STREAM_SEEK_SET, NULL); if (FAILED(hr)) goto end; @@ -1845,8 +1833,6 @@ static HRESULT PropertyStorage_WriteToSt DWORD numProps, prop, sectionOffset, dwTemp; PROPVARIANT var; - assert(This); - PropertyStorage_WriteHeadersToStream(This); /* Count properties. Always at least one property, the code page */ @@ -1933,7 +1919,6 @@ end: */ static void PropertyStorage_DestroyDictionaries(PropertyStorage_impl *This) { - assert(This); dictionary_destroy(This->name_to_propid); This->name_to_propid = NULL; dictionary_destroy(This->propid_to_name); @@ -1946,7 +1931,6 @@ static HRESULT PropertyStorage_CreateDic { HRESULT hr = S_OK; - assert(This); This->name_to_propid = dictionary_create( PropertyStorage_PropNameCompare, PropertyStorage_PropNameDestroy, This);

1 0

Mike McCormack : ole32: Small cleanups.
by Alexandre Julliard 12 Jul '06

12 Jul '06

Module: wine Branch: refs/heads/master Commit: 49ff182fc410ba333a5c61ff3cc0da61fc4d69da URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=49ff182fc410ba333a5c61f… Author: Mike McCormack <mike(a)codeweavers.com> Date: Wed Jul 12 18:25:16 2006 +0900 ole32: Small cleanups. --- dlls/ole32/clipboard.c | 21 ++++++++------------- 1 files changed, 8 insertions(+), 13 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 8445ed4..621bbe5 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -80,11 +80,6 @@ #include "storage32.h" #define HANDLE_ERROR(err) { hr = err; TRACE("(HRESULT=%lx)\n", (HRESULT)err); goto CLEANUP; } -/* For CoGetMalloc (MEMCTX_TASK is currently ignored) */ -#ifndef MEMCTX_TASK -# define MEMCTX_TASK -1 -#endif - WINE_DEFAULT_DEBUG_CHANNEL(ole); /**************************************************************************** @@ -178,7 +173,7 @@ static OLEClipbrd* OLEClipbrd_Construct( static void OLEClipbrd_Destroy(OLEClipbrd* ptrToDestroy); static HWND OLEClipbrd_CreateWindow(void); static void OLEClipbrd_DestroyWindow(HWND hwnd); -LRESULT CALLBACK OLEClipbrd_WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam); +static LRESULT CALLBACK OLEClipbrd_WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam); static HRESULT OLEClipbrd_RenderFormat( IDataObject *pIDataObject, LPFORMATETC pFormatetc ); static HGLOBAL OLEClipbrd_GlobalDupMem( HGLOBAL hGlobalSrc ); @@ -283,7 +278,7 @@ static const IEnumFORMATETCVtbl efvt = /* * Name of our registered OLE clipboard window class */ -CHAR OLEClipbrd_WNDCLASS[] = "CLIPBRDWNDCLASS"; +static const CHAR OLEClipbrd_WNDCLASS[] = "CLIPBRDWNDCLASS"; /* * If we need to store state info we can store it here. @@ -487,7 +482,7 @@ HRESULT WINAPI OleGetClipboard(IDataObje * data object exposed through OleGetClipboard must convert this TYMED_HGLOBAL * back to TYMED_IStorage. */ -HRESULT WINAPI OleFlushClipboard() +HRESULT WINAPI OleFlushClipboard(void) { IEnumFORMATETC* penumFormatetc = NULL; FORMATETC rgelt; @@ -577,7 +572,7 @@ CLEANUP: /*********************************************************************** * OleIsCurrentClipboard [OLE32.@] */ -HRESULT WINAPI OleIsCurrentClipboard ( IDataObject *pDataObject) +HRESULT WINAPI OleIsCurrentClipboard(IDataObject *pDataObject) { TRACE("()\n"); /* @@ -638,7 +633,7 @@ void OLEClipbrd_UnInitialize(void) /********************************************************* * Construct the OLEClipbrd class. */ -static OLEClipbrd* OLEClipbrd_Construct() +static OLEClipbrd* OLEClipbrd_Construct(void) { OLEClipbrd* newObject = NULL; HGLOBAL hNewObject = 0; @@ -711,7 +706,7 @@ static void OLEClipbrd_Destroy(OLEClipbr * OLEClipbrd_CreateWindow() * Create the clipboard window */ -static HWND OLEClipbrd_CreateWindow() +static HWND OLEClipbrd_CreateWindow(void) { HWND hwnd = 0; WNDCLASSEXA wcex; @@ -776,7 +771,7 @@ static void OLEClipbrd_DestroyWindow(HWN * has been placed in the clipboard via OleSetClipboard(). * i.e. Only when OLE owns the windows clipboard. */ -LRESULT CALLBACK OLEClipbrd_WndProc +static LRESULT CALLBACK OLEClipbrd_WndProc (HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam) { switch (message) @@ -1559,7 +1554,7 @@ static HRESULT WINAPI OLEClipbrd_IDataOb * NOTE: this does not AddRef the interface. */ -LPENUMFORMATETC OLEClipbrd_IEnumFORMATETC_Construct(UINT cfmt, const FORMATETC afmt[], +static LPENUMFORMATETC OLEClipbrd_IEnumFORMATETC_Construct(UINT cfmt, const FORMATETC afmt[], LPUNKNOWN pUnkDataObj) { IEnumFORMATETCImpl* ef;

1 0

Mike McCormack : ole32: Only return S_FALSE in IPropertyStorage:: ReadMultiple if a property was not found.
by Alexandre Julliard 12 Jul '06

12 Jul '06

Module: wine Branch: refs/heads/master Commit: 6a15ac7f0a505e74c5e7c2a6d84e4d08012650ce URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=6a15ac7f0a505e74c5e7c2a… Author: Mike McCormack <mike(a)codeweavers.com> Date: Wed Jul 12 18:24:10 2006 +0900 ole32: Only return S_FALSE in IPropertyStorage::ReadMultiple if a property was not found. --- dlls/ole32/stg_prop.c | 4 +- dlls/ole32/tests/stg_prop.c | 114 ++++++++++++++++++++++--------------------- 2 files changed, 60 insertions(+), 58 deletions(-) Diff: http://source.winehq.org/git/?p=wine.git;a=commitdiff;h=6a15ac7f0a505e74c5e…

1 0

Juan Lang : msi: Split up ACTION_AppSearchReg.
by Alexandre Julliard 12 Jul '06

12 Jul '06

Module: wine Branch: refs/heads/master Commit: ab24569b85271ea0b0d14b7f15c5821cdaa9b40c URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=ab24569b85271ea0b0d14b7… Author: Juan Lang <juan_lang(a)yahoo.com> Date: Wed Jul 12 09:55:13 2006 -0700 msi: Split up ACTION_AppSearchReg. --- dlls/msi/appsearch.c | 94 +++++++++++++++++++++++++++----------------------- 1 files changed, 51 insertions(+), 43 deletions(-) diff --git a/dlls/msi/appsearch.c b/dlls/msi/appsearch.c index 6f173dc..9a5bec2 100644 --- a/dlls/msi/appsearch.c +++ b/dlls/msi/appsearch.c @@ -236,6 +236,54 @@ end: return rc; } +static void ACTION_ConvertRegValue(DWORD regType, const BYTE *value, DWORD sz, + LPWSTR *appValue) +{ + static const WCHAR dwordFmt[] = { '#','%','d','\0' }; + static const WCHAR expandSzFmt[] = { '#','%','%','%','s','\0' }; + static const WCHAR binFmt[] = { '#','x','%','x','\0' }; + DWORD i; + + switch (regType) + { + case REG_SZ: + if (*(LPWSTR)value == '#') + { + /* escape leading pound with another */ + *appValue = msi_alloc(sz + sizeof(WCHAR)); + (*appValue)[0] = '#'; + strcpyW(*appValue + 1, (LPCWSTR)value); + } + else + { + *appValue = msi_alloc(sz); + strcpyW(*appValue, (LPCWSTR)value); + } + break; + case REG_DWORD: + /* 7 chars for digits, 1 for NULL, 1 for #, and 1 for sign + * char if needed + */ + *appValue = msi_alloc(10 * sizeof(WCHAR)); + sprintfW(*appValue, dwordFmt, *(const DWORD *)value); + break; + case REG_EXPAND_SZ: + /* space for extra #% characters in front */ + *appValue = msi_alloc(sz + 2 * sizeof(WCHAR)); + sprintfW(*appValue, expandSzFmt, (LPCWSTR)value); + break; + case REG_BINARY: + /* 3 == length of "#x<nibble>" */ + *appValue = msi_alloc((sz * 3 + 1) * sizeof(WCHAR)); + for (i = 0; i < sz; i++) + sprintfW(*appValue + i * 3, binFmt, value[i]); + break; + default: + WARN("unimplemented for values of type %ld\n", regType); + *appValue = NULL; + } +} + static UINT ACTION_AppSearchReg(MSIPACKAGE *package, BOOL *appFound, MSISIGNATURE *sig) { @@ -247,9 +295,6 @@ static UINT ACTION_AppSearchReg(MSIPACKA 'R','e','g','L','o','c','a','t','o','r',' ', 'w','h','e','r','e',' ','S','i','g','n','a','t','u','r','e','_',' ','=',' ', '\'','%','s','\'',0}; - static const WCHAR dwordFmt[] = { '#','%','d','\0' }; - static const WCHAR expandSzFmt[] = { '#','%','%','%','s','\0' }; - static const WCHAR binFmt[] = { '#','x','%','x','\0' }; TRACE("(package %p, appFound %p, sig %p)\n", package, appFound, sig); *appFound = FALSE; @@ -260,7 +305,7 @@ static UINT ACTION_AppSearchReg(MSIPACKA LPWSTR keyPath = NULL, valueName = NULL, propertyValue = NULL; int root, type; HKEY rootKey, key = NULL; - DWORD sz = 0, regType, i; + DWORD sz = 0, regType; LPBYTE value = NULL; rc = MSI_ViewExecute(view, 0); @@ -342,45 +387,8 @@ static UINT ACTION_AppSearchReg(MSIPACKA rc = ERROR_SUCCESS; goto end; } - - switch (regType) - { - case REG_SZ: - if (*(LPWSTR)value == '#') - { - /* escape leading pound with another */ - propertyValue = msi_alloc( sz + sizeof(WCHAR)); - propertyValue[0] = '#'; - strcpyW(propertyValue + 1, (LPWSTR)value); - } - else - { - propertyValue = msi_alloc( sz); - strcpyW(propertyValue, (LPWSTR)value); - } - break; - case REG_DWORD: - /* 7 chars for digits, 1 for NULL, 1 for #, and 1 for sign - * char if needed - */ - propertyValue = msi_alloc( 10 * sizeof(WCHAR)); - sprintfW(propertyValue, dwordFmt, *(DWORD *)value); - break; - case REG_EXPAND_SZ: - /* space for extra #% characters in front */ - propertyValue = msi_alloc( sz + 2 * sizeof(WCHAR)); - sprintfW(propertyValue, expandSzFmt, (LPWSTR)value); - break; - case REG_BINARY: - /* 3 == length of "#x<nibble>" */ - propertyValue = msi_alloc( (sz * 3 + 1) * sizeof(WCHAR)); - for (i = 0; i < sz; i++) - sprintfW(propertyValue + i * 3, binFmt, value[i]); - break; - default: - WARN("unimplemented for values of type %ld\n", regType); - goto end; - } + + ACTION_ConvertRegValue(regType, value, sz, &propertyValue); TRACE("found registry value, setting %s to %s\n", debugstr_w(sig->Property), debugstr_w(propertyValue));

1 0