April 2011 - Wine-Commits - list.winehq.org

Alexandre Julliard : winex11: Add infrastructure for managing the extra data of generic extension events .
by Alexandre Julliard April 13, 2011

April 13, 2011

Module: wine Branch: master Commit: a8575a8b4dc6f3678654dd3b72ece6d6f58632fa URL: http://source.winehq.org/git/wine.git/?a=commit;h=a8575a8b4dc6f3678654dd3b7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Apr 12 21:16:15 2011 +0200 winex11: Add infrastructure for managing the extra data of generic extension events. --- configure | 12 ++++++++++++ configure.ac | 2 +- dlls/winex11.drv/event.c | 32 ++++++++++++++++++++++++++------ dlls/winex11.drv/x11drv.h | 3 +++ dlls/winex11.drv/x11drv_main.c | 13 +++++++++++++ dlls/winex11.drv/xrandr.c | 4 +--- dlls/winex11.drv/xrender.c | 2 -- include/config.h.in | 3 +++ 8 files changed, 59 insertions(+), 12 deletions(-) diff --git a/configure b/configure index 76ecd55..8e8d044 100755 --- a/configure +++ b/configure @@ -8733,6 +8733,18 @@ _ACEOF fi +ac_fn_c_check_member "$LINENO" "XEvent" "xcookie" "ac_cv_member_XEvent_xcookie" "#ifdef HAVE_X11_XLIB_H +#include <X11/Xlib.h> +#endif +" +if test "x$ac_cv_member_XEvent_xcookie" = x""yes; then : + +cat >>confdefs.h <<_ACEOF +#define HAVE_XEVENT_XCOOKIE 1 +_ACEOF + + +fi diff --git a/configure.ac b/configure.ac index 21c8d85..e82d193 100644 --- a/configure.ac +++ b/configure.ac @@ -1045,7 +1045,7 @@ then [libxcomposite ${notice_platform}development files not found, Xcomposite won't be supported.]) dnl *** Check for XICCallback struct - AC_CHECK_MEMBERS([XICCallback.callback],,, + AC_CHECK_MEMBERS([XICCallback.callback, XEvent.xcookie],,, [#ifdef HAVE_X11_XLIB_H #include <X11/Xlib.h> #endif]) diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c index 474e288..a05e906 100644 --- a/dlls/winex11.drv/event.c +++ b/dlls/winex11.drv/event.c @@ -86,6 +86,9 @@ extern BOOL ximInComposeMode; #define XEMBED_UNREGISTER_ACCELERATOR 13 #define XEMBED_ACTIVATE_ACCELERATOR 14 +Bool (*pXGetEventData)( Display *display, XEvent /*XGenericEventCookie*/ *event ) = NULL; +void (*pXFreeEventData)( Display *display, XEvent /*XGenericEventCookie*/ *event ) = NULL; + /* Event handlers */ static void X11DRV_FocusIn( HWND hwnd, XEvent *event ); static void X11DRV_FocusOut( HWND hwnd, XEvent *event ); @@ -140,8 +143,6 @@ static x11drv_event_handler handlers[MAX_EVENT_HANDLERS] = NULL /* 35 GenericEvent */ }; - - static const char * event_names[MAX_EVENT_HANDLERS] = { NULL, NULL, "KeyPress", "KeyRelease", "ButtonPress", "ButtonRelease", @@ -152,13 +153,29 @@ static const char * event_names[MAX_EVENT_HANDLERS] = "CirculateNotify", "CirculateRequest", "PropertyNotify", "SelectionClear", "SelectionRequest", "SelectionNotify", "ColormapNotify", "ClientMessage", "MappingNotify", "GenericEvent" }; + /* return the name of an X event */ static const char *dbgstr_event( int type ) { - if (type < MAX_EVENT_HANDLERS && event_names[type]) return event_names[type - KeyPress]; + if (type < MAX_EVENT_HANDLERS && event_names[type]) return event_names[type]; return wine_dbg_sprintf( "Unknown event %d", type ); } +static inline void get_event_data( XEvent *event ) +{ +#if defined(GenericEvent) && defined(HAVE_XEVENT_XCOOKIE) + if (event->xany.type != GenericEvent) return; + if (!pXGetEventData || !pXGetEventData( event->xany.display, event )) event->xcookie.data = NULL; +#endif +} + +static inline void free_event_data( XEvent *event ) +{ +#if defined(GenericEvent) && defined(HAVE_XEVENT_XCOOKIE) + if (event->xany.type != GenericEvent) return; + if (event->xcookie.data) pXFreeEventData( event->xany.display, event ); +#endif +} /*********************************************************************** * X11DRV_register_event_handler @@ -333,22 +350,25 @@ static int process_events( Display *display, Bool (*filter)(Display*, XEvent*,XP else continue; /* filtered, ignore it */ } + get_event_data( &event ); if (prev_event.type) action = merge_events( &prev_event, &event ); switch( action ) { - case MERGE_DISCARD: /* discard prev, keep new */ - prev_event = event; - break; case MERGE_HANDLE: /* handle prev, keep new */ call_event_handler( display, &prev_event ); + /* fall through */ + case MERGE_DISCARD: /* discard prev, keep new */ + free_event_data( &prev_event ); prev_event = event; break; case MERGE_KEEP: /* handle new, keep prev for future merging */ call_event_handler( display, &event ); + free_event_data( &event ); break; } } if (prev_event.type) call_event_handler( display, &prev_event ); + free_event_data( &prev_event ); XFlush( gdi_display ); wine_tsx11_unlock(); if (count) TRACE( "processed %d events\n", count ); diff --git a/dlls/winex11.drv/x11drv.h b/dlls/winex11.drv/x11drv.h index a06de4f..5f6c5cf 100644 --- a/dlls/winex11.drv/x11drv.h +++ b/dlls/winex11.drv/x11drv.h @@ -729,6 +729,9 @@ extern void X11DRV_SelectionRequest( HWND hWnd, XEvent *event ); extern void X11DRV_SelectionClear( HWND hWnd, XEvent *event ); extern void X11DRV_MappingNotify( HWND hWnd, XEvent *event ); +extern Bool (*pXGetEventData)( Display *display, XEvent /*XGenericEventCookie*/ *event ); +extern void (*pXFreeEventData)( Display *display, XEvent /*XGenericEventCookie*/ *event ); + extern DWORD EVENT_x11_time_to_win32_time(Time time); /* X11 driver private messages, must be in the range 0x80001000..0x80001fff */ diff --git a/dlls/winex11.drv/x11drv_main.c b/dlls/winex11.drv/x11drv_main.c index 473b61f..9ac20c3 100644 --- a/dlls/winex11.drv/x11drv_main.c +++ b/dlls/winex11.drv/x11drv_main.c @@ -529,7 +529,20 @@ sym_not_found: */ static BOOL process_attach(void) { + char error[1024]; Display *display; + void *libx11 = wine_dlopen( SONAME_LIBX11, RTLD_NOW|RTLD_GLOBAL, error, sizeof(error) ); + + if (!libx11) + { + ERR( "failed to load %s: %s\n", SONAME_LIBX11, error ); + return FALSE; + } + pXGetEventData = wine_dlsym( libx11, "XGetEventData", NULL, 0 ); + pXFreeEventData = wine_dlsym( libx11, "XFreeEventData", NULL, 0 ); +#ifdef SONAME_LIBXEXT + wine_dlopen( SONAME_LIBXEXT, RTLD_NOW|RTLD_GLOBAL, NULL, 0 ); +#endif setup_options(); diff --git a/dlls/winex11.drv/xrandr.c b/dlls/winex11.drv/xrandr.c index c313bab..0aea463 100644 --- a/dlls/winex11.drv/xrandr.c +++ b/dlls/winex11.drv/xrandr.c @@ -71,9 +71,7 @@ static int load_xrandr(void) { int r = 0; - if (wine_dlopen(SONAME_LIBX11, RTLD_NOW|RTLD_GLOBAL, NULL, 0) && - wine_dlopen(SONAME_LIBXEXT, RTLD_NOW|RTLD_GLOBAL, NULL, 0) && - wine_dlopen(SONAME_LIBXRENDER, RTLD_NOW|RTLD_GLOBAL, NULL, 0) && + if (wine_dlopen(SONAME_LIBXRENDER, RTLD_NOW|RTLD_GLOBAL, NULL, 0) && (xrandr_handle = wine_dlopen(SONAME_LIBXRANDR, RTLD_NOW, NULL, 0))) { diff --git a/dlls/winex11.drv/xrender.c b/dlls/winex11.drv/xrender.c index 33e420a..52e7f25 100644 --- a/dlls/winex11.drv/xrender.c +++ b/dlls/winex11.drv/xrender.c @@ -346,8 +346,6 @@ void X11DRV_XRender_Init(void) int event_base, i; if (client_side_with_render && - wine_dlopen(SONAME_LIBX11, RTLD_NOW|RTLD_GLOBAL, NULL, 0) && - wine_dlopen(SONAME_LIBXEXT, RTLD_NOW|RTLD_GLOBAL, NULL, 0) && (xrender_handle = wine_dlopen(SONAME_LIBXRENDER, RTLD_NOW, NULL, 0))) { diff --git a/include/config.h.in b/include/config.h.in index b7460cc..19a6b48 100644 --- a/include/config.h.in +++ b/include/config.h.in @@ -1118,6 +1118,9 @@ /* Define to 1 if you have the <X11/Xutil.h> header file. */ #undef HAVE_X11_XUTIL_H +/* Define to 1 if `xcookie' is a member of `XEvent'. */ +#undef HAVE_XEVENT_XCOOKIE + /* Define to 1 if `callback' is a member of `XICCallback'. */ #undef HAVE_XICCALLBACK_CALLBACK

1 0

Henri Verbeet : comctl32: Allocate the correct amount of memory for tab items in TAB_InsertItemT().
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: e5c3883c923d345848315a39a090607bdddac936 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e5c3883c923d345848315a39a… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Apr 12 17:39:19 2011 +0200 comctl32: Allocate the correct amount of memory for tab items in TAB_InsertItemT(). --- dlls/comctl32/tab.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/comctl32/tab.c b/dlls/comctl32/tab.c index 6744ed7..a9a0548 100644 --- a/dlls/comctl32/tab.c +++ b/dlls/comctl32/tab.c @@ -2644,7 +2644,7 @@ TAB_InsertItemT (TAB_INFO *infoPtr, INT iItem, const TCITEMW *pti, BOOL bUnicode TAB_DumpItemExternalT(pti, iItem, bUnicode); - if (!(item = Alloc(sizeof(TAB_ITEM_SIZE(infoPtr))))) return FALSE; + if (!(item = Alloc(TAB_ITEM_SIZE(infoPtr)))) return FALSE; if (DPA_InsertPtr(infoPtr->items, iItem, item) == -1) { Free(item);

1 0

Huw Davies : gdi32: Don't set the bitfields when the dib section is BI_RGB.
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: 48f083b39d5ecb861329de9a77043c8075f30494 URL: http://source.winehq.org/git/wine.git/?a=commit;h=48f083b39d5ecb861329de9a7… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Apr 12 13:06:34 2011 +0100 gdi32: Don't set the bitfields when the dib section is BI_RGB. --- dlls/gdi32/dib.c | 35 ++++++++++++++++--------------- dlls/gdi32/tests/dib.c | 52 +++++++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 69 insertions(+), 18 deletions(-) diff --git a/dlls/gdi32/dib.c b/dlls/gdi32/dib.c index b63eb13..38b95c3 100644 --- a/dlls/gdi32/dib.c +++ b/dlls/gdi32/dib.c @@ -1216,8 +1216,9 @@ HBITMAP WINAPI CreateDIBSection(HDC hdc, CONST BITMAPINFO *bmi, UINT usage, if (!(dib = HeapAlloc( GetProcessHeap(), 0, sizeof(*dib) ))) return 0; - TRACE("format (%d,%d), planes %d, bpp %d, size %d, %s\n", - width, height, planes, bpp, sizeImage, usage == DIB_PAL_COLORS? "PAL" : "RGB"); + TRACE("format (%d,%d), planes %d, bpp %d, %s, size %d %s\n", + width, height, planes, bpp, compression == BI_BITFIELDS? "BI_BITFIELDS" : "BI_RGB", + sizeImage, usage == DIB_PAL_COLORS? "PAL" : "RGB"); dib->dsBm.bmType = 0; dib->dsBm.bmWidth = width; @@ -1255,24 +1256,24 @@ HBITMAP WINAPI CreateDIBSection(HDC hdc, CONST BITMAPINFO *bmi, UINT usage, dib->dsBmih.biSizeImage = dib->dsBm.bmWidthBytes * dib->dsBm.bmHeight; /* set dsBitfields values */ - if (usage == DIB_PAL_COLORS || bpp <= 8) + dib->dsBitfields[0] = dib->dsBitfields[1] = dib->dsBitfields[2] = 0; + + if((bpp == 15 || bpp == 16) && compression == BI_RGB) { - dib->dsBitfields[0] = dib->dsBitfields[1] = dib->dsBitfields[2] = 0; + /* In this case Windows changes biCompression to BI_BITFIELDS, + however for now we won't do this, as there are a lot + of places where BI_BITFIELDS is currently unsupported. */ + + /* dib->dsBmih.biCompression = compression = BI_BITFIELDS;*/ + dib->dsBitfields[0] = 0x7c00; + dib->dsBitfields[1] = 0x03e0; + dib->dsBitfields[2] = 0x001f; } - else switch( bpp ) + else if(compression == BI_BITFIELDS) { - case 15: - case 16: - dib->dsBitfields[0] = (compression == BI_BITFIELDS) ? *(const DWORD *)bmi->bmiColors : 0x7c00; - dib->dsBitfields[1] = (compression == BI_BITFIELDS) ? *((const DWORD *)bmi->bmiColors + 1) : 0x03e0; - dib->dsBitfields[2] = (compression == BI_BITFIELDS) ? *((const DWORD *)bmi->bmiColors + 2) : 0x001f; - break; - case 24: - case 32: - dib->dsBitfields[0] = (compression == BI_BITFIELDS) ? *(const DWORD *)bmi->bmiColors : 0xff0000; - dib->dsBitfields[1] = (compression == BI_BITFIELDS) ? *((const DWORD *)bmi->bmiColors + 1) : 0x00ff00; - dib->dsBitfields[2] = (compression == BI_BITFIELDS) ? *((const DWORD *)bmi->bmiColors + 2) : 0x0000ff; - break; + dib->dsBitfields[0] = *(const DWORD *)bmi->bmiColors; + dib->dsBitfields[1] = *((const DWORD *)bmi->bmiColors + 1); + dib->dsBitfields[2] = *((const DWORD *)bmi->bmiColors + 2); } /* get storage location for DIB bits */ diff --git a/dlls/gdi32/tests/dib.c b/dlls/gdi32/tests/dib.c index 784f0dd..d89a6fa 100644 --- a/dlls/gdi32/tests/dib.c +++ b/dlls/gdi32/tests/dib.c @@ -204,10 +204,14 @@ static void test_simple_graphics(void) { char bmibuf[sizeof(BITMAPINFO) + 256 * sizeof(RGBQUAD)]; BITMAPINFO *bmi = (BITMAPINFO *)bmibuf; + DWORD *bit_fields = (DWORD*)(bmibuf + sizeof(BITMAPINFOHEADER)); HDC mem_dc; BYTE *bits; HBITMAP dib, orig_bm; const char **sha1; + DIBSECTION ds; + + mem_dc = CreateCompatibleDC(NULL); /* a8r8g8b8 */ trace("8888\n"); @@ -221,7 +225,12 @@ static void test_simple_graphics(void) dib = CreateDIBSection(0, bmi, DIB_RGB_COLORS, (void**)&bits, NULL, 0); ok(dib != NULL, "ret NULL\n"); - mem_dc = CreateCompatibleDC(NULL); + ok(GetObjectW( dib, sizeof(ds), &ds ), "GetObject failed\n"); + ok(ds.dsBitfields[0] == 0, "got %08x\n", ds.dsBitfields[0]); + ok(ds.dsBitfields[1] == 0, "got %08x\n", ds.dsBitfields[1]); + ok(ds.dsBitfields[2] == 0, "got %08x\n", ds.dsBitfields[2]); + ok(ds.dsBmih.biCompression == BI_RGB, "got %x\n", ds.dsBmih.biCompression); + orig_bm = SelectObject(mem_dc, dib); sha1 = sha1_graphics_a8r8g8b8; @@ -229,6 +238,47 @@ static void test_simple_graphics(void) SelectObject(mem_dc, orig_bm); DeleteObject(dib); + + /* a8r8g8b8 - bitfields. Should be the same as the regular 32 bit case.*/ + trace("8888 - bitfields\n"); + bmi->bmiHeader.biBitCount = 32; + bmi->bmiHeader.biCompression = BI_BITFIELDS; + bit_fields[0] = 0xff0000; + bit_fields[1] = 0x00ff00; + bit_fields[2] = 0x0000ff; + + dib = CreateDIBSection(mem_dc, bmi, DIB_RGB_COLORS, (void**)&bits, NULL, 0); + ok(dib != NULL, "ret NULL\n"); + ok(GetObjectW( dib, sizeof(ds), &ds ), "GetObject failed\n"); + ok(ds.dsBitfields[0] == 0xff0000, "got %08x\n", ds.dsBitfields[0]); + ok(ds.dsBitfields[1] == 0x00ff00, "got %08x\n", ds.dsBitfields[1]); + ok(ds.dsBitfields[2] == 0x0000ff, "got %08x\n", ds.dsBitfields[2]); + ok(ds.dsBmih.biCompression == BI_BITFIELDS, "got %x\n", ds.dsBmih.biCompression); + + orig_bm = SelectObject(mem_dc, dib); + + sha1 = sha1_graphics_a8r8g8b8; + draw_graphics(mem_dc, bmi, bits, &sha1); + + SelectObject(mem_dc, orig_bm); + DeleteObject(dib); + + + /* r5g5b5 */ + bmi->bmiHeader.biBitCount = 16; + bmi->bmiHeader.biCompression = BI_RGB; + + dib = CreateDIBSection(0, bmi, DIB_RGB_COLORS, (void**)&bits, NULL, 0); + ok(dib != NULL, "ret NULL\n"); + ok(GetObjectW( dib, sizeof(ds), &ds ), "GetObject failed\n"); + ok(ds.dsBitfields[0] == 0x7c00, "got %08x\n", ds.dsBitfields[0]); + ok(ds.dsBitfields[1] == 0x03e0, "got %08x\n", ds.dsBitfields[1]); + ok(ds.dsBitfields[2] == 0x001f, "got %08x\n", ds.dsBitfields[2]); +todo_wine + ok(ds.dsBmih.biCompression == BI_BITFIELDS, "got %x\n", ds.dsBmih.biCompression); + + DeleteObject(dib); + DeleteDC(mem_dc); }

1 0

Jacek Caban : shdocvw: Register known protocols as URL protocols.
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: 9e18a77046558e544137aecdde853c48c8addcfa URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e18a77046558e544137aecdd… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Apr 12 13:27:39 2011 +0200 shdocvw: Register known protocols as URL protocols. --- dlls/shdocvw/shdocvw.rgs | 17 +++++++++++++++++ 1 files changed, 17 insertions(+), 0 deletions(-) diff --git a/dlls/shdocvw/shdocvw.rgs b/dlls/shdocvw/shdocvw.rgs index 9e9fcb4..8f8bc53 100644 --- a/dlls/shdocvw/shdocvw.rgs +++ b/dlls/shdocvw/shdocvw.rgs @@ -61,4 +61,21 @@ HKCR } } ForceRemove '.url' = s 'InternetShortcut' + + ftp + { + val 'URL Protocol' = s '' + } + http + { + val 'URL Protocol' = s '' + } + https + { + val 'URL Protocol' = s '' + } + mailto + { + val 'URL Protocol' = s '' + } }

1 0

Huw Davies : gdi32: Don' t use the bitfields for the BI_RGB case and also cope with zero masks.
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: bb28917bc2e1c939edef0a5fb234fa075a509bc4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bb28917bc2e1c939edef0a5fb… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Apr 12 12:49:08 2011 +0100 gdi32: Don't use the bitfields for the BI_RGB case and also cope with zero masks. --- dlls/gdi32/dibdrv/dc.c | 17 ++++++++++++++--- 1 files changed, 14 insertions(+), 3 deletions(-) diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index d784c94..e5048d4 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -40,6 +40,12 @@ static void calc_shift_and_len(DWORD mask, int *shift, int *len) { int s, l; + if(!mask) + { + *shift = *len = 0; + return; + } + s = 0; while ((mask & 1) == 0) { @@ -90,11 +96,16 @@ static BOOL init_dib(dib_info *dib, const BITMAPINFOHEADER *bi, const DWORD *bit switch(dib->bit_count) { case 32: - init_bit_fields(dib, bit_fields); - if(dib->red_mask == 0xff0000 && dib->green_mask == 0x00ff00 && dib->blue_mask == 0x0000ff) + if(bi->biCompression == BI_RGB) dib->funcs = &funcs_8888; else - dib->funcs = &funcs_32; + { + init_bit_fields(dib, bit_fields); + if(dib->red_mask == 0xff0000 && dib->green_mask == 0x00ff00 && dib->blue_mask == 0x0000ff) + dib->funcs = &funcs_8888; + else + dib->funcs = &funcs_32; + } break; default:

1 0

Alexandre Julliard : winex11: Get rid of the old ddraw HAL support.
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: f13c65b4f5f3d14a18afbd8e1743d27787571d7b URL: http://source.winehq.org/git/wine.git/?a=commit;h=f13c65b4f5f3d14a18afbd8e1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Apr 12 16:56:30 2011 +0200 winex11: Get rid of the old ddraw HAL support. --- dlls/winex11.drv/Makefile.in | 1 - dlls/winex11.drv/init.c | 9 - dlls/winex11.drv/settings.c | 37 ---- dlls/winex11.drv/x11ddraw.c | 453 ---------------------------------------- dlls/winex11.drv/x11drv.h | 5 - dlls/winex11.drv/x11drv_main.c | 4 - 6 files changed, 0 insertions(+), 509 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=f13c65b4f5f3d14a18afb…

1 0

Huw Davies : gdi32: Don't use the bitfields for the BI_RGB case.
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: 673dd73548e1d52afd89041f5e9970760cd0cf07 URL: http://source.winehq.org/git/wine.git/?a=commit;h=673dd73548e1d52afd89041f5… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Apr 12 12:46:24 2011 +0100 gdi32: Don't use the bitfields for the BI_RGB case. --- dlls/gdi32/dib.c | 15 +++++++++++++-- 1 files changed, 13 insertions(+), 2 deletions(-) diff --git a/dlls/gdi32/dib.c b/dlls/gdi32/dib.c index 96f6920..b63eb13 100644 --- a/dlls/gdi32/dib.c +++ b/dlls/gdi32/dib.c @@ -774,7 +774,17 @@ INT WINAPI GetDIBits( case 16: if (info->bmiHeader.biCompression == BI_BITFIELDS) { - if (bmp->dib) memcpy( info->bmiColors, bmp->dib->dsBitfields, 3 * sizeof(DWORD) ); + if (bmp->dib) + { + if (bmp->dib->dsBmih.biCompression == BI_BITFIELDS) + memcpy( info->bmiColors, bmp->dib->dsBitfields, 3 * sizeof(DWORD) ); + else + { + ((PDWORD)info->bmiColors)[0] = 0x7c00; + ((PDWORD)info->bmiColors)[1] = 0x03e0; + ((PDWORD)info->bmiColors)[2] = 0x001f; + } + } else { ((PDWORD)info->bmiColors)[0] = 0xf800; @@ -788,7 +798,8 @@ INT WINAPI GetDIBits( case 32: if (info->bmiHeader.biCompression == BI_BITFIELDS) { - if (bmp->dib) memcpy( info->bmiColors, bmp->dib->dsBitfields, 3 * sizeof(DWORD) ); + if (bmp->dib && bmp->dib->dsBmih.biCompression == BI_BITFIELDS) + memcpy( info->bmiColors, bmp->dib->dsBitfields, 3 * sizeof(DWORD) ); else { ((PDWORD)info->bmiColors)[0] = 0xff0000;

1 0

Jacek Caban : shdocvw: Added more InternetShortcut tests.
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: 6db795873d81b1248599db4efdcf39b368dc2bf8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6db795873d81b1248599db4ef… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Apr 12 13:27:55 2011 +0200 shdocvw: Added more InternetShortcut tests. --- dlls/shdocvw/tests/intshcut.c | 72 +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 72 insertions(+), 0 deletions(-) diff --git a/dlls/shdocvw/tests/intshcut.c b/dlls/shdocvw/tests/intshcut.c index 108e4f5..ca28450 100644 --- a/dlls/shdocvw/tests/intshcut.c +++ b/dlls/shdocvw/tests/intshcut.c @@ -128,6 +128,24 @@ static void test_QueryInterface(void) IUnknown_Release(pUnknown); } +#define test_shortcut_url(a,b) _test_shortcut_url(__LINE__,a,b) +static void _test_shortcut_url(unsigned line, IUnknown *unk, const char *exurl) +{ + IUniformResourceLocatorA *locator_a; + char *url_a; + HRESULT hres; + + hres = IUnknown_QueryInterface(unk, &IID_IUniformResourceLocatorA, (void**)&locator_a); + ok_(__FILE__,line)(hres == S_OK, "Could not get IUniformResourceLocatorA iface: %08x\n", hres); + + hres = locator_a->lpVtbl->GetURL(locator_a, &url_a); + ok_(__FILE__,line)(hres == S_OK, "GetURL failed: %08x\n", hres); + ok_(__FILE__,line)(!strcmp(url_a, exurl), "unexpected URL, got %s, expected %s\n", url_a, exurl); + CoTaskMemFree(url_a); + + IUnknown_Release(locator_a); +} + #define check_string_transform(a,b,c,d,e) _check_string_transform(__LINE__,a,b,c,d,e) static void _check_string_transform(unsigned line, IUniformResourceLocatorA *urlA, LPCSTR input, DWORD flags, LPCSTR expectedOutput, BOOL is_todo) @@ -314,6 +332,59 @@ static void test_NullURLs(void) urlA->lpVtbl->Release(urlA); } +typedef struct { + const char *data; + const char *url; +} load_test_t; + +static const load_test_t load_tests[] = { + {"[InternetShortcut]\n" + "URL=http://www.winehq.org/\n" + "HotKey=0\n" + "IDList=\n" + "[{000214A0-0000-0000-C000-000000000046}]\n" + "Prop0=1,2\n", + + "http://www.winehq.org/" + } +}; + +static void test_Load(void) +{ + IPersistFile *persist_file; + const load_test_t *test; + WCHAR file_path[MAX_PATH]; + DWORD size; + HANDLE file; + HRESULT hres; + + static const WCHAR test_urlW[] = {'t','e','s','t','.','u','r','l',0}; + + GetTempPathW(MAX_PATH, file_path); + lstrcatW(file_path, test_urlW); + + for(test = load_tests; test < load_tests + sizeof(load_tests)/sizeof(*load_tests); test++) { + file = CreateFileW(file_path, GENERIC_WRITE, 0, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL); + ok(file != INVALID_HANDLE_VALUE, "could not create test file\n"); + if(file == INVALID_HANDLE_VALUE) + continue; + + WriteFile(file, test->data, strlen(test->data), &size, NULL); + CloseHandle(file); + + hres = CoCreateInstance(&CLSID_InternetShortcut, NULL, CLSCTX_ALL, &IID_IPersistFile, (void**)&persist_file); + ok(hres == S_OK, "Could not create InternetShortcut instance: %08x\n", hres); + + hres = IPersistFile_Load(persist_file, file_path, 0); + ok(hres == S_OK, "Load failed: %08x\n", hres); + + test_shortcut_url((IUnknown*)persist_file, test->url); + + IPersistFile_Release(persist_file); + DeleteFileW(file_path); + } +} + static void test_SetURLFlags(void) { HRESULT hr; @@ -347,6 +418,7 @@ static void test_InternetShortcut(void) test_NullURLs(); test_SetURLFlags(); test_ReadAndWriteProperties(); + test_Load(); } START_TEST(intshcut)

1 0

Huw Davies : winex11: Don't use the bitfields for the BI_RGB case.
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: b295b53630d70673c0b857ef262f78903b96c0b1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b295b53630d70673c0b857ef2… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Apr 12 12:36:18 2011 +0100 winex11: Don't use the bitfields for the BI_RGB case. --- dlls/winex11.drv/xrender.c | 15 +++++++++++++-- 1 files changed, 13 insertions(+), 2 deletions(-) diff --git a/dlls/winex11.drv/xrender.c b/dlls/winex11.drv/xrender.c index fde0b91..33e420a 100644 --- a/dlls/winex11.drv/xrender.c +++ b/dlls/winex11.drv/xrender.c @@ -1132,14 +1132,25 @@ BOOL X11DRV_XRender_SetPhysBitmapDepth(X_PHYSBITMAP *physBitmap, int bits_pixel, if (dib) { - X11DRV_PALETTE_ComputeColorShifts(&shifts, dib->dsBitfields[0], dib->dsBitfields[1], dib->dsBitfields[2]); + const DWORD *bitfields; + static const DWORD bitfields_32[3] = {0xff0000, 0x00ff00, 0x0000ff}; + static const DWORD bitfields_16[3] = {0x7c00, 0x03e0, 0x001f}; + + if(dib->dsBmih.biCompression == BI_BITFIELDS) + bitfields = dib->dsBitfields; + else if(bits_pixel == 24 || bits_pixel == 32) + bitfields = bitfields_32; + else + bitfields = bitfields_16; + + X11DRV_PALETTE_ComputeColorShifts(&shifts, bitfields[0], bitfields[1], bitfields[2]); fmt = get_xrender_format_from_color_shifts(dib->dsBm.bmBitsPixel, &shifts); /* Common formats should be in our picture format table. */ if (!fmt) { TRACE("Unhandled dibsection format bpp=%d, redMask=%x, greenMask=%x, blueMask=%x\n", - dib->dsBm.bmBitsPixel, dib->dsBitfields[0], dib->dsBitfields[1], dib->dsBitfields[2]); + dib->dsBm.bmBitsPixel, bitfields[0], bitfields[1], bitfields[2]); return FALSE; } }

1 0

Stefan Dösinger : wined3d: Remove FLOAT16 vertex attribute conversion support.
by Alexandre Julliard April 12, 2011

April 12, 2011

Module: wine Branch: master Commit: 5627ee49ea54ff3be89ba4ba62462e8577123ec0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5627ee49ea54ff3be89ba4ba6… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Feb 6 23:47:49 2011 +0100 wined3d: Remove FLOAT16 vertex attribute conversion support. All mayor drivers support GL_ARB_half_float_vertex now and the half float conversion doesn't work well in practise. The only game that was ever playable with it was Eve online. --- dlls/wined3d/buffer.c | 318 ++++++++-------------------------------- dlls/wined3d/state.c | 32 +---- dlls/wined3d/wined3d_private.h | 3 - 3 files changed, 65 insertions(+), 288 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=5627ee49ea54ff3be89ba…

1 0