November 2012 - Wine-commits

Henri Verbeet : d3drm: Avoid LPDIRECT3DVIEWPORT.
by Alexandre Julliard 21 Nov '12

21 Nov '12

Module: wine Branch: master Commit: 0ac2b823b5f52dfd2dac4e8b362629ddb7c0e7b0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0ac2b823b5f52dfd2dac4e8b3… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Nov 20 22:40:18 2012 +0100 d3drm: Avoid LPDIRECT3DVIEWPORT. --- dlls/d3drm/viewport.c | 8 ++++---- include/d3drmobj.h | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/d3drm/viewport.c b/dlls/d3drm/viewport.c index fb93dc6..75eb225 100644 --- a/dlls/d3drm/viewport.c +++ b/dlls/d3drm/viewport.c @@ -448,8 +448,8 @@ static D3DRMPROJECTIONTYPE WINAPI IDirect3DRMViewportImpl_GetProjection(IDirect3 return IDirect3DRMViewport2_GetProjection(&This->IDirect3DRMViewport2_iface); } -static HRESULT WINAPI IDirect3DRMViewportImpl_GetDirect3DViewport(IDirect3DRMViewport* iface, - LPDIRECT3DVIEWPORT * viewport) +static HRESULT WINAPI IDirect3DRMViewportImpl_GetDirect3DViewport(IDirect3DRMViewport *iface, + IDirect3DViewport **viewport) { IDirect3DRMViewportImpl *This = impl_from_IDirect3DRMViewport(iface); @@ -883,8 +883,8 @@ static D3DRMPROJECTIONTYPE WINAPI IDirect3DRMViewport2Impl_GetProjection(IDirect return This->projection; } -static HRESULT WINAPI IDirect3DRMViewport2Impl_GetDirect3DViewport(IDirect3DRMViewport2* iface, - LPDIRECT3DVIEWPORT * viewport) +static HRESULT WINAPI IDirect3DRMViewport2Impl_GetDirect3DViewport(IDirect3DRMViewport2 *iface, + IDirect3DViewport **viewport) { IDirect3DRMViewportImpl *This = impl_from_IDirect3DRMViewport2(iface); diff --git a/include/d3drmobj.h b/include/d3drmobj.h index 3bafd9d..2265b56 100644 --- a/include/d3drmobj.h +++ b/include/d3drmobj.h @@ -835,7 +835,7 @@ DECLARE_INTERFACE_(IDirect3DRMViewport,IDirect3DRMObject) STDMETHOD_(D3DVALUE, GetBack)(THIS) PURE; STDMETHOD_(D3DVALUE, GetFront)(THIS) PURE; STDMETHOD_(D3DRMPROJECTIONTYPE, GetProjection)(THIS) PURE; - STDMETHOD(GetDirect3DViewport)(THIS_ LPDIRECT3DVIEWPORT *) PURE; + STDMETHOD(GetDirect3DViewport)(THIS_ IDirect3DViewport **viewport) PURE; }; #undef INTERFACE @@ -975,7 +975,7 @@ DECLARE_INTERFACE_(IDirect3DRMViewport2,IDirect3DRMObject) STDMETHOD_(D3DVALUE, GetBack)(THIS) PURE; STDMETHOD_(D3DVALUE, GetFront)(THIS) PURE; STDMETHOD_(D3DRMPROJECTIONTYPE, GetProjection)(THIS) PURE; - STDMETHOD(GetDirect3DViewport)(THIS_ LPDIRECT3DVIEWPORT *) PURE; + STDMETHOD(GetDirect3DViewport)(THIS_ IDirect3DViewport **viewport) PURE; STDMETHOD(TransformVectors)(THIS_ DWORD NumVectors, LPD3DRMVECTOR4D pDstVectors, LPD3DVECTOR pSrcVectors) PURE; STDMETHOD(InverseTransformVectors)(THIS_ DWORD NumVectors, LPD3DVECTOR pDstVectors,

1 0

Nikolay Sivov : oledb32: Implement IDataInitialize::GetInitializationString .
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: 92013d7bd263ff07db15d38d23cc01f1c4ee06fb URL: http://source.winehq.org/git/wine.git/?a=commit;h=92013d7bd263ff07db15d38d2… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Nov 20 12:40:28 2012 -0500 oledb32: Implement IDataInitialize::GetInitializationString. --- dlls/oledb32/datainit.c | 199 +++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 194 insertions(+), 5 deletions(-) diff --git a/dlls/oledb32/datainit.c b/dlls/oledb32/datainit.c index b990dd8..b53e08a 100644 --- a/dlls/oledb32/datainit.c +++ b/dlls/oledb32/datainit.c @@ -27,13 +27,15 @@ #include "ole2.h" #include "msdasc.h" #include "oledberr.h" - +#include "initguid.h" #include "oledb_private.h" #include "wine/debug.h" +#include "wine/unicode.h" WINE_DEFAULT_DEBUG_CHANNEL(oledb); +DEFINE_GUID(DBPROPSET_DBINIT, 0xc8b522bc, 0x5cf3, 0x11ce, 0xad, 0xe5, 0x00, 0xaa, 0x00, 0x44, 0x77, 0x3d); typedef struct datainit { @@ -288,14 +290,201 @@ static HRESULT WINAPI datainit_GetDataSource(IDataInitialize *iface, IUnknown *p return E_NOTIMPL; } -static HRESULT WINAPI datainit_GetInitializationString(IDataInitialize *iface, IUnknown *pDataSource, - boolean fIncludePassword, LPWSTR *ppwszInitString) +/* returns character length of string representation */ +static int get_propvalue_length(DBPROP *prop) +{ + VARIANT str; + HRESULT hr; + + if (V_VT(&prop->vValue) == VT_BSTR) return SysStringLen(V_BSTR(&prop->vValue)); + + VariantInit(&str); + hr = VariantChangeType(&str, &prop->vValue, 0, VT_BSTR); + if (hr == S_OK) + { + int len = SysStringLen(V_BSTR(&str)); + VariantClear(&str); + return len; + } + + return 0; +} + +static void write_propvalue_str(WCHAR *str, DBPROP *prop) +{ + VARIANT *v = &prop->vValue; + VARIANT vstr; + HRESULT hr; + + if (V_VT(v) == VT_BSTR) + { + strcatW(str, V_BSTR(v)); + return; + } + + VariantInit(&vstr); + hr = VariantChangeType(&vstr, v, VARIANT_ALPHABOOL, VT_BSTR); + if (hr == S_OK) + { + strcatW(str, V_BSTR(&vstr)); + VariantClear(&vstr); + } +} + +static WCHAR *get_propinfo_descr(DBPROP *prop, DBPROPINFOSET *propinfoset) { + int i; + + for (i = 0; i < propinfoset->cPropertyInfos; i++) + if (propinfoset->rgPropertyInfos[i].dwPropertyID == prop->dwPropertyID) + return propinfoset->rgPropertyInfos[i].pwszDescription; + + return NULL; +} + +static void free_dbpropset(ULONG count, DBPROPSET *propset) +{ + int i; + + for (i = 0; i < count; i++) + { + int p; + + for (p = 0; p < propset[i].cProperties; p++) + { + VariantClear(&propset[i].rgProperties[p].vValue); + CoTaskMemFree(&propset[i].rgProperties[p]); + } + CoTaskMemFree(&propset[i]); + } +} + +static void free_dbpropinfoset(ULONG count, DBPROPINFOSET *propinfoset) +{ + int i; + + for (i = 0; i < count; i++) + { + int p; + + for (p = 0; p < propinfoset[i].cPropertyInfos; p++) + { + VariantClear(&propinfoset[i].rgPropertyInfos[p].vValues); + CoTaskMemFree(&propinfoset[i].rgPropertyInfos[p]); + } + CoTaskMemFree(&propinfoset[i]); + } +} + +static HRESULT WINAPI datainit_GetInitializationString(IDataInitialize *iface, IUnknown *datasource, + boolean include_pass, LPWSTR *init_string) +{ + static const WCHAR provW[] = {'P','r','o','v','i','d','e','r','=',0}; + static const WCHAR colW[] = {';',0}; datainit *This = impl_from_IDataInitialize(iface); + DBPROPINFOSET *propinfoset; + IDBProperties *props; + DBPROPIDSET propidset; + ULONG count, infocount; + WCHAR *progid, *desc; + DBPROPSET *propset; + IPersist *persist; + HRESULT hr; + CLSID clsid; + int i, len; + + TRACE("(%p)->(%p %d %p)\n", This, datasource, include_pass, init_string); + + /* IPersist support is mandatory for data sources */ + hr = IUnknown_QueryInterface(datasource, &IID_IPersist, (void**)&persist); + if (FAILED(hr)) return hr; + + memset(&clsid, 0, sizeof(clsid)); + hr = IPersist_GetClassID(persist, &clsid); + IPersist_Release(persist); + if (FAILED(hr)) return hr; + + progid = NULL; + ProgIDFromCLSID(&clsid, &progid); + TRACE("clsid=%s, progid=%s\n", debugstr_guid(&clsid), debugstr_w(progid)); + + /* now get initialization properties */ + hr = IUnknown_QueryInterface(datasource, &IID_IDBProperties, (void**)&props); + if (FAILED(hr)) + { + WARN("IDBProperties not supported\n"); + CoTaskMemFree(progid); + return hr; + } - FIXME("(%p)->(%p %d %p)\n", This, pDataSource, fIncludePassword, ppwszInitString); + propidset.rgPropertyIDs = NULL; + propidset.cPropertyIDs = 0; + propidset.guidPropertySet = DBPROPSET_DBINIT; + propset = NULL; + count = 0; + hr = IDBProperties_GetProperties(props, 1, &propidset, &count, &propset); + if (FAILED(hr)) + { + WARN("failed to get data source properties, 0x%08x\n", hr); + CoTaskMemFree(progid); + return hr; + } - return E_NOTIMPL; + infocount = 0; + IDBProperties_GetPropertyInfo(props, 1, &propidset, &infocount, &propinfoset, &desc); + IDBProperties_Release(props); + + /* check if we need to skip password */ + len = strlenW(progid) + strlenW(provW) + 1; /* including ';' */ + for (i = 0; i < count; i++) + { + WCHAR *descr = get_propinfo_descr(&propset->rgProperties[i], propinfoset); + if (descr) + { + /* include '=' and ';' */ + len += strlenW(descr) + 2; + len += get_propvalue_length(&propset->rgProperties[i]); + } + + if ((propset->rgProperties[i].dwPropertyID == DBPROP_AUTH_PERSIST_SENSITIVE_AUTHINFO) && + (V_BOOL(&propset->rgProperties[i].vValue) == VARIANT_FALSE)) + include_pass = FALSE; + } + + len *= sizeof(WCHAR); + *init_string = CoTaskMemAlloc(len); + *init_string[0] = 0; + + /* provider name */ + strcatW(*init_string, provW); + strcatW(*init_string, progid); + strcatW(*init_string, colW); + CoTaskMemFree(progid); + + for (i = 0; i < count; i++) + { + WCHAR *descr; + + if (!include_pass && propset->rgProperties[i].dwPropertyID == DBPROP_AUTH_PASSWORD) continue; + + descr = get_propinfo_descr(&propset->rgProperties[i], propinfoset); + if (descr) + { + static const WCHAR eqW[] = {'=',0}; + strcatW(*init_string, descr); + strcatW(*init_string, eqW); + write_propvalue_str(*init_string, &propset->rgProperties[i]); + strcatW(*init_string, colW); + } + } + + free_dbpropset(count, propset); + free_dbpropinfoset(infocount, propinfoset); + CoTaskMemFree(desc); + + if (!include_pass) + TRACE("%s\n", debugstr_w(*init_string)); + return S_OK; } static HRESULT WINAPI datainit_CreateDBInstance(IDataInitialize *iface, REFCLSID provider,

1 0

Alexandre Julliard : odbc32: Fix some data types for 64-bit.
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: 1ce71ab7d85629f3cc98bc51d7ca0b0d5bc90b7c URL: http://source.winehq.org/git/wine.git/?a=commit;h=1ce71ab7d85629f3cc98bc51d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Nov 20 13:22:05 2012 +0100 odbc32: Fix some data types for 64-bit. --- dlls/odbc32/proxyodbc.c | 44 ++++++++++++++++++++++---------------------- include/sqltypes.h | 23 ++++++++++------------- 2 files changed, 32 insertions(+), 35 deletions(-) diff --git a/dlls/odbc32/proxyodbc.c b/dlls/odbc32/proxyodbc.c index 10adf16..c7a5038 100644 --- a/dlls/odbc32/proxyodbc.c +++ b/dlls/odbc32/proxyodbc.c @@ -688,7 +688,7 @@ static BOOL ODBC_LoadDMFunctions(void) SQLRETURN WINAPI SQLAllocConnect(SQLHENV EnvironmentHandle, SQLHDBC *ConnectionHandle) { SQLRETURN ret; - TRACE("Env=%lx\n",EnvironmentHandle); + TRACE("Env=%p\n",EnvironmentHandle); if (!pSQLAllocConnect) { @@ -698,7 +698,7 @@ SQLRETURN WINAPI SQLAllocConnect(SQLHENV EnvironmentHandle, SQLHDBC *ConnectionH } ret = pSQLAllocConnect(EnvironmentHandle, ConnectionHandle); - TRACE("Returns ret=%d, Handle %lx\n",ret, *ConnectionHandle); + TRACE("Returns ret=%d, Handle %p\n",ret, *ConnectionHandle); return ret; } @@ -719,7 +719,7 @@ SQLRETURN WINAPI SQLAllocEnv(SQLHENV *EnvironmentHandle) } ret = pSQLAllocEnv(EnvironmentHandle); - TRACE("Returns ret=%d, Env=%lx\n",ret, *EnvironmentHandle); + TRACE("Returns ret=%d, Env=%p\n",ret, *EnvironmentHandle); return ret; } @@ -730,7 +730,7 @@ SQLRETURN WINAPI SQLAllocEnv(SQLHENV *EnvironmentHandle) SQLRETURN WINAPI SQLAllocHandle(SQLSMALLINT HandleType, SQLHANDLE InputHandle, SQLHANDLE *OutputHandle) { SQLRETURN ret; - TRACE("(Type=%d, Handle=%lx)\n",HandleType,InputHandle); + TRACE("(Type=%d, Handle=%p)\n",HandleType,InputHandle); if (!pSQLAllocHandle) { @@ -751,7 +751,7 @@ SQLRETURN WINAPI SQLAllocHandle(SQLSMALLINT HandleType, SQLHANDLE InputHandle, S } ret = pSQLAllocHandle(HandleType, InputHandle, OutputHandle); - TRACE("Returns ret=%d, Handle=%lx\n",ret, *OutputHandle); + TRACE("Returns ret=%d, Handle=%p\n",ret, *OutputHandle); return ret; } @@ -763,7 +763,7 @@ SQLRETURN WINAPI SQLAllocStmt(SQLHDBC ConnectionHandle, SQLHSTMT *StatementHandl { SQLRETURN ret; - TRACE("(Connection=%lx)\n",ConnectionHandle); + TRACE("(Connection=%p)\n",ConnectionHandle); if (!pSQLAllocStmt) { @@ -773,7 +773,7 @@ SQLRETURN WINAPI SQLAllocStmt(SQLHDBC ConnectionHandle, SQLHSTMT *StatementHandl } ret = pSQLAllocStmt(ConnectionHandle, StatementHandle); - TRACE ("Returns ret=%d, Handle=%lx\n", ret, *StatementHandle); + TRACE ("Returns ret=%d, Handle=%p\n", ret, *StatementHandle); return ret; } @@ -864,7 +864,7 @@ SQLRETURN WINAPI SQLCancel(SQLHSTMT StatementHandle) SQLRETURN WINAPI SQLCloseCursor(SQLHSTMT StatementHandle) { SQLRETURN ret; - TRACE("(Handle=%lx)\n",StatementHandle); + TRACE("(Handle=%p)\n",StatementHandle); if (!pSQLCloseCursor) return SQL_ERROR; @@ -951,7 +951,7 @@ SQLRETURN WINAPI SQLDataSources(SQLHENV EnvironmentHandle, { SQLRETURN ret; - TRACE("EnvironmentHandle = %p\n", (LPVOID)EnvironmentHandle); + TRACE("EnvironmentHandle = %p\n", EnvironmentHandle); if (!pSQLDataSources) return SQL_ERROR; @@ -979,7 +979,7 @@ SQLRETURN WINAPI SQLDataSourcesA(SQLHENV EnvironmentHandle, { SQLRETURN ret; - TRACE("EnvironmentHandle = %p\n", (void*)EnvironmentHandle); + TRACE("EnvironmentHandle = %p\n", EnvironmentHandle); if (!pSQLDataSourcesA) return SQL_ERROR; @@ -1021,7 +1021,7 @@ SQLRETURN WINAPI SQLDescribeCol(SQLHSTMT StatementHandle, SQLRETURN WINAPI SQLDisconnect(SQLHDBC ConnectionHandle) { SQLRETURN ret; - TRACE("(Handle=%lx)\n", ConnectionHandle); + TRACE("(Handle=%p)\n", ConnectionHandle); if (!pSQLDisconnect) return SQL_ERROR; @@ -1114,7 +1114,7 @@ SQLRETURN WINAPI SQLFetchScroll(SQLHSTMT StatementHandle, SQLSMALLINT FetchOrien SQLRETURN WINAPI SQLFreeConnect(SQLHDBC ConnectionHandle) { SQLRETURN ret; - TRACE("(Handle=%lx)\n",ConnectionHandle); + TRACE("(Handle=%p)\n",ConnectionHandle); if (!pSQLFreeConnect) return SQL_ERROR; @@ -1130,7 +1130,7 @@ SQLRETURN WINAPI SQLFreeConnect(SQLHDBC ConnectionHandle) SQLRETURN WINAPI SQLFreeEnv(SQLHENV EnvironmentHandle) { SQLRETURN ret; - TRACE("(Env=%lx)\n",EnvironmentHandle); + TRACE("(Env=%p)\n",EnvironmentHandle); if (!pSQLFreeEnv) return SQL_ERROR; @@ -1146,7 +1146,7 @@ SQLRETURN WINAPI SQLFreeEnv(SQLHENV EnvironmentHandle) SQLRETURN WINAPI SQLFreeHandle(SQLSMALLINT HandleType, SQLHANDLE Handle) { SQLRETURN ret; - TRACE("(Type=%d, Handle=%lx)\n",HandleType,Handle); + TRACE("(Type=%d, Handle=%p)\n",HandleType,Handle); if (!pSQLFreeHandle) return SQL_ERROR; @@ -1162,7 +1162,7 @@ SQLRETURN WINAPI SQLFreeHandle(SQLSMALLINT HandleType, SQLHANDLE Handle) SQLRETURN WINAPI SQLFreeStmt(SQLHSTMT StatementHandle, SQLUSMALLINT Option) { SQLRETURN ret; - TRACE("(Handle %lx, Option=%d)\n",StatementHandle, Option); + TRACE("(Handle %p, Option=%d)\n",StatementHandle, Option); if (!pSQLFreeStmt) return SQL_ERROR; @@ -2039,7 +2039,7 @@ SQLRETURN WINAPI SQLColAttributesW( { SQLRETURN iResult; - TRACE("hstmt=0x%08lx icol=%d fDescType=%d rgbDesc=%p cbDescMax=%d pcbDesc=%p pfDesc=%p\n", + TRACE("hstmt=%p icol=%d fDescType=%d rgbDesc=%p cbDescMax=%d pcbDesc=%p pfDesc=%p\n", hstmt, icol, fDescType, rgbDesc, cbDescMax, pcbDesc, pfDesc); if (!pSQLColAttributesW) return SQL_ERROR; @@ -2085,7 +2085,7 @@ SQLRETURN WINAPI SQLConnectW(SQLHDBC ConnectionHandle, SQLRETURN WINAPI SQLDescribeColW(SQLHSTMT StatementHandle, SQLUSMALLINT ColumnNumber, WCHAR *ColumnName, SQLSMALLINT BufferLength, SQLSMALLINT *NameLength, - SQLSMALLINT *DataType, SQLUINTEGER *ColumnSize, + SQLSMALLINT *DataType, SQLULEN *ColumnSize, SQLSMALLINT *DecimalDigits, SQLSMALLINT *Nullable) { SQLRETURN iResult; @@ -2182,7 +2182,7 @@ SQLRETURN WINAPI SQLColAttributeW (SQLHSTMT StatementHandle, { SQLRETURN iResult; - TRACE("StatementHandle=0x%08lx ColumnNumber=%d FieldIdentifier=%d CharacterAttribute=%p BufferLength=%d StringLength=%p NumericAttribute=%p\n", + TRACE("StatementHandle=%p ColumnNumber=%d FieldIdentifier=%d CharacterAttribute=%p BufferLength=%d StringLength=%p NumericAttribute=%p\n", StatementHandle, ColumnNumber, FieldIdentifier, CharacterAttribute, BufferLength, StringLength, NumericAttribute); @@ -2240,7 +2240,7 @@ SQLRETURN WINAPI SQLGetDescRecW(SQLHDESC DescriptorHandle, SQLSMALLINT RecNumber, WCHAR *Name, SQLSMALLINT BufferLength, SQLSMALLINT *StringLength, SQLSMALLINT *Type, SQLSMALLINT *SubType, - SQLINTEGER *Length, SQLSMALLINT *Precision, + SQLLEN *Length, SQLSMALLINT *Precision, SQLSMALLINT *Scale, SQLSMALLINT *Nullable) { TRACE("\n"); @@ -2289,7 +2289,7 @@ SQLRETURN WINAPI SQLGetStmtAttrW(SQLHSTMT StatementHandle, { SQLRETURN iResult; - TRACE("Attribute = (%02ld) Value = %p BufferLength = (%ld) StringLength = %p\n", + TRACE("Attribute = (%02d) Value = %p BufferLength = (%d) StringLength = %p\n", Attribute, Value, BufferLength, StringLength); if (Value == NULL) { @@ -2507,7 +2507,7 @@ SQLRETURN WINAPI SQLDataSourcesW(SQLHENV EnvironmentHandle, { SQLRETURN ret; - TRACE("EnvironmentHandle = %p\n", (LPVOID)EnvironmentHandle); + TRACE("EnvironmentHandle = %p\n", EnvironmentHandle); if (!pSQLDataSourcesW) return SQL_ERROR; @@ -2689,7 +2689,7 @@ SQLRETURN WINAPI SQLSetStmtAttrW(SQLHSTMT StatementHandle, SQLINTEGER StringLength) { SQLRETURN iResult; - TRACE("Attribute = (%02ld) Value = %p StringLength = (%ld)\n", + TRACE("Attribute = (%02d) Value = %p StringLength = (%d)\n", Attribute, Value, StringLength); if (!pSQLSetStmtAttrW) return SQL_ERROR; diff --git a/include/sqltypes.h b/include/sqltypes.h index 5c8f3ca..4f3f8fe 100644 --- a/include/sqltypes.h +++ b/include/sqltypes.h @@ -46,7 +46,7 @@ typedef double SQLDOUBLE; typedef double SQLFLOAT; #endif -typedef long SQLINTEGER; +typedef int SQLINTEGER; #if (ODBCVER >= 0x0300) typedef unsigned char SQLNUMERIC; @@ -69,27 +69,23 @@ typedef unsigned char SQLVARCHAR; typedef SQLSMALLINT SQLRETURN; -/* typedef void * SQLHANDLE; */ - -typedef SQLINTEGER SQLHANDLE; #if (ODBCVER >= 0x0300) - +typedef void *SQLHANDLE; typedef SQLHANDLE SQLHENV; typedef SQLHANDLE SQLHDBC; typedef SQLHANDLE SQLHSTMT; typedef SQLHANDLE SQLHDESC; - #else -typedef SQLINTEGER SQLHENV; -typedef SQLINTEGER SQLHDBC; -typedef SQLINTEGER SQLHSTMT; +typedef void *SQLHENV; +typedef void *SQLHDBC; +typedef void *SQLHSTMT; #endif typedef signed char SCHAR; typedef SCHAR SQLSCHAR; -typedef long int SDWORD; +typedef int SDWORD; typedef signed short int SWORD; -typedef unsigned long int UDWORD; +typedef unsigned int UDWORD; typedef unsigned short int UWORD; typedef UDWORD SQLUINTEGER; typedef signed long SLONG; @@ -207,7 +203,8 @@ typedef struct tagSQL_INTERVAL_STRUCT #endif #if (ODBCVER >= 0x0300) -#define ODBCINT64 long +#define ODBCINT64 __int64 +#define UODBCINT64 unsigned __int64 #ifdef ODBCINT64 typedef ODBCINT64 SQLBIGINT; typedef unsigned ODBCINT64 SQLUBIGINT; @@ -240,7 +237,7 @@ typedef struct tagSQLGUID #endif -typedef unsigned long int BOOKMARK; +typedef SQLULEN BOOKMARK; #define SQL_WCHAR (-8) #define SQL_C_WCHAR SQL_WCHAR

1 0

Daniel Jelinski : xolehlp: Add some stubs.
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: da87b102e07a0eafa87d283541ad43f55926a030 URL: http://source.winehq.org/git/wine.git/?a=commit;h=da87b102e07a0eafa87d28354… Author: Daniel Jelinski <djelinski1(a)gmail.com> Date: Sat Nov 3 23:43:36 2012 +0100 xolehlp: Add some stubs. --- dlls/xolehlp/xolehlp.c | 24 ++++++++++++++++++++++++ dlls/xolehlp/xolehlp.spec | 6 +++--- 2 files changed, 27 insertions(+), 3 deletions(-) diff --git a/dlls/xolehlp/xolehlp.c b/dlls/xolehlp/xolehlp.c index 3b970ef..9573744 100644 --- a/dlls/xolehlp/xolehlp.c +++ b/dlls/xolehlp/xolehlp.c @@ -39,3 +39,27 @@ BOOL WINAPI DllMain( HINSTANCE hinst, DWORD reason, LPVOID reserved ) } return TRUE; } + +HRESULT WINAPI DtcGetTransactionManager(char *host, char *tm_name, REFIID riid, + DWORD dwReserved1, WORD wcbReserved2, void *pvReserved2, void **ppv) +{ + FIXME("(%s, %s, %s, %d, %d, %p, %p): stub\n", debugstr_a(host), debugstr_a(tm_name), + debugstr_guid(riid), dwReserved1, wcbReserved2, pvReserved2, ppv); + return E_NOTIMPL; +} + +HRESULT WINAPI DtcGetTransactionManagerExA(CHAR *host, CHAR *tm_name, REFIID riid, + DWORD options, void *config, void **ppv) +{ + FIXME("(%s, %s, %s, %d, %p, %p): stub\n", debugstr_a(host), debugstr_a(tm_name), + debugstr_guid(riid), options, config, ppv); + return E_NOTIMPL; +} + +HRESULT WINAPI DtcGetTransactionManagerExW(WCHAR *host, WCHAR *tm_name, REFIID riid, + DWORD options, void *config, void **ppv) +{ + FIXME("(%s, %s, %s, %d, %p, %p): stub\n", debugstr_w(host), debugstr_w(tm_name), + debugstr_guid(riid), options, config, ppv); + return E_NOTIMPL; +} diff --git a/dlls/xolehlp/xolehlp.spec b/dlls/xolehlp/xolehlp.spec index f774542..0b149de 100644 --- a/dlls/xolehlp/xolehlp.spec +++ b/dlls/xolehlp/xolehlp.spec @@ -1,6 +1,6 @@ @ stub GetDtcLocaleResourceHandle -@ stub DtcGetTransactionManager +@ stdcall DtcGetTransactionManager(str str ptr long long ptr ptr) @ stub DtcGetTransactionManagerEx @ stub DtcGetTransactionManagerC -@ stub DtcGetTransactionManagerExA -@ stub DtcGetTransactionManagerExW +@ stdcall DtcGetTransactionManagerExA(str str ptr long ptr ptr) +@ stdcall DtcGetTransactionManagerExW(wstr wstr ptr long ptr ptr)

1 0

Christian Costa : wmvcore: Add stub for DllRegisterServer.
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: 37419c4af206828a218e49e663ebc544d02d2e86 URL: http://source.winehq.org/git/wine.git/?a=commit;h=37419c4af206828a218e49e66… Author: Christian Costa <titan.costa(a)gmail.com> Date: Fri Nov 16 15:13:14 2012 +0100 wmvcore: Add stub for DllRegisterServer. --- dlls/wmvcore/wmvcore.spec | 2 +- dlls/wmvcore/wmvcore_main.c | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletions(-) diff --git a/dlls/wmvcore/wmvcore.spec b/dlls/wmvcore/wmvcore.spec index ee85eb6..5b7e4f1 100644 --- a/dlls/wmvcore/wmvcore.spec +++ b/dlls/wmvcore/wmvcore.spec @@ -4,7 +4,7 @@ @ stub WMCreateSyncReaderPriv @ stub WMIsAvailableOffline @ stub WMValidateData -@ stub DllRegisterServer +@ stdcall -private DllRegisterServer() @ stub WMCreateBackupRestorer @ stub WMCreateEditor @ stub WMCreateIndexer diff --git a/dlls/wmvcore/wmvcore_main.c b/dlls/wmvcore/wmvcore_main.c index 5d07159..846b8d6 100644 --- a/dlls/wmvcore/wmvcore_main.c +++ b/dlls/wmvcore/wmvcore_main.c @@ -45,6 +45,13 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) return TRUE; } +HRESULT WINAPI DllRegisterServer(void) +{ + FIXME("(): stub\n"); + + return S_OK; +} + HRESULT WINAPI WMCreateReader(IUnknown *reserved, DWORD rights, IWMSyncReader **reader) { FIXME("(%p, %x, %p): stub\n", reserved, rights, reader);

1 0

Frédéric Delanoy : msi: Fix a couple of leaks ( coverity).
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: fcbc5cc5bb5af7fced9c282dbf250a1941342898 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fcbc5cc5bb5af7fced9c282db… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Fri Nov 16 18:38:02 2012 +0100 msi: Fix a couple of leaks (coverity). --- dlls/msi/files.c | 6 ++++-- dlls/msi/package.c | 7 +++++++ 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/dlls/msi/files.c b/dlls/msi/files.c index 11913ef..d831313 100644 --- a/dlls/msi/files.c +++ b/dlls/msi/files.c @@ -336,7 +336,8 @@ UINT ACTION_InstallFiles(MSIPACKAGE *package) if (rc != ERROR_SUCCESS) { ERR("Unable to load media info for %s (%u)\n", debugstr_w(file->File), rc); - return ERROR_FUNCTION_FAILED; + rc = ERROR_FUNCTION_FAILED; + goto done; } if (!file->Component->Enabled) continue; @@ -513,7 +514,8 @@ UINT ACTION_PatchFiles( MSIPACKAGE *package ) if (rc != ERROR_SUCCESS) { ERR("Unable to load media info for %s (%u)\n", debugstr_w(file->File), rc); - return ERROR_FUNCTION_FAILED; + rc = ERROR_FUNCTION_FAILED; + goto done; } comp->Action = msi_get_component_action( package, comp ); if (!comp->Enabled || comp->Action != INSTALLSTATE_LOCAL) continue; diff --git a/dlls/msi/package.c b/dlls/msi/package.c index 015af00..3055175 100644 --- a/dlls/msi/package.c +++ b/dlls/msi/package.c @@ -1567,13 +1567,17 @@ UINT MSI_OpenPackageW(LPCWSTR szPackage, MSIPACKAGE **pPackage) { r = msi_create_empty_local_file( localfile, dotmsi ); if (r != ERROR_SUCCESS) + { + msi_free ( base_url ); return r; + } if (!CopyFileW( file, localfile, FALSE )) { r = GetLastError(); WARN("unable to copy package %s to %s (%u)\n", debugstr_w(file), debugstr_w(localfile), r); DeleteFileW( localfile ); + msi_free ( base_url ); return r; } delete_on_close = TRUE; @@ -1581,7 +1585,10 @@ UINT MSI_OpenPackageW(LPCWSTR szPackage, MSIPACKAGE **pPackage) TRACE("opening package %s\n", debugstr_w( localfile )); r = MSI_OpenDatabaseW( localfile, MSIDBOPEN_TRANSACT, &db ); if (r != ERROR_SUCCESS) + { + msi_free ( base_url ); return r; + } } package = MSI_CreatePackage( db, base_url ); msi_free( base_url );

1 0

Dmitry Timoshkov : windowscodecs: Test QueryCapability of the TIFF decoder.
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: a98fe810a4af56626bc5c1475eb8032a137d6389 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a98fe810a4af56626bc5c1475… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Tue Nov 20 18:37:08 2012 +0800 windowscodecs: Test QueryCapability of the TIFF decoder. --- dlls/windowscodecs/tests/tiffformat.c | 148 ++++++++++++++++++++++++++++----- 1 files changed, 128 insertions(+), 20 deletions(-) diff --git a/dlls/windowscodecs/tests/tiffformat.c b/dlls/windowscodecs/tests/tiffformat.c index 0874053..9b5db52 100644 --- a/dlls/windowscodecs/tests/tiffformat.c +++ b/dlls/windowscodecs/tests/tiffformat.c @@ -94,6 +94,8 @@ static const struct tiff_1bpp_data }; #include "poppack.h" +static IWICImagingFactory *factory; + static const char *debugstr_guid(const GUID *guid) { static char buf[50]; @@ -106,23 +108,35 @@ static const char *debugstr_guid(const GUID *guid) return buf; } -static IWICBitmapDecoder *create_decoder(IWICImagingFactory *factory, - const void *image_data, UINT image_size) +static IStream *create_stream(const void *data, int data_size) { - HGLOBAL hmem; - BYTE *data; HRESULT hr; - IWICBitmapDecoder *decoder = NULL; IStream *stream; - GUID guid; + HGLOBAL hdata; + void *locked_data; + + hdata = GlobalAlloc(GMEM_MOVEABLE, data_size); + ok(hdata != 0, "GlobalAlloc failed\n"); + if (!hdata) return NULL; - hmem = GlobalAlloc(0, image_size); - data = GlobalLock(hmem); - memcpy(data, image_data, image_size); - GlobalUnlock(hmem); + locked_data = GlobalLock(hdata); + memcpy(locked_data, data, data_size); + GlobalUnlock(hdata); - hr = CreateStreamOnHGlobal(hmem, TRUE, &stream); - ok(hr == S_OK, "CreateStreamOnHGlobal error %#x\n", hr); + hr = CreateStreamOnHGlobal(hdata, TRUE, &stream); + ok(hr == S_OK, "CreateStreamOnHGlobal failed, hr=%x\n", hr); + + return stream; +} + +static IWICBitmapDecoder *create_decoder(const void *image_data, UINT image_size) +{ + HRESULT hr; + IStream *stream; + IWICBitmapDecoder *decoder = NULL; + GUID guid; + + stream = create_stream(image_data, image_size); hr = IWICImagingFactory_CreateDecoderFromStream(factory, stream, NULL, 0, &decoder); ok(hr == S_OK, "CreateDecoderFromStream error %#x\n", hr); @@ -139,19 +153,14 @@ static IWICBitmapDecoder *create_decoder(IWICImagingFactory *factory, static void test_tiff_palette(void) { HRESULT hr; - IWICImagingFactory *factory; IWICBitmapDecoder *decoder; IWICBitmapFrameDecode *frame; IWICPalette *palette; GUID format; - hr = CoCreateInstance(&CLSID_WICImagingFactory, NULL, CLSCTX_INPROC_SERVER, - &IID_IWICImagingFactory, (void **)&factory); - ok(hr == S_OK, "CoCreateInstance error %#x\n", hr); - if (FAILED(hr)) return; - - decoder = create_decoder(factory, &tiff_1bpp_data, sizeof(tiff_1bpp_data)); + decoder = create_decoder(&tiff_1bpp_data, sizeof(tiff_1bpp_data)); ok(decoder != 0, "Failed to load TIFF image data\n"); + hr = IWICBitmapDecoder_GetFrame(decoder, 0, &frame); ok(hr == S_OK, "GetFrame error %#x\n", hr); @@ -169,14 +178,113 @@ static void test_tiff_palette(void) IWICPalette_Release(palette); IWICBitmapFrameDecode_Release(frame); IWICBitmapDecoder_Release(decoder); - IWICImagingFactory_Release(factory); +} + +static void test_QueryCapability(void) +{ + HRESULT hr; + IStream *stream; + IWICBitmapDecoder *decoder; + static const DWORD exp_caps = WICBitmapDecoderCapabilityCanDecodeAllImages | + WICBitmapDecoderCapabilityCanDecodeSomeImages | + WICBitmapDecoderCapabilityCanEnumerateMetadata; + static const DWORD exp_caps_xp = WICBitmapDecoderCapabilityCanDecodeAllImages | + WICBitmapDecoderCapabilityCanDecodeSomeImages; + DWORD capability; + LARGE_INTEGER pos; + ULARGE_INTEGER cur_pos; + UINT frame_count; + + stream = create_stream(&tiff_1bpp_data, sizeof(tiff_1bpp_data)); + if (!stream) return; + + hr = IWICImagingFactory_CreateDecoder(factory, &GUID_ContainerFormatTiff, NULL, &decoder); + ok(hr == S_OK, "CreateDecoder error %#x\n", hr); + + frame_count = 0xdeadbeef; + hr = IWICBitmapDecoder_GetFrameCount(decoder, &frame_count); +todo_wine + ok(hr == S_OK || broken(hr == E_POINTER) /* XP */, "GetFrameCount error %#x\n", hr); +todo_wine + ok(frame_count == 0, "expected 0, got %u\n", frame_count); + + pos.QuadPart = 4; + hr = IStream_Seek(stream, pos, SEEK_SET, NULL); + ok(hr == S_OK, "IStream_Seek error %#x\n", hr); + + capability = 0xdeadbeef; + hr = IWICBitmapDecoder_QueryCapability(decoder, stream, &capability); +todo_wine + ok(hr == S_OK, "QueryCapability error %#x\n", hr); +todo_wine + ok(capability == exp_caps || capability == exp_caps_xp, + "expected %#x, got %#x\n", exp_caps, capability); + + frame_count = 0xdeadbeef; + hr = IWICBitmapDecoder_GetFrameCount(decoder, &frame_count); +todo_wine + ok(hr == S_OK, "GetFrameCount error %#x\n", hr); +todo_wine + ok(frame_count == 1, "expected 1, got %u\n", frame_count); + + pos.QuadPart = 0; + hr = IStream_Seek(stream, pos, SEEK_CUR, &cur_pos); + ok(hr == S_OK, "IStream_Seek error %#x\n", hr); +todo_wine + ok(cur_pos.QuadPart > 4 && cur_pos.QuadPart < sizeof(tiff_1bpp_data), + "current stream pos is at %x/%x\n", cur_pos.u.LowPart, cur_pos.u.HighPart); + + hr = IWICBitmapDecoder_QueryCapability(decoder, stream, &capability); +todo_wine + ok(hr == WINCODEC_ERR_WRONGSTATE, "expected WINCODEC_ERR_WRONGSTATE, got %#x\n", hr); + + hr = IWICBitmapDecoder_Initialize(decoder, stream, WICDecodeMetadataCacheOnDemand); +todo_wine + ok(hr == WINCODEC_ERR_WRONGSTATE, "expected WINCODEC_ERR_WRONGSTATE, got %#x\n", hr); + + IWICBitmapDecoder_Release(decoder); + + hr = IWICImagingFactory_CreateDecoderFromStream(factory, stream, NULL, 0, &decoder); +todo_wine + ok(hr == WINCODEC_ERR_COMPONENTNOTFOUND, "expected WINCODEC_ERR_COMPONENTNOTFOUND, got %#x\n", hr); + + pos.QuadPart = 0; + hr = IStream_Seek(stream, pos, SEEK_SET, NULL); + ok(hr == S_OK, "IStream_Seek error %#x\n", hr); + + hr = IWICImagingFactory_CreateDecoderFromStream(factory, stream, NULL, 0, &decoder); + ok(hr == S_OK, "CreateDecoderFromStream error %#x\n", hr); + + frame_count = 0xdeadbeef; + hr = IWICBitmapDecoder_GetFrameCount(decoder, &frame_count); + ok(hr == S_OK, "GetFrameCount error %#x\n", hr); + ok(frame_count == 1, "expected 1, got %u\n", frame_count); + + hr = IWICBitmapDecoder_Initialize(decoder, stream, WICDecodeMetadataCacheOnDemand); + ok(hr == WINCODEC_ERR_WRONGSTATE, "expected WINCODEC_ERR_WRONGSTATE, got %#x\n", hr); + + hr = IWICBitmapDecoder_QueryCapability(decoder, stream, &capability); +todo_wine + ok(hr == WINCODEC_ERR_WRONGSTATE, "expected WINCODEC_ERR_WRONGSTATE, got %#x\n", hr); + + IWICBitmapDecoder_Release(decoder); + IStream_Release(stream); } START_TEST(tiffformat) { + HRESULT hr; + CoInitializeEx(NULL, COINIT_APARTMENTTHREADED); + hr = CoCreateInstance(&CLSID_WICImagingFactory, NULL, CLSCTX_INPROC_SERVER, + &IID_IWICImagingFactory, (void **)&factory); + ok(hr == S_OK, "CoCreateInstance error %#x\n", hr); + if (FAILED(hr)) return; + test_tiff_palette(); + test_QueryCapability(); + IWICImagingFactory_Release(factory); CoUninitialize(); }

1 0

Akihiro Sagawa : winmm/tests: Add EOF and buffer pointer tests for mmio.
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: 1aa15750edf060b12a50b547ebadb086d9fc9ca2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1aa15750edf060b12a50b547e… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Wed Nov 14 19:20:03 2012 +0900 winmm/tests: Add EOF and buffer pointer tests for mmio. --- dlls/winmm/tests/mmio.c | 106 +++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 106 insertions(+), 0 deletions(-) diff --git a/dlls/winmm/tests/mmio.c b/dlls/winmm/tests/mmio.c index a4e7a2d..232f84b 100644 --- a/dlls/winmm/tests/mmio.c +++ b/dlls/winmm/tests/mmio.c @@ -787,6 +787,110 @@ static void test_mmioSeek(void) DeleteFileA(test_file); } +static void test_mmio_end_of_file(void) +{ + char test_file[MAX_PATH], buffer[128], data[16]; + MMIOINFO mmio; + HMMIO hmmio; + LONG ret; + MMRESULT res; + + if (!create_test_file(test_file)) return; + + memset(&mmio, 0, sizeof(mmio)); + mmio.fccIOProc = FOURCC_DOS; + mmio.pchBuffer = buffer; + mmio.cchBuffer = sizeof(buffer); + hmmio = mmioOpen(test_file, &mmio, MMIO_READ); + ok(hmmio != NULL, "mmioOpen error %u\n", mmio.wErrorRet); + if (hmmio == NULL) { + DeleteFileA(test_file); + return; + } + + ret = mmioSeek(hmmio, 0, SEEK_END); + ok(sizeof(RIFF_buf) == ret, "got %d\n", ret); + + ret = mmioRead(hmmio, data, sizeof(data)); + todo_wine ok(ret == 0, "expected %d, got %d\n", 0, ret); + + res = mmioGetInfo(hmmio, &mmio, 0); + ok(res == MMSYSERR_NOERROR, "expected 0, got %d\n", res); + + res = mmioAdvance(hmmio, &mmio, MMIO_READ); + ok(res == MMSYSERR_NOERROR, "expected 0, got %d\n", res); + todo_wine ok(mmio.pchNext == mmio.pchEndRead, "expected %p, got %p\n", mmio.pchEndRead, mmio.pchNext); + + mmioClose(hmmio, 0); + DeleteFileA(test_file); +} + +static void test_mmio_buffer_pointer(void) +{ + char test_file[MAX_PATH]; + char buffer[5], data[16]; + MMIOINFO mmio; + HMMIO hmmio; + LONG size, pos; + MMRESULT res; + + if (!create_test_file(test_file)) return; + + memset(&mmio, 0, sizeof(mmio)); + mmio.fccIOProc = FOURCC_DOS; + mmio.pchBuffer = buffer; + mmio.cchBuffer = sizeof(buffer); + hmmio = mmioOpen(test_file, &mmio, MMIO_READ); + ok(hmmio != NULL, "mmioOpen error %u\n", mmio.wErrorRet); + if (hmmio == NULL) { + DeleteFileA(test_file); + return; + } + + /* the buffer is empty */ + size = mmioRead(hmmio, data, 0); + ok(size == 0, "expected 0, got %d\n", size); + res = mmioGetInfo(hmmio, &mmio, 0); + ok(res == MMSYSERR_NOERROR, "expected 0, got %d\n", res); + ok(mmio.pchEndRead == mmio.pchBuffer, "expected %p, got %p\n", mmio.pchBuffer, mmio.pchEndRead); + + /* fill the buffer */ + size = mmioAdvance(hmmio, &mmio, MMIO_READ); + ok(mmio.pchEndRead-mmio.pchBuffer == sizeof(buffer), "got %d\n", (int)(mmio.pchEndRead-mmio.pchBuffer)); + + /* seeking to the same buffer chunk, the buffer is kept */ + size = sizeof(buffer)/2; + pos = mmioSeek(hmmio, size, SEEK_SET); + ok(pos == size, "failed to seek, expected %d, got %d\n", size, pos); + res = mmioGetInfo(hmmio, &mmio, 0); + ok(res == MMSYSERR_NOERROR, "expected 0, got %d\n", res); + ok(mmio.lBufOffset == 0, "expected 0, got %d\n", mmio.lBufOffset); + ok(mmio.pchNext-mmio.pchBuffer == size, "expected %d, got %d\n", size, (int)(mmio.pchNext-mmio.pchBuffer)); + ok(mmio.pchEndRead-mmio.pchBuffer == sizeof(buffer), "got %d\n", (int)(mmio.pchEndRead-mmio.pchBuffer)); + + /* seeking to another buffer chunk, the buffer is empty */ + size = sizeof(buffer) * 3 + sizeof(buffer) / 2; + pos = mmioSeek(hmmio, size, SEEK_SET); + ok(pos == size, "failed to seek, got %d\n", pos); + res = mmioGetInfo(hmmio, &mmio, 0); + ok(res == MMSYSERR_NOERROR, "expected 0, got %d\n", res); + todo_wine ok(mmio.lBufOffset == size, "expected %d, got %d\n", size, mmio.lBufOffset); + todo_wine ok(mmio.pchNext == mmio.pchBuffer, "expected %p, got %p\n", mmio.pchBuffer, mmio.pchNext); + todo_wine ok(mmio.pchEndRead == mmio.pchBuffer, "expected %p, got %p\n", mmio.pchBuffer, mmio.pchEndRead); + + /* reading a lot (as sizeof(data) > mmio.cchBuffer), the buffer is empty */ + size = mmioRead(hmmio, data, sizeof(data)); + ok(size == sizeof(data), "failed to read, got %d\n", size); + res = mmioGetInfo(hmmio, &mmio, 0); + ok(res == MMSYSERR_NOERROR, "expected 0, got %d\n", res); + todo_wine ok(mmio.lBufOffset == pos+size, "expected %d, got %d\n", pos+size, mmio.lBufOffset); + todo_wine ok(mmio.pchNext == mmio.pchBuffer, "expected %p, got %p\n", mmio.pchBuffer, mmio.pchNext); + todo_wine ok(mmio.pchEndRead == mmio.pchBuffer, "expected %p, got %p\n", mmio.pchBuffer, mmio.pchEndRead); + + mmioClose(hmmio, 0); + DeleteFileA(test_file); +} + START_TEST(mmio) { /* Make it possible to run the tests against a specific AVI file in @@ -803,4 +907,6 @@ START_TEST(mmio) test_mmioSetBuffer(fname); test_mmioOpen_fourcc(); test_mmioSeek(); + test_mmio_end_of_file(); + test_mmio_buffer_pointer(); }

1 0

Alexandre Julliard : comctl32/pager: Use DrawFrameControl to paint the buttons.
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: f3b8e68296767f77da98f944d332fc996a047e68 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f3b8e68296767f77da98f944d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Nov 20 16:01:47 2012 +0100 comctl32/pager: Use DrawFrameControl to paint the buttons. --- dlls/comctl32/pager.c | 173 ++++++------------------------------------------- 1 files changed, 20 insertions(+), 153 deletions(-) diff --git a/dlls/comctl32/pager.c b/dlls/comctl32/pager.c index e1420e0..c270723 100644 --- a/dlls/comctl32/pager.c +++ b/dlls/comctl32/pager.c @@ -84,9 +84,6 @@ typedef struct INT direction; /* direction of the scroll, (e.g. PGF_SCROLLUP) */ } PAGER_INFO; -#define MIN_ARROW_WIDTH 8 -#define MIN_ARROW_HEIGHT 5 - #define TIMERID1 1 #define TIMERID2 2 #define INITIAL_DELAY 500 @@ -116,115 +113,13 @@ PAGER_GetButtonRects(const PAGER_INFO* infoPtr, RECT* prcTopLeft, RECT* prcBotto } } -/* the horizontal arrows are: - * - * 01234 01234 - * 1 * * - * 2 ** ** - * 3*** *** - * 4*** *** - * 5 ** ** - * 6 * * - * 7 - * - */ -static void -PAGER_DrawHorzArrow (HDC hdc, RECT r, INT colorRef, BOOL left) -{ - INT x, y, w, h; - HPEN hPen, hOldPen; - - w = r.right - r.left + 1; - h = r.bottom - r.top + 1; - if ((h < MIN_ARROW_WIDTH) || (w < MIN_ARROW_HEIGHT)) - return; /* refuse to draw partial arrow */ - - if (!(hPen = CreatePen( PS_SOLID, 1, GetSysColor( colorRef )))) return; - hOldPen = SelectObject ( hdc, hPen ); - if (left) - { - x = r.left + ((w - MIN_ARROW_HEIGHT) / 2) + 3; - y = r.top + ((h - MIN_ARROW_WIDTH) / 2) + 1; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x--, y+5); y++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x--, y+3); y++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x, y+1); - } - else - { - x = r.left + ((w - MIN_ARROW_HEIGHT) / 2) + 1; - y = r.top + ((h - MIN_ARROW_WIDTH) / 2) + 1; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x++, y+5); y++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x++, y+3); y++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x, y+1); - } - - SelectObject( hdc, hOldPen ); - DeleteObject( hPen ); -} - -/* the vertical arrows are: - * - * 01234567 01234567 - * 1****** ** - * 2 **** **** - * 3 ** ****** - * 4 - * - */ -static void -PAGER_DrawVertArrow (HDC hdc, RECT r, INT colorRef, BOOL up) -{ - INT x, y, w, h; - HPEN hPen, hOldPen; - - w = r.right - r.left + 1; - h = r.bottom - r.top + 1; - if ((h < MIN_ARROW_WIDTH) || (w < MIN_ARROW_HEIGHT)) - return; /* refuse to draw partial arrow */ - - if (!(hPen = CreatePen( PS_SOLID, 1, GetSysColor( colorRef )))) return; - hOldPen = SelectObject ( hdc, hPen ); - if (up) - { - x = r.left + ((w - MIN_ARROW_HEIGHT) / 2) + 1; - y = r.top + ((h - MIN_ARROW_WIDTH) / 2) + 3; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+5, y--); x++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+3, y--); x++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+1, y); - } - else - { - x = r.left + ((w - MIN_ARROW_HEIGHT) / 2) + 1; - y = r.top + ((h - MIN_ARROW_WIDTH) / 2) + 1; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+5, y++); x++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+3, y++); x++; - MoveToEx (hdc, x, y, NULL); - LineTo (hdc, x+1, y); - } - - SelectObject( hdc, hOldPen ); - DeleteObject( hPen ); -} - static void -PAGER_DrawButton(HDC hdc, COLORREF clrBk, RECT arrowRect, +PAGER_DrawButton(HDC hdc, COLORREF clrBk, RECT rc, BOOL horz, BOOL topLeft, INT btnState) { - HBRUSH hBrush, hOldBrush; - RECT rc = arrowRect; + UINT flags; - TRACE("arrowRect = %s, btnState = %d\n", wine_dbgstr_rect(&arrowRect), btnState); + TRACE("rc = %s, btnState = %d\n", wine_dbgstr_rect(&rc), btnState); if (btnState == PGF_INVISIBLE) return; @@ -232,54 +127,26 @@ PAGER_DrawButton(HDC hdc, COLORREF clrBk, RECT arrowRect, if ((rc.right - rc.left <= 0) || (rc.bottom - rc.top <= 0)) return; - hBrush = CreateSolidBrush(clrBk); - hOldBrush = SelectObject(hdc, hBrush); - - FillRect(hdc, &rc, hBrush); + if (horz) + flags = topLeft ? DFCS_SCROLLLEFT : DFCS_SCROLLRIGHT; + else + flags = topLeft ? DFCS_SCROLLUP : DFCS_SCROLLDOWN; - if (btnState == PGF_HOT) + switch (btnState) { - DrawEdge( hdc, &rc, BDR_RAISEDINNER, BF_RECT); - if (horz) - PAGER_DrawHorzArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - else - PAGER_DrawVertArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - } - else if (btnState == PGF_NORMAL) - { - DrawEdge (hdc, &rc, BDR_OUTER, BF_FLAT); - if (horz) - PAGER_DrawHorzArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - else - PAGER_DrawVertArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - } - else if (btnState == PGF_DEPRESSED) - { - DrawEdge( hdc, &rc, BDR_SUNKENOUTER, BF_RECT); - if (horz) - PAGER_DrawHorzArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - else - PAGER_DrawVertArrow(hdc, rc, COLOR_WINDOWFRAME, topLeft); - } - else if (btnState == PGF_GRAYED) - { - DrawEdge (hdc, &rc, BDR_OUTER, BF_FLAT); - if (horz) - { - PAGER_DrawHorzArrow(hdc, rc, COLOR_3DHIGHLIGHT, topLeft); - rc.left++, rc.top++; rc.right++, rc.bottom++; - PAGER_DrawHorzArrow(hdc, rc, COLOR_3DSHADOW, topLeft); - } - else - { - PAGER_DrawVertArrow(hdc, rc, COLOR_3DHIGHLIGHT, topLeft); - rc.left++, rc.top++; rc.right++, rc.bottom++; - PAGER_DrawVertArrow(hdc, rc, COLOR_3DSHADOW, topLeft); - } + case PGF_HOT: + break; + case PGF_NORMAL: + flags |= DFCS_FLAT; + break; + case PGF_DEPRESSED: + flags |= DFCS_PUSHED; + break; + case PGF_GRAYED: + flags |= DFCS_INACTIVE | DFCS_FLAT; + break; } - - SelectObject( hdc, hOldBrush ); - DeleteObject(hBrush); + DrawFrameControl( hdc, &rc, DFC_SCROLL, flags ); } /* << PAGER_GetDropTarget >> */

1 0

Alexandre Julliard : comctl32/pager: Fix tracking of mouse position inside the buttons.
by Alexandre Julliard 20 Nov '12

20 Nov '12

Module: wine Branch: master Commit: 7a78eed463903639f39458b23b6e8403963f8367 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7a78eed463903639f39458b23… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Nov 20 16:02:09 2012 +0100 comctl32/pager: Fix tracking of mouse position inside the buttons. --- dlls/comctl32/pager.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/comctl32/pager.c b/dlls/comctl32/pager.c index c270723..d541374 100644 --- a/dlls/comctl32/pager.c +++ b/dlls/comctl32/pager.c @@ -314,9 +314,10 @@ PAGER_UpdateBtns(PAGER_INFO *infoPtr, INT scrollRange, BOOL hideGrayBtns) RECT rcTopLeft, rcBottomRight; /* get button rects */ - PAGER_GetButtonRects(infoPtr, &rcTopLeft, &rcBottomRight, FALSE); + PAGER_GetButtonRects(infoPtr, &rcTopLeft, &rcBottomRight, TRUE); GetCursorPos(&pt); + ScreenToClient( infoPtr->hwndSelf, &pt ); /* update states based on scroll position */ if (infoPtr->nPos > 0) @@ -324,7 +325,7 @@ PAGER_UpdateBtns(PAGER_INFO *infoPtr, INT scrollRange, BOOL hideGrayBtns) if (infoPtr->TLbtnState == PGF_INVISIBLE || infoPtr->TLbtnState == PGF_GRAYED) infoPtr->TLbtnState = PGF_NORMAL; } - else if (PtInRect(&rcTopLeft, pt)) + else if (!hideGrayBtns && PtInRect(&rcTopLeft, pt)) infoPtr->TLbtnState = PGF_GRAYED; else infoPtr->TLbtnState = PGF_INVISIBLE; @@ -339,7 +340,7 @@ PAGER_UpdateBtns(PAGER_INFO *infoPtr, INT scrollRange, BOOL hideGrayBtns) if (infoPtr->BRbtnState == PGF_INVISIBLE || infoPtr->BRbtnState == PGF_GRAYED) infoPtr->BRbtnState = PGF_NORMAL; } - else if (PtInRect(&rcBottomRight, pt)) + else if (!hideGrayBtns && PtInRect(&rcBottomRight, pt)) infoPtr->BRbtnState = PGF_GRAYED; else infoPtr->BRbtnState = PGF_INVISIBLE;

1 0