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

Dmitry Timoshkov : user: Add a TrackMouseEvent test, make it pass under Wine.
by Alexandre Julliard 21 Jul '06

21 Jul '06

Module: wine Branch: refs/heads/master Commit: f1e46cbe85cc203de87dc51528fc83163adbf21c URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=f1e46cbe85cc203de87dc51… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Thu Jul 20 22:27:29 2006 +0900 user: Add a TrackMouseEvent test, make it pass under Wine. --- dlls/user/input.c | 316 ++++++++++++++++--------------------------------- dlls/user/tests/msg.c | 154 ++++++++++++++++++++++++ 2 files changed, 254 insertions(+), 216 deletions(-) Diff: http://source.winehq.org/git/?p=wine.git;a=commitdiff;h=f1e46cbe85cc203de87…

1 0

Roderick Colenbrander : dinput8: DirectInput8Create rewrite.
by Alexandre Julliard 21 Jul '06

21 Jul '06

Module: wine Branch: refs/heads/master Commit: 951f4657b796dd128c6797e65fe497faee4f12f5 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=951f4657b796dd128c6797e… Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Thu Jul 20 12:42:23 2006 +0200 dinput8: DirectInput8Create rewrite. --- dlls/dinput8/Makefile.in | 2 +- dlls/dinput8/dinput8_main.c | 39 +++++++++++++++++++++++++++++++++++---- tools/wine.inf | 3 +++ 3 files changed, 39 insertions(+), 5 deletions(-) diff --git a/dlls/dinput8/Makefile.in b/dlls/dinput8/Makefile.in index 34135c1..4466d1d 100644 --- a/dlls/dinput8/Makefile.in +++ b/dlls/dinput8/Makefile.in @@ -4,7 +4,7 @@ SRCDIR = @srcdir@ VPATH = @srcdir@ MODULE = dinput8.dll IMPORTLIB = libdinput8.$(IMPLIBEXT) -IMPORTS = dinput kernel32 +IMPORTS = dinput ole32 advapi32 kernel32 EXTRALIBS = -luuid -ldxguid C_SRCS = \ diff --git a/dlls/dinput8/dinput8_main.c b/dlls/dinput8/dinput8_main.c index 54ea66e..95aa919 100644 --- a/dlls/dinput8/dinput8_main.c +++ b/dlls/dinput8/dinput8_main.c @@ -51,8 +51,39 @@ static void UnlockModule(void) * DirectInput8Create (DINPUT8.@) */ HRESULT WINAPI DirectInput8Create(HINSTANCE hinst, DWORD dwVersion, REFIID riid, LPVOID *ppDI, LPUNKNOWN punkOuter) { - /* TODO: Create the interface using CoCreateInstance as that's what windows does too and check if the version number >= 0x800 */ - return DirectInputCreateEx(hinst, dwVersion, riid, ppDI, punkOuter); + HRESULT hr; + + TRACE("hInst (%p), dwVersion: %ld, riid (%s), punkOuter (%p))\n", hinst, dwVersion, debugstr_guid(riid), punkOuter); + + /* The specified version needs to be dinput8 (0x800) or higher */ + if(dwVersion < 0x800) + return DIERR_OLDDIRECTINPUTVERSION; + + if( !(IsEqualGUID(&IID_IDirectInput8A, riid) || IsEqualGUID(&IID_IDirectInput8W, riid) || IsEqualGUID(&IID_IUnknown, riid)) ) + return DIERR_INVALIDPARAM; + + CoInitialize(NULL); + + hr = CoCreateInstance( &CLSID_DirectInput8, punkOuter, CLSCTX_INPROC_SERVER, riid, ppDI); + if(FAILED(hr)) { + ERR("CoCreateInstance failed with hr = %ld\n", hr); + return DIERR_INVALIDPARAM; + } + + CoUninitialize(); + + /* When aggregation is used (punkOuter!=NULL) the application needs to manually call Initialize. */ + if(punkOuter == NULL && IsEqualGUID(&IID_IDirectInput8A, riid)) { + LPDIRECTINPUTA DI = (LPDIRECTINPUTA)*ppDI; + IDirectInput8_Initialize(DI, hinst, dwVersion); + } + + if(punkOuter == NULL && IsEqualGUID(&IID_IDirectInput8W, riid)) { + LPDIRECTINPUTW DI = (LPDIRECTINPUTW)*ppDI; + IDirectInput8_Initialize(DI, hinst, dwVersion); + } + + return S_OK; } /******************************************************************************* @@ -84,8 +115,8 @@ static HRESULT WINAPI DI8CF_CreateInstan IClassFactoryImpl *This = (IClassFactoryImpl *)iface; TRACE("(%p)->(%p,%s,%p)\n",This,pOuter,debugstr_guid(riid),ppobj); - if( IsEqualGUID( &IID_IDirectInput8A, riid ) || IsEqualGUID( &IID_IDirectInput8W, riid ) ) { - return DirectInput8Create(0, DIRECTINPUT_VERSION, riid, ppobj, pOuter); + if( IsEqualGUID( &IID_IDirectInput8A, riid ) || IsEqualGUID( &IID_IDirectInput8W, riid ) || IsEqualGUID( &IID_IUnknown, riid )) { + return DirectInputCreateEx(0, DIRECTINPUT_VERSION, riid, ppobj, pOuter); } ERR("(%p,%p,%s,%p) Interface not found!\n",This,pOuter,debugstr_guid(riid),ppobj); diff --git a/tools/wine.inf b/tools/wine.inf index d1a280e..316c25a 100644 --- a/tools/wine.inf +++ b/tools/wine.inf @@ -126,6 +126,9 @@ HKCR,AVIFile\Extensions\AVI,,,"{00020000 HKCR,AVIFile\Extensions\WAV,,,"{00020003-0000-0000-C000-000000000046}" HKCR,AVIFile\RIFFHandlers\AVI,,,"{00020000-0000-0000-C000-000000000046}" HKCR,AVIFile\RIFFHandlers\WAVE,,,"{00020003-0000-0000-C000-000000000046}" +HKCR,CLSID\{25E609E4-B259-11CF-BFC7-444553540000},,,"DirectInput8 Object" +HKCR,CLSID\{25E609E4-B259-11CF-BFC7-444553540000}\InProcServer32,,,"dinput8.dll" +HKCR,CLSID\{25E609E4-B259-11CF-BFC7-444553540000}\InProcServer32,ThreadingModel,,"Both" HKCR,TypeLib\{00020430-0000-0000-C000-000000000046}\1.0,,,"OLE Automation" HKCR,TypeLib\{00020430-0000-0000-C000-000000000046}\1.0\0\win16,,,"stdole.tlb" HKCR,TypeLib\{00020430-0000-0000-C000-000000000046}\1.0\0\win32,,,"stdole32.tlb"

1 0

Juan Lang : msi: Don't use fixed-size buffers in AppSearch action.
by Alexandre Julliard 21 Jul '06

21 Jul '06

Module: wine Branch: refs/heads/master Commit: fb6bd2414bdf8634d633b8056d663b5ad789f160 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=fb6bd2414bdf8634d633b80… Author: Juan Lang <juan_lang(a)yahoo.com> Date: Thu Jul 20 06:55:48 2006 -0700 msi: Don't use fixed-size buffers in AppSearch action. --- dlls/msi/appsearch.c | 64 ++++++++++++++------------------------------------ 1 files changed, 18 insertions(+), 46 deletions(-) diff --git a/dlls/msi/appsearch.c b/dlls/msi/appsearch.c index 598965e..2e317c1 100644 --- a/dlls/msi/appsearch.c +++ b/dlls/msi/appsearch.c @@ -872,9 +872,8 @@ static UINT ACTION_AppSearchDr(MSIPACKAG if (rc == ERROR_SUCCESS) { MSIRECORD *row = 0; - WCHAR buffer[MAX_PATH], expanded[MAX_PATH]; - LPWSTR path = NULL, parent = NULL; - DWORD sz; + WCHAR expanded[MAX_PATH]; + LPWSTR parentName = NULL, path = NULL, parent = NULL; int depth; rc = MSI_ViewExecute(view, 0); @@ -892,36 +891,25 @@ static UINT ACTION_AppSearchDr(MSIPACKAG } /* check whether parent is set */ - buffer[0] = 0; - sz=sizeof(buffer)/sizeof(buffer[0]); - rc = MSI_RecordGetStringW(row,2,buffer,&sz); - if (rc != ERROR_SUCCESS) - { - ERR("Error is %x\n",rc); - goto end; - } - else if (buffer[0]) + parentName = msi_dup_record_field(row,2); + if (parentName) { MSISIGNATURE parentSig; - rc = ACTION_AppSearchSigName(package, buffer, &parentSig, &parent); + rc = ACTION_AppSearchSigName(package, parentName, &parentSig, + &parent); ACTION_FreeSignature(&parentSig); + msi_free(parentName); } /* now look for path */ - buffer[0] = 0; - sz=sizeof(buffer)/sizeof(buffer[0]); - rc = MSI_RecordGetStringW(row,3,buffer,&sz); - if (rc != ERROR_SUCCESS) - { - ERR("Error is %x\n",rc); - goto end; - } + path = msi_dup_record_field(row,3); if (MSI_RecordIsNull(row,4)) depth = 0; else depth = MSI_RecordGetInteger(row,4); - ACTION_ExpandAnyPath(package, buffer, expanded, + ACTION_ExpandAnyPath(package, path, expanded, sizeof(expanded) / sizeof(expanded[0])); + msi_free(path); if (parent) { path = HeapAlloc(GetProcessHeap(), 0, strlenW(parent) + @@ -994,8 +982,7 @@ UINT ACTION_AppSearch(MSIPACKAGE *packag if (rc == ERROR_SUCCESS) { MSIRECORD *row = 0; - WCHAR propBuf[0x100], sigBuf[0x100]; - DWORD sz; + LPWSTR propName, sigName; rc = MSI_ViewExecute(view, 0); if (rc != ERROR_SUCCESS) @@ -1014,36 +1001,21 @@ UINT ACTION_AppSearch(MSIPACKAGE *packag } /* get property and signature */ - propBuf[0] = 0; - sz=sizeof(propBuf)/sizeof(propBuf[0]); - rc = MSI_RecordGetStringW(row,1,propBuf,&sz); - if (rc != ERROR_SUCCESS) - { - ERR("Error is %x\n",rc); - msiobj_release(&row->hdr); - break; - } - sigBuf[0] = 0; - sz=sizeof(sigBuf)/sizeof(sigBuf[0]); - rc = MSI_RecordGetStringW(row,2,sigBuf,&sz); - if (rc != ERROR_SUCCESS) - { - ERR("Error is %x\n",rc); - msiobj_release(&row->hdr); - break; - } + propName = msi_dup_record_field(row,1); + sigName = msi_dup_record_field(row,2); TRACE("Searching for Property %s, Signature_ %s\n", - debugstr_w(propBuf), debugstr_w(sigBuf)); + debugstr_w(propName), debugstr_w(sigName)); - rc = ACTION_AppSearchSigName(package, sigBuf, &sig, &value); + rc = ACTION_AppSearchSigName(package, sigName, &sig, &value); if (value) { - MSI_SetPropertyW(package, propBuf, value); + MSI_SetPropertyW(package, propName, value); msi_free(value); } - ACTION_FreeSignature(&sig); + msi_free(propName); + msi_free(sigName); msiobj_release(&row->hdr); }

1 0

Robert Shearman : ole32: Implement CoRegisterPSClsid.
by Alexandre Julliard 21 Jul '06

21 Jul '06

Module: wine Branch: refs/heads/master Commit: 2308d49bcba08d12582c99e2f43aebc1e0568df8 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=2308d49bcba08d12582c99e… Author: Robert Shearman <rob(a)codeweavers.com> Date: Thu Jul 20 19:30:21 2006 +0100 ole32: Implement CoRegisterPSClsid. --- dlls/ole32/compobj.c | 105 ++++++++++++++++++++++++++++- dlls/ole32/compobj_private.h | 1 dlls/ole32/ole32.spec | 2 - dlls/ole32/tests/compobj.c | 152 ++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 256 insertions(+), 4 deletions(-) Diff: http://source.winehq.org/git/?p=wine.git;a=commitdiff;h=2308d49bcba08d12582…

1 0

Ge van Geldorp : gdi: Use variables of correct size.
by Alexandre Julliard 21 Jul '06

21 Jul '06

Module: wine Branch: refs/heads/master Commit: 1e554e870b89a1c87f6af58e78ee07d443279f61 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=1e554e870b89a1c87f6af58… Author: Ge van Geldorp <ge(a)gse.nl> Date: Wed Jul 19 09:17:49 2006 +0200 gdi: Use variables of correct size. --- dlls/gdi/freetype.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/gdi/freetype.c b/dlls/gdi/freetype.c index 7fb62bd..2a6024c 100644 --- a/dlls/gdi/freetype.c +++ b/dlls/gdi/freetype.c @@ -814,7 +814,7 @@ #ifdef HAVE_FREETYPE_FTWINFNT_H CHARSETINFO csi; TRACE("pix_h %d charset %d dpi %dx%d pt %d\n", winfnt_header.pixel_height, winfnt_header.charset, winfnt_header.vertical_resolution,winfnt_header.horizontal_resolution, winfnt_header.nominal_point_size); - if(TranslateCharsetInfo((DWORD*)(UINT)winfnt_header.charset, &csi, TCI_SRCCHARSET)) + if(TranslateCharsetInfo((DWORD*)(UINT_PTR)winfnt_header.charset, &csi, TCI_SRCCHARSET)) memcpy(&fs, &csi.fs, sizeof(csi.fs)); internal_leading = winfnt_header.internal_leading; } @@ -1901,7 +1901,7 @@ static int get_nearest_charset(Face *fac DWORD fs0; *cp = acp; - if(TranslateCharsetInfo((DWORD*)acp, &csi, TCI_SRCCODEPAGE)) + if(TranslateCharsetInfo((DWORD*)(INT_PTR)acp, &csi, TCI_SRCCODEPAGE)) if(csi.fs.fsCsb[0] & face->fs.fsCsb[0]) return csi.ciCharset; @@ -2305,7 +2305,7 @@ GdiFont WineEngCreateFontInstance(DC *dc if(!strcmpiW(lf.lfFaceName, SymbolW)) lf.lfCharSet = SYMBOL_CHARSET; - if(!TranslateCharsetInfo((DWORD*)(INT)lf.lfCharSet, &csi, TCI_SRCCHARSET)) { + if(!TranslateCharsetInfo((DWORD*)(INT_PTR)lf.lfCharSet, &csi, TCI_SRCCHARSET)) { switch(lf.lfCharSet) { case DEFAULT_CHARSET: csi.fs.fsCsb[0] = 0; @@ -2351,7 +2351,7 @@ GdiFont WineEngCreateFontInstance(DC *dc corresponding to the current ansi codepage */ if(!csi.fs.fsCsb[0]) { INT acp = GetACP(); - if(!TranslateCharsetInfo((DWORD*)acp, &csi, TCI_SRCCODEPAGE)) { + if(!TranslateCharsetInfo((DWORD*)(INT_PTR)acp, &csi, TCI_SRCCODEPAGE)) { FIXME("TCI failed on codepage %d\n", acp); csi.fs.fsCsb[0] = 0; } else @@ -3924,7 +3924,7 @@ DWORD WineEngGetFontData(GdiFont font, D DWORD cbData) { FT_Face ft_face = font->ft_face; - DWORD len; + FT_ULong len; FT_Error err; TRACE("font=%p, table=%08lx, offset=%08lx, buf=%p, cbData=%lx\n", @@ -3947,7 +3947,7 @@ DWORD WineEngGetFontData(GdiFont font, D if(pFT_Load_Sfnt_Table) { /* make sure value of len is the value freetype says it needs */ if( buf && len) { - DWORD needed = 0; + FT_ULong needed = 0; err = pFT_Load_Sfnt_Table(ft_face, table, offset, NULL, &needed); if( !err && needed < len) len = needed; } @@ -3969,7 +3969,7 @@ #ifdef HAVE_FREETYPE_INTERNAL_SFNT_H } /* make sure value of len is the value freetype says it needs */ if( buf && len) { - DWORD needed = 0; + FT_ULong needed = 0; err = sfnt->load_any(tt_face, table, offset, NULL, &needed); if( !err && needed < len) len = needed; }

1 0

Bang Jun-Young : comdlg32: Remove redundant headers.
by Alexandre Julliard 21 Jul '06

21 Jul '06

Module: wine Branch: refs/heads/master Commit: 4988fad2325fafd5e6c076e8efd896fc5a94d21b URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=4988fad2325fafd5e6c076e… Author: Bang Jun-Young <bang.junyoung(a)gmail.com> Date: Wed Jul 19 17:22:55 2006 +0900 comdlg32: Remove redundant headers. --- dlls/comdlg32/filedlg31.c | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/dlls/comdlg32/filedlg31.c b/dlls/comdlg32/filedlg31.c index 79141fa..55c94b8 100644 --- a/dlls/comdlg32/filedlg31.c +++ b/dlls/comdlg32/filedlg31.c @@ -33,11 +33,7 @@ #include "wine/debug.h" #include "cderr.h" #include "winreg.h" #include "winternl.h" -#include "winuser.h" #include "commdlg.h" -#include "cderr.h" -#include "winreg.h" -#include "winternl.h" #include "shlwapi.h" WINE_DEFAULT_DEBUG_CHANNEL(commdlg);

1 0

Benjamin Arai : oleaut32: Implements OLEFontImpl_GetIDsOfNames.
by Alexandre Julliard 21 Jul '06

21 Jul '06

Module: wine Branch: refs/heads/master Commit: f75b9f190f368956949277983a09757d3e5b1a67 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=f75b9f190f3689569492779… Author: Benjamin Arai <me(a)benjaminarai.com> Date: Wed Jul 19 18:06:14 2006 -0700 oleaut32: Implements OLEFontImpl_GetIDsOfNames. --- dlls/oleaut32/olefont.c | 33 +++++++++++++++++++++++++++++---- 1 files changed, 29 insertions(+), 4 deletions(-) diff --git a/dlls/oleaut32/olefont.c b/dlls/oleaut32/olefont.c index 6c5ea27..8ce3e00 100644 --- a/dlls/oleaut32/olefont.c +++ b/dlls/oleaut32/olefont.c @@ -5,6 +5,7 @@ * interface and the OleCreateFontIndirect API call. * * Copyright 1999 Francis Beaudet + * Copyright 2006 (Google) Benjamin Arai * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -1347,11 +1348,35 @@ static HRESULT WINAPI OLEFontImpl_GetIDs LCID lcid, DISPID* rgDispId) { + ITypeInfo * pTInfo; + HRESULT hres; + OLEFontImpl *this = impl_from_IDispatch(iface); - FIXME("(%p,%s,%p,%d,%04x,%p), stub!\n", this, debugstr_guid(riid), rgszNames, - cNames, (int)lcid, rgDispId - ); - return E_NOTIMPL; + + TRACE("(%p,%s,%p,cNames=%d,lcid=%04x,%p)\n", this, debugstr_guid(riid), + rgszNames, cNames, (int)lcid, rgDispId); + + if (cNames == 0) + { + return E_INVALIDARG; + } + else + { + /* retrieve type information */ + hres = OLEFontImpl_GetTypeInfo(iface, 0, lcid, &pTInfo); + + if (FAILED(hres)) + { + ERR("GetTypeInfo failed.\n"); + return hres; + } + + /* convert names to DISPIDs */ + hres = DispGetIDsOfNames (pTInfo, rgszNames, cNames, rgDispId); + ITypeInfo_Release(pTInfo); + + return hres; + } } /************************************************************************

1 0

Benjamin Arai : oleaut32: OLEFontImpl_GetIDsOfNames conformance test.
by Alexandre Julliard 21 Jul '06

21 Jul '06

Module: wine Branch: refs/heads/master Commit: 31e3ad7150f5e308e5ead6f49fa0e87f7d2bafc4 URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=31e3ad7150f5e308e5ead6f… Author: Benjamin Arai <me(a)benjaminarai.com> Date: Wed Jul 19 18:08:38 2006 -0700 oleaut32: OLEFontImpl_GetIDsOfNames conformance test. --- dlls/oleaut32/tests/olefont.c | 111 +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 111 insertions(+), 0 deletions(-) diff --git a/dlls/oleaut32/tests/olefont.c b/dlls/oleaut32/tests/olefont.c index 9525eb3..e17d66a 100644 --- a/dlls/oleaut32/tests/olefont.c +++ b/dlls/oleaut32/tests/olefont.c @@ -2,6 +2,7 @@ * OLEFONT test program * * Copyright 2003 Marcus Meissner + * Copyright 2006 (Google) Benjamin Arai * * * This library is free software; you can redistribute it and/or @@ -295,6 +296,115 @@ static void test_font_events_disp(void) IFont_Release(pFont2); } +void test_names_ids(WCHAR* w_name_1, const char* a_name_1, + WCHAR* w_name_2, const char* a_name_2, + LCID lcid, DISPID id_1, DISPID id_2, + HRESULT hres_expect, int numnames) +{ + LPVOID pvObj = NULL; + IFontDisp *fontdisp = NULL; + HRESULT hres; + DISPID rgDispId[2] = {0xdeadbeef, 0xdeadbeef}; + LPOLESTR names[2] = {w_name_1, w_name_2}; + + pOleCreateFontIndirect(NULL, &IID_IFontDisp, &pvObj); + fontdisp = pvObj; + + hres = IFontDisp_GetIDsOfNames(fontdisp, &IID_NULL, names, numnames, + lcid, rgDispId); + + /* test hres */ + ok(hres == hres_expect, + "GetIDsOfNames: \"%s\", \"%s\" returns 0x%08lx, expected 0x%08lx.\n", + a_name_1, a_name_2, hres, hres_expect); + + /* test first DISPID */ + ok(rgDispId[0]==id_1, + "GetIDsOfNames: \"%s\" gets DISPID 0x%08lx, expected 0x%08lx.\n", + a_name_1, rgDispId[0], id_1); + + /* test second DISPID is present */ + if (numnames == 2) + { + ok(rgDispId[1]==id_2, + "GetIDsOfNames: ..., \"%s\" gets DISPID 0x%08lx, expected 0x%08lx.\n", + a_name_2, rgDispId[1], id_2); + } + + IFontDisp_Release(fontdisp); +} + +void test_GetIDsOfNames(void) +{ + WCHAR name_Name[] = {'N','a','m','e',0}; + WCHAR name_Italic[] = {'I','t','a','l','i','c',0}; + WCHAR name_Size[] = {'S','i','z','e',0}; + WCHAR name_Bold[] = {'B','o','l','d',0}; + WCHAR name_Underline[] = {'U','n','d','e','r','l','i','n','e',0}; + WCHAR name_Strikethrough[] = {'S','t','r','i','k','e','t','h','r','o','u','g','h',0}; + WCHAR name_Weight[] = {'W','e','i','g','h','t',0}; + WCHAR name_Charset[] = {'C','h','a','r','s','e','t',0}; + WCHAR name_Foo[] = {'F','o','o',0}; + WCHAR name_nAmE[] = {'n','A','m','E',0}; + WCHAR name_Nom[] = {'N','o','m',0}; + + LCID en_us = MAKELCID(MAKELANGID(LANG_ENGLISH,SUBLANG_ENGLISH_US), + SORT_DEFAULT); + LCID fr_fr = MAKELCID(MAKELANGID(LANG_FRENCH,SUBLANG_FRENCH), + SORT_DEFAULT); + + /* Test DISPID_FONTs for the various properties. */ + test_names_ids(name_Name, "Name", NULL, "", en_us, + DISPID_FONT_NAME, 0, S_OK,1); + test_names_ids(name_Size, "Size", NULL, "", en_us, + DISPID_FONT_SIZE, 0, S_OK,1); + test_names_ids(name_Bold, "Bold", NULL, "", en_us, + DISPID_FONT_BOLD, 0, S_OK,1); + test_names_ids(name_Italic, "Italic", NULL, "", en_us, + DISPID_FONT_ITALIC, 0, S_OK,1); + test_names_ids(name_Underline, "Underline", NULL, "", en_us, + DISPID_FONT_UNDER, 0, S_OK,1); + test_names_ids(name_Strikethrough, "Strikethrough", NULL, "", en_us, + DISPID_FONT_STRIKE, 0, S_OK,1); + test_names_ids(name_Weight, "Weight", NULL, "", en_us, + DISPID_FONT_WEIGHT, 0, S_OK,1); + test_names_ids(name_Charset, "Charset", NULL, "", en_us, + DISPID_FONT_CHARSET, 0, S_OK,1); + + /* Capitalization doesn't matter. */ + test_names_ids(name_nAmE, "nAmE", NULL, "", en_us, + DISPID_FONT_NAME, 0, S_OK,1); + + /* Unknown name. */ + test_names_ids(name_Foo, "Foo", NULL, "", en_us, + DISPID_UNKNOWN, 0, DISP_E_UNKNOWNNAME,1); + + /* Pass several names: first is processed, */ + /* second gets DISPID_UNKNOWN and doesn't affect retval. */ + test_names_ids(name_Italic, "Italic", name_Name, "Name", en_us, + DISPID_FONT_ITALIC, DISPID_UNKNOWN, S_OK,2); + test_names_ids(name_Italic, "Italic", name_Foo, "Foo", en_us, + DISPID_FONT_ITALIC, DISPID_UNKNOWN, S_OK,2); + + /* Locale ID has no effect. */ + test_names_ids(name_Name, "Name", NULL, "", fr_fr, + DISPID_FONT_NAME, 0, S_OK,1); + test_names_ids(name_Nom, "This is not a font", NULL, "", fr_fr, + DISPID_UNKNOWN, 0, DISP_E_UNKNOWNNAME,1); + + /* One of the arguments are invalid */ + test_names_ids(name_Name, "Name", NULL, "", en_us, + 0xdeadbeef, 0xdeadbeef, E_INVALIDARG,0); + test_names_ids(name_Italic, "Italic", NULL, "", en_us, + 0xdeadbeef, 0xdeadbeef, E_INVALIDARG,0); + test_names_ids(name_Foo, "Foo", NULL, "", en_us, + 0xdeadbeef, 0xdeadbeef, E_INVALIDARG,0); + + /* Crazy locale ID? */ + test_names_ids(name_Name, "Name", NULL, "", -1, + DISPID_FONT_NAME, 0, S_OK,1); +} + START_TEST(olefont) { hOleaut32 = LoadLibraryA("oleaut32.dll"); @@ -316,4 +426,5 @@ START_TEST(olefont) /* test_ifont_sizes(186000, 0, 72, 2540, -19, "rounding"); */ /* test rounding */ test_font_events_disp(); + test_GetIDsOfNames(); }

1 0

appdb/ ./index.php ./scripts.js include/html.p ...
by WineHQ 21 Jul '06

21 Jul '06

ChangeSet ID: 26699 CVSROOT: /opt/cvs-commit Module name: appdb Changes by: wineowner(a)winehq.org 2006/07/20 23:34:58 Modified files: . : index.php scripts.js include : html.php testData.php util.php Log message: Chris Morgan <cmorgan(a)alum.wpi.edu> Make table rows clickable using javascript. Use this functionality in the top X lists and in the test results table Patch: http://cvs.winehq.org/patch.py?id=26699 Old revision New revision Changes Path 1.37 1.38 +0 -1 appdb/index.php 1.3 1.4 +19 -0 appdb/scripts.js 1.8 1.9 +8 -0 appdb/include/html.php 1.5 1.6 +3 -2 appdb/include/testData.php 1.72 1.73 +3 -2 appdb/include/util.php Index: appdb/index.php diff -u -p appdb/index.php:1.37 appdb/index.php:1.38 --- appdb/index.php:1.37 21 Jul 2006 4:34:58 -0000 +++ appdb/index.php 21 Jul 2006 4:34:58 -0000 @@ -13,7 +13,6 @@ require(BASE."include/incl.php"); apidb_header("Wine Application Database"); ?> - <img src="images/appdb_montage.jpg" width=391 height=266 align=right alt="Wine AppDB"> <h1>Welcome</h1> Index: appdb/scripts.js diff -u -p appdb/scripts.js:1.3 appdb/scripts.js:1.4 --- appdb/scripts.js:1.3 21 Jul 2006 4:34:58 -0000 +++ appdb/scripts.js 21 Jul 2006 4:34:58 -0000 @@ -18,3 +18,22 @@ function deleteURL(text, url) { self.location = url; } } + +function ChangeTrColor(tableRow, bHighLight, sHighlightColor, sInactiveColor) +{ + if (bHighLight) + { + tableRow.style.backgroundColor = sHighlightColor; + tableRow.style.cursor = "hand"; + } + else + { + tableRow.style.backgroundColor = sInactiveColor; + tableRow.style.cursor = "pointer"; + } +} + +function DoNav(sUrl) +{ + document.location.href = sUrl; +} Index: appdb/include/html.php diff -u -p appdb/include/html.php:1.8 appdb/include/html.php:1.9 --- appdb/include/html.php:1.8 21 Jul 2006 4:34:58 -0000 +++ appdb/include/html.php 21 Jul 2006 4:34:58 -0000 @@ -62,6 +62,14 @@ function html_tr($arr, $class = "", $ext return do_html_tr("td", $arr, $class, $extra); } +function html_tr_highlight_clickable($sClass, $sHighlightColor, $sInactiveColor, $sUrl) +{ + echo '<tr class='.$sClass.' '. + 'onmouseover="ChangeTrColor(this, true, \''.$sHighlightColor.'\', \''.$sInactiveColor.'\');"'. + 'onmouseout="ChangeTrColor(this, false, \''.$sHighlightColor.'\', \''.$sInactiveColor.'\');"'. + 'onclick="DoNav(\''.$sUrl.'\');">'; +} + // HTML TABLE function html_table_begin($extra = "") { Index: appdb/include/testData.php diff -u -p appdb/include/testData.php:1.5 appdb/include/testData.php:1.6 --- appdb/include/testData.php:1.5 21 Jul 2006 4:34:58 -0000 +++ appdb/include/testData.php 21 Jul 2006 4:34:58 -0000 @@ -409,15 +409,16 @@ class testData{ $oSubmitter = new User($oTest->iSubmitterId); $oDistribution = new distribution($oTest->iDistributionId); $bgcolor = $oTest->sTestedRating; - echo '<tr class='.$bgcolor.'>',"\n"; /* if the test we are displaying is this test then */ /* mark it as the current test */ if ($oTest->iTestingId == $this->iTestingId) { + echo '<tr class='.$bgcolor.'>',"\n"; echo ' <td align="center" class="color2"><b>Current</b></td>',"\n"; - } else + } else /* make all non-current rows clickable so clicking on them selects the test as current */ { + html_tr_highlight_clickable($bgcolor, "", "color2", $link.$oTest->iTestingId); echo ' <td align="center" class="color2">[<a href="'.$link.$oTest->iTestingId; if(is_string($showAll)) Index: appdb/include/util.php diff -u -p appdb/include/util.php:1.72 appdb/include/util.php:1.73 --- appdb/include/util.php:1.72 21 Jul 2006 4:34:58 -0000 +++ appdb/include/util.php 21 Jul 2006 4:34:58 -0000 @@ -280,9 +280,10 @@ function outputTopXRow($oRow) $oVersion = new Version($oRow->versionId); $oApp = new Application($oVersion->iAppId); $img = Screenshot::get_random_screenshot_img(null, $oRow->versionId, false); // image, disable extra formatting + html_tr_highlight_clickable("white", "white", "white", 'appview.php?iVersionId='.$oRow->versionId); echo ' - <tr class="white"> - <td class="app_name"><a href="appview.php?iVersionId='.$oRow->versionId.'">'.$oApp->sName.' '.$oVersion->sName.'</a></td> + <td class="app_name"> + <a href="appview.php?iVersionId='.$oRow->versionId.'">'.$oApp->sName.' '.$oVersion->sName.'</a></td> <td>'.util_trim_description($oApp->sDescription).'</td> <td><center>'.$img.'</center></td> </tr>';

1 0

appdb/. apidb.css
by WineHQ 21 Jul '06

21 Jul '06

ChangeSet ID: 26698 CVSROOT: /opt/cvs-commit Module name: appdb Changes by: wineowner(a)winehq.org 2006/07/20 23:27:01 Modified files: . : apidb.css Log message: Chris Morgan <cmorgan(a)alum.wpi.edu> Shift magnifying glass icon 10px left to put it near the corner of the thumbnail it is overlaying Patch: http://cvs.winehq.org/patch.py?id=26698 Old revision New revision Changes Path 1.16 1.17 +1 -1 appdb/apidb.css Index: appdb/apidb.css diff -u -p appdb/apidb.css:1.16 appdb/apidb.css:1.17 --- appdb/apidb.css:1.16 21 Jul 2006 4:27: 1 -0000 +++ appdb/apidb.css 21 Jul 2006 4:27: 1 -0000 @@ -234,4 +234,4 @@ TD.app_name { width: 150px } #editor { height: 25em; width:700px; } /* zoom image overlay style */ -.zoom_overlay { position:relative; top:10px; left:0px; margin-left:-32px; padding:0px; } +.zoom_overlay { position:relative; top:10px; left:10px; margin-left:-32px; padding:0px; }

1 0