wine-commits October 2012

wine-commits@winehq.org

2 participants
960 discussions

Matteo Bruni : wined3d: Recognize Radeon HD 6970 as reported on some Macs.
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 3d71c2d1cae32543fa1415de9e30a47568fb62ff URL: http://source.winehq.org/git/wine.git/?a=commit;h=3d71c2d1cae32543fa1415de9… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Wed Oct 10 18:25:56 2012 +0200 wined3d: Recognize Radeon HD 6970 as reported on some Macs. --- dlls/wined3d/directx.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 1120a9e..24ed040 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -1708,6 +1708,7 @@ static enum wined3d_pci_device select_card_amd_binary(const struct wined3d_gl_in {"HD 7800", CARD_AMD_RADEON_HD7800}, {"HD 7700", CARD_AMD_RADEON_HD7700}, /* Northern Islands */ + {"HD 6970", CARD_AMD_RADEON_HD6900}, {"HD 6900", CARD_AMD_RADEON_HD6900}, {"HD 6800", CARD_AMD_RADEON_HD6800}, {"HD 6770M",CARD_AMD_RADEON_HD6600M},

1 0

Roman Dadkov : ole32: Fix dwClsContext parameter of a CoCreateInstance call in DefaultHandler_Run .
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 9e145eebdfbaa0fee22ff8c4eb7c33da67040981 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e145eebdfbaa0fee22ff8c4e… Author: Roman Dadkov <romand(a)etersoft.ru> Date: Tue Oct 9 14:49:50 2012 +0400 ole32: Fix dwClsContext parameter of a CoCreateInstance call in DefaultHandler_Run. --- dlls/ole32/defaulthandler.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/defaulthandler.c b/dlls/ole32/defaulthandler.c index 577d976..b4e8203 100644 --- a/dlls/ole32/defaulthandler.c +++ b/dlls/ole32/defaulthandler.c @@ -1322,7 +1322,7 @@ static HRESULT WINAPI DefaultHandler_Run( release_delegates(This); - hr = CoCreateInstance(&This->clsid, NULL, CLSCTX_LOCAL_SERVER, + hr = CoCreateInstance(&This->clsid, NULL, CLSCTX_ALL, &IID_IOleObject, (void **)&This->pOleDelegate); if (FAILED(hr)) return hr;

1 0

Jacek Caban : vbscript: Fixed tests on some non-English configs (without '. ' as decimal point).
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 2e1da7ec8b56227b1c377127069c41477443ed88 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2e1da7ec8b56227b1c3771270… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Oct 10 17:38:35 2012 +0200 vbscript: Fixed tests on some non-English configs (without '.' as decimal point). --- dlls/vbscript/tests/api.vbs | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/vbscript/tests/api.vbs b/dlls/vbscript/tests/api.vbs index ff188ab..7ccbaa1 100644 --- a/dlls/vbscript/tests/api.vbs +++ b/dlls/vbscript/tests/api.vbs @@ -47,7 +47,7 @@ End Sub TestCStr "test", "test" TestCStr 3, "3" -TestCStr 3.5, "3.5" +if isEnglishLang then TestCStr 3.5, "3.5" if isEnglishLang then TestCStr true, "True" Call ok(isObject(new EmptyClass), "isObject(new EmptyClass) is not true?")

1 0

Andrew Eikum : dsound: Don't exit early in mixer thread.
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 8ba4090fc3049930acd296b3b90529d7004830c0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8ba4090fc3049930acd296b3b… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Oct 10 10:17:10 2012 -0500 dsound: Don't exit early in mixer thread. --- dlls/dsound/mixer.c | 6 ------ 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/dlls/dsound/mixer.c b/dlls/dsound/mixer.c index fed9165..264d64d 100644 --- a/dlls/dsound/mixer.c +++ b/dlls/dsound/mixer.c @@ -684,12 +684,6 @@ static void DSOUND_PerformMix(DirectSoundDevice *device) to_mix_frags = device->prebuf - (pad * device->pwfx->nBlockAlign + device->fraglen - 1) / device->fraglen; - if(to_mix_frags == 0){ - /* nothing to do! */ - LeaveCriticalSection(&device->mixlock); - return; - } - to_mix_bytes = to_mix_frags * device->fraglen; if(device->in_mmdev_bytes > 0){

1 0

Andrew Eikum : vbscript: Undefined variables resolve as EMPTY without Option Explicit.
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 39d345320990ea9b581963ad1b2bdb3b456cde27 URL: http://source.winehq.org/git/wine.git/?a=commit;h=39d345320990ea9b581963ad1… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Oct 10 10:16:27 2012 -0500 vbscript: Undefined variables resolve as EMPTY without Option Explicit. --- dlls/vbscript/interp.c | 20 +++++++++++++++++--- dlls/vbscript/tests/run.c | 10 ++++++++++ 2 files changed, 27 insertions(+), 3 deletions(-) diff --git a/dlls/vbscript/interp.c b/dlls/vbscript/interp.c index 3a5d9b2..eace041 100644 --- a/dlls/vbscript/interp.c +++ b/dlls/vbscript/interp.c @@ -206,7 +206,8 @@ static HRESULT lookup_identifier(exec_ctx_t *ctx, BSTR name, vbdisp_invoke_type_ return S_OK; } -static HRESULT add_dynamic_var(exec_ctx_t *ctx, const WCHAR *name, BOOL is_const, VARIANT *val, BOOL own_val) +static HRESULT add_dynamic_var(exec_ctx_t *ctx, const WCHAR *name, + BOOL is_const, VARIANT *val, BOOL own_val, VARIANT **out_var) { dynamic_var_t *new_var; vbsheap_t *heap; @@ -245,6 +246,9 @@ static HRESULT add_dynamic_var(exec_ctx_t *ctx, const WCHAR *name, BOOL is_const ctx->dynamic_vars = new_var; } + if(out_var) + *out_var = &new_var->v; + return S_OK; } @@ -526,6 +530,16 @@ static HRESULT do_icall(exec_ctx_t *ctx, VARIANT *res) } break; case REF_NONE: + if(res && !ctx->func->code_ctx->option_explicit && arg_cnt == 0) { + VARIANT v, *new; + VariantInit(&v); + hres = add_dynamic_var(ctx, identifier, FALSE, &v, FALSE, &new); + if(FAILED(hres)) + return hres; + V_VT(res) = VT_BYREF|VT_VARIANT; + V_BYREF(res) = new; + break; + } FIXME("%s not found\n", debugstr_w(identifier)); return DISP_E_UNKNOWNNAME; } @@ -653,7 +667,7 @@ static HRESULT assign_ident(exec_ctx_t *ctx, BSTR name, DISPPARAMS *dp) } TRACE("creating variable %s\n", debugstr_w(name)); - hres = add_dynamic_var(ctx, name, FALSE, dp->rgvarg, FALSE); + hres = add_dynamic_var(ctx, name, FALSE, dp->rgvarg, FALSE, NULL); } } @@ -810,7 +824,7 @@ static HRESULT interp_const(exec_ctx_t *ctx) if(FAILED(hres)) return hres; - return add_dynamic_var(ctx, arg, TRUE, val.v, val.owned); + return add_dynamic_var(ctx, arg, TRUE, val.v, val.owned, NULL); } static HRESULT interp_val(exec_ctx_t *ctx) diff --git a/dlls/vbscript/tests/run.c b/dlls/vbscript/tests/run.c index 8c541bd..24a3c8d 100644 --- a/dlls/vbscript/tests/run.c +++ b/dlls/vbscript/tests/run.c @@ -1780,6 +1780,8 @@ static void run_from_res(const char *name) static void run_tests(void) { + HRESULT hres; + strict_dispid_check = TRUE; parse_script_a(""); @@ -1880,6 +1882,14 @@ static void run_tests(void) "End Sub\n" "Call testsub()"); + parse_script_a("Call ok(getVT(x) = \"VT_EMPTY*\", \"getVT(x) = \" & getVT(x))\n"); + parse_script_a("Call ok(x = \"\", \"x = \" & x)\n"); + parse_script_a("x = y\n" + "Call ok(getVT(x) = \"VT_EMPTY*\", \"getVT(x) = \" & getVT(x))\n" + "Call ok(getVT(y) = \"VT_EMPTY*\", \"getVT(y) = \" & getVT(y))"); + hres = parse_script_ar("x = y(\"a\")"); + ok(FAILED(hres), "script didn't fail\n"); + run_from_res("lang.vbs"); run_from_res("api.vbs");

1 0

Alexandre Julliard : winex11: Don' t return an alpha channel for DIBs read from the screen.
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: fad3ca07e6972920eb463935e1e125f0af4dd2be URL: http://source.winehq.org/git/wine.git/?a=commit;h=fad3ca07e6972920eb463935e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 10 17:09:01 2012 +0200 winex11: Don't return an alpha channel for DIBs read from the screen. --- dlls/winex11.drv/bitblt.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/winex11.drv/bitblt.c b/dlls/winex11.drv/bitblt.c index c3cf056..17ebde7 100644 --- a/dlls/winex11.drv/bitblt.c +++ b/dlls/winex11.drv/bitblt.c @@ -946,7 +946,7 @@ static inline int get_dib_image_size( const BITMAPINFO *info ) } /* store the palette or color mask data in the bitmap info structure */ -static void set_color_info( const XVisualInfo *vis, BITMAPINFO *info ) +static void set_color_info( const XVisualInfo *vis, BITMAPINFO *info, BOOL has_alpha ) { DWORD *colors = (DWORD *)((char *)info + info->bmiHeader.biSize); @@ -984,7 +984,7 @@ static void set_color_info( const XVisualInfo *vis, BITMAPINFO *info ) colors[0] = vis->red_mask; colors[1] = vis->green_mask; colors[2] = vis->blue_mask; - if (colors[0] != 0xff0000 || colors[1] != 0x00ff00 || colors[2] != 0x0000ff) + if (colors[0] != 0xff0000 || colors[1] != 0x00ff00 || colors[2] != 0x0000ff || !has_alpha) info->bmiHeader.biCompression = BI_BITFIELDS; break; } @@ -1291,7 +1291,7 @@ update_format: info->bmiHeader.biPlanes = 1; info->bmiHeader.biBitCount = format->bits_per_pixel; if (info->bmiHeader.biHeight > 0) info->bmiHeader.biHeight = -info->bmiHeader.biHeight; - set_color_info( &vis, info ); + set_color_info( &vis, info, FALSE ); return ERROR_BAD_FORMAT; } @@ -1339,7 +1339,7 @@ DWORD X11DRV_GetImage( PHYSDEV dev, BITMAPINFO *info, info->bmiHeader.biXPelsPerMeter = 0; info->bmiHeader.biYPelsPerMeter = 0; info->bmiHeader.biClrImportant = 0; - set_color_info( &vis, info ); + set_color_info( &vis, info, FALSE ); if (!bits) return ERROR_SUCCESS; /* just querying the color information */ @@ -1445,7 +1445,7 @@ update_format: info->bmiHeader.biPlanes = 1; info->bmiHeader.biBitCount = format->bits_per_pixel; if (info->bmiHeader.biHeight > 0) info->bmiHeader.biHeight = -info->bmiHeader.biHeight; - set_color_info( vis, info ); + set_color_info( vis, info, FALSE ); return ERROR_BAD_FORMAT; } @@ -1525,7 +1525,7 @@ DWORD get_pixmap_image( Pixmap pixmap, int width, int height, const XVisualInfo info->bmiHeader.biXPelsPerMeter = 0; info->bmiHeader.biYPelsPerMeter = 0; info->bmiHeader.biClrImportant = 0; - set_color_info( vis, info ); + set_color_info( vis, info, FALSE ); if (!bits) return ERROR_SUCCESS; /* just querying the color information */ @@ -1992,7 +1992,7 @@ struct window_surface *create_surface( Window window, const XVisualInfo *vis, co surface->info.bmiHeader.biPlanes = 1; surface->info.bmiHeader.biBitCount = format->bits_per_pixel; surface->info.bmiHeader.biSizeImage = get_dib_image_size( &surface->info ); - set_color_info( vis, &surface->info ); + set_color_info( vis, &surface->info, use_alpha ); InitializeCriticalSection( &surface->crit ); surface->crit.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": surface");

1 0

Alexandre Julliard : gdi32: Handle a missing source alpha directly in the blend_rect primitive instead of modifying the source data .
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 6e325afaa78f2037521e899838be91fa1dc35a16 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6e325afaa78f2037521e89983… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 10 17:19:42 2012 +0200 gdi32: Handle a missing source alpha directly in the blend_rect primitive instead of modifying the source data. --- dlls/gdi32/bitblt.c | 8 ++++---- dlls/gdi32/dib.c | 8 ++++---- dlls/gdi32/dibdrv/dc.c | 14 +------------- dlls/gdi32/dibdrv/primitives.c | 14 +++++++++++++- dlls/gdi32/gdi_private.h | 4 ++-- 5 files changed, 24 insertions(+), 24 deletions(-) diff --git a/dlls/gdi32/bitblt.c b/dlls/gdi32/bitblt.c index ea6d8a7..4750d1b 100644 --- a/dlls/gdi32/bitblt.c +++ b/dlls/gdi32/bitblt.c @@ -158,7 +158,7 @@ void free_heap_bits( struct gdi_image_bits *bits ) } DWORD convert_bits( const BITMAPINFO *src_info, struct bitblt_coords *src, - BITMAPINFO *dst_info, struct gdi_image_bits *bits, BOOL add_alpha ) + BITMAPINFO *dst_info, struct gdi_image_bits *bits ) { void *ptr; DWORD err; @@ -172,7 +172,7 @@ DWORD convert_bits( const BITMAPINFO *src_info, struct bitblt_coords *src, if (!(ptr = HeapAlloc( GetProcessHeap(), 0, dst_info->bmiHeader.biSizeImage ))) return ERROR_OUTOFMEMORY; - err = convert_bitmapinfo( src_info, bits->ptr, src, dst_info, ptr, add_alpha ); + err = convert_bitmapinfo( src_info, bits->ptr, src, dst_info, ptr ); if (bits->free) bits->free( bits ); bits->ptr = ptr; bits->is_copy = TRUE; @@ -287,7 +287,7 @@ BOOL nulldrv_StretchBlt( PHYSDEV dst_dev, struct bitblt_coords *dst, get_mono_dc_colors( src_dev->hdc, dst_info, 2 ); } - if (!(err = convert_bits( src_info, src, dst_info, &bits, FALSE ))) + if (!(err = convert_bits( src_info, src, dst_info, &bits ))) { /* get rid of the fake destination table */ dst_info->bmiHeader.biClrUsed = dst_colors; @@ -330,7 +330,7 @@ BOOL nulldrv_AlphaBlend( PHYSDEV dst_dev, struct bitblt_coords *dst, err = dst_dev->funcs->pBlendImage( dst_dev, dst_info, &bits, src, dst, func ); if (err == ERROR_BAD_FORMAT) { - err = convert_bits( src_info, src, dst_info, &bits, TRUE ); + err = convert_bits( src_info, src, dst_info, &bits ); if (!err) err = dst_dev->funcs->pBlendImage( dst_dev, dst_info, &bits, src, dst, func ); } diff --git a/dlls/gdi32/dib.c b/dlls/gdi32/dib.c index 91cfae7..eed57e8 100644 --- a/dlls/gdi32/dib.c +++ b/dlls/gdi32/dib.c @@ -572,7 +572,7 @@ INT nulldrv_StretchDIBits( PHYSDEV dev, INT xDst, INT yDst, INT widthDst, INT he dst_info->bmiHeader.biClrUsed = 1; } - if (!(err = convert_bits( src_info, &src, dst_info, &src_bits, FALSE ))) + if (!(err = convert_bits( src_info, &src, dst_info, &src_bits ))) { /* get rid of the fake 1-bpp table */ if (dst_info->bmiHeader.biClrUsed == 1) dst_info->bmiHeader.biClrUsed = 0; @@ -738,7 +738,7 @@ INT WINAPI SetDIBits( HDC hdc, HBITMAP hbitmap, UINT startscan, err = put_image_into_bitmap( bitmap, clip, dst_info, &src_bits, &src, &dst ); if (err == ERROR_BAD_FORMAT) { - err = convert_bits( src_info, &src, dst_info, &src_bits, FALSE ); + err = convert_bits( src_info, &src, dst_info, &src_bits ); if (!err) err = put_image_into_bitmap( bitmap, clip, dst_info, &src_bits, &src, &dst ); } if(err) result = 0; @@ -855,7 +855,7 @@ INT nulldrv_SetDIBitsToDevice( PHYSDEV dev, INT x_dst, INT y_dst, DWORD cx, DWOR err = dev->funcs->pPutImage( dev, clip, dst_info, &src_bits, &src, &dst, SRCCOPY ); if (err == ERROR_BAD_FORMAT) { - err = convert_bits( src_info, &src, dst_info, &src_bits, FALSE ); + err = convert_bits( src_info, &src, dst_info, &src_bits ); if (!err) err = dev->funcs->pPutImage( dev, clip, dst_info, &src_bits, &src, &dst, SRCCOPY ); } if (err) lines = 0; @@ -1387,7 +1387,7 @@ INT WINAPI GetDIBits( else dst_info->bmiHeader.biHeight = -src.height; - convert_bitmapinfo( src_info, src_bits.ptr, &src, dst_info, bits, FALSE ); + convert_bitmapinfo( src_info, src_bits.ptr, &src, dst_info, bits ); if (src_bits.free) src_bits.free( &src_bits ); ret = lines; } diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index 0e68f80..0b6013d 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -214,7 +214,7 @@ void copy_dib_color_info(dib_info *dst, const dib_info *src) } DWORD convert_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, struct bitblt_coords *src, - const BITMAPINFO *dst_info, void *dst_bits, BOOL add_alpha ) + const BITMAPINFO *dst_info, void *dst_bits ) { dib_info src_dib, dst_dib; DWORD ret; @@ -234,24 +234,12 @@ DWORD convert_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, struct bit } __ENDTRY - /* We shared the color tables, so there's no need to free the dib_infos here */ if(!ret) return ERROR_BAD_FORMAT; /* update coordinates, the destination rectangle is always stored at 0,0 */ src->x -= src->visrect.left; src->y -= src->visrect.top; offset_rect( &src->visrect, -src->visrect.left, -src->visrect.top ); - - if (add_alpha && dst_dib.funcs == &funcs_8888 && src_dib.funcs != &funcs_8888) - { - DWORD *pixel = dst_dib.bits.ptr; - int x, y; - - for (y = src->visrect.top; y < src->visrect.bottom; y++, pixel += dst_dib.stride / 4) - for (x = src->visrect.left; x < src->visrect.right; x++) - pixel[x] |= 0xff000000; - } - return ERROR_SUCCESS; } diff --git a/dlls/gdi32/dibdrv/primitives.c b/dlls/gdi32/dibdrv/primitives.c index 07dd1d5..6feb819 100644 --- a/dlls/gdi32/dibdrv/primitives.c +++ b/dlls/gdi32/dibdrv/primitives.c @@ -3838,6 +3838,14 @@ static inline DWORD blend_argb_constant_alpha( DWORD dst, DWORD src, DWORD alpha blend_color( dst >> 24, src >> 24, alpha ) << 24); } +static inline DWORD blend_argb_no_src_alpha( DWORD dst, DWORD src, DWORD alpha ) +{ + return (blend_color( dst, src, alpha ) | + blend_color( dst >> 8, src >> 8, alpha ) << 8 | + blend_color( dst >> 16, src >> 16, alpha ) << 16 | + blend_color( dst >> 24, 255, alpha ) << 24); +} + static inline DWORD blend_argb( DWORD dst, DWORD src ) { BYTE b = (BYTE)src; @@ -3898,10 +3906,14 @@ static void blend_rect_8888(const dib_info *dst, const RECT *rc, for (x = 0; x < rc->right - rc->left; x++) dst_ptr[x] = blend_argb_alpha( dst_ptr[x], src_ptr[x], blend.SourceConstantAlpha ); } - else + else if (src->compression == BI_RGB) for (y = rc->top; y < rc->bottom; y++, dst_ptr += dst->stride / 4, src_ptr += src->stride / 4) for (x = 0; x < rc->right - rc->left; x++) dst_ptr[x] = blend_argb_constant_alpha( dst_ptr[x], src_ptr[x], blend.SourceConstantAlpha ); + else + for (y = rc->top; y < rc->bottom; y++, dst_ptr += dst->stride / 4, src_ptr += src->stride / 4) + for (x = 0; x < rc->right - rc->left; x++) + dst_ptr[x] = blend_argb_no_src_alpha( dst_ptr[x], src_ptr[x], blend.SourceConstantAlpha ); } static void blend_rect_32(const dib_info *dst, const RECT *rc, diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index 29d4286..5b79526 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -222,7 +222,7 @@ extern BOOL BIDI_Reorder( HDC hDC, LPCWSTR lpString, INT uCount, DWORD dwFlags, /* bitblt.c */ extern DWORD convert_bits( const BITMAPINFO *src_info, struct bitblt_coords *src, - BITMAPINFO *dst_info, struct gdi_image_bits *bits, BOOL add_alpha ) DECLSPEC_HIDDEN; + BITMAPINFO *dst_info, struct gdi_image_bits *bits ) DECLSPEC_HIDDEN; extern BOOL intersect_vis_rectangles( struct bitblt_coords *dst, struct bitblt_coords *src ) DECLSPEC_HIDDEN; extern DWORD stretch_bits( const BITMAPINFO *src_info, struct bitblt_coords *src, BITMAPINFO *dst_info, struct bitblt_coords *dst, @@ -264,7 +264,7 @@ extern void fill_default_color_table( BITMAPINFO *info ) DECLSPEC_HIDDEN; extern void get_ddb_bitmapinfo( BITMAPOBJ *bmp, BITMAPINFO *info ) DECLSPEC_HIDDEN; extern BITMAPINFO *copy_packed_dib( const BITMAPINFO *src_info, UINT usage ) DECLSPEC_HIDDEN; extern DWORD convert_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, struct bitblt_coords *src, - const BITMAPINFO *dst_info, void *dst_bits, BOOL add_alpha ) DECLSPEC_HIDDEN; + const BITMAPINFO *dst_info, void *dst_bits ) DECLSPEC_HIDDEN; extern DWORD stretch_bitmapinfo( const BITMAPINFO *src_info, void *src_bits, struct bitblt_coords *src, const BITMAPINFO *dst_info, void *dst_bits, struct bitblt_coords *dst,

1 0

Alexandre Julliard : gdi32: Use BI_BITFIELDS to indicate that a 32-bpp DIB doesn' t have an alpha channel.
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 76272e077364d3d78ceaf8912e3bc22ff6a31a48 URL: http://source.winehq.org/git/wine.git/?a=commit;h=76272e077364d3d78ceaf8912… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 10 17:08:05 2012 +0200 gdi32: Use BI_BITFIELDS to indicate that a 32-bpp DIB doesn't have an alpha channel. --- dlls/gdi32/bitblt.c | 8 ++++++-- dlls/gdi32/dibdrv/bitblt.c | 8 ++++++-- dlls/gdi32/dibdrv/dc.c | 13 +++++++++++++ dlls/winex11.drv/xrender.c | 2 +- 4 files changed, 26 insertions(+), 5 deletions(-) diff --git a/dlls/gdi32/bitblt.c b/dlls/gdi32/bitblt.c index 151434f..ea6d8a7 100644 --- a/dlls/gdi32/bitblt.c +++ b/dlls/gdi32/bitblt.c @@ -357,6 +357,7 @@ DWORD nulldrv_BlendImage( PHYSDEV dev, BITMAPINFO *info, const struct gdi_image_ BITMAPINFO *dst_info = (BITMAPINFO *)buffer; struct gdi_image_bits dst_bits; struct bitblt_coords orig_dst; + DWORD *masks = (DWORD *)info->bmiColors; DC *dc = get_nulldrv_dc( dev ); DWORD err; @@ -364,7 +365,7 @@ DWORD nulldrv_BlendImage( PHYSDEV dev, BITMAPINFO *info, const struct gdi_image_ if (info->bmiHeader.biBitCount != 32) goto update_format; if (info->bmiHeader.biCompression == BI_BITFIELDS) { - DWORD *masks = (DWORD *)info->bmiColors; + if (blend.AlphaFormat & AC_SRC_ALPHA) return ERROR_INVALID_PARAMETER; if (masks[0] != 0xff0000 || masks[1] != 0x00ff00 || masks[2] != 0x0000ff) goto update_format; } @@ -390,8 +391,11 @@ update_format: info->bmiHeader.biPlanes = 1; info->bmiHeader.biBitCount = 32; - info->bmiHeader.biCompression = BI_RGB; + info->bmiHeader.biCompression = BI_BITFIELDS; info->bmiHeader.biClrUsed = 0; + masks[0] = 0xff0000; + masks[1] = 0x00ff00; + masks[2] = 0x0000ff; return ERROR_BAD_FORMAT; } diff --git a/dlls/gdi32/dibdrv/bitblt.c b/dlls/gdi32/dibdrv/bitblt.c index bcaa585..ccddac9 100644 --- a/dlls/gdi32/dibdrv/bitblt.c +++ b/dlls/gdi32/dibdrv/bitblt.c @@ -1001,6 +1001,7 @@ DWORD dibdrv_BlendImage( PHYSDEV dev, BITMAPINFO *info, const struct gdi_image_b { dibdrv_physdev *pdev = get_dibdrv_pdev( dev ); dib_info src_dib; + DWORD *masks = (DWORD *)info->bmiColors; TRACE( "%p %p\n", dev, info ); @@ -1008,7 +1009,7 @@ DWORD dibdrv_BlendImage( PHYSDEV dev, BITMAPINFO *info, const struct gdi_image_b if (info->bmiHeader.biBitCount != 32) goto update_format; if (info->bmiHeader.biCompression == BI_BITFIELDS) { - DWORD *masks = (DWORD *)info->bmiColors; + if (blend.AlphaFormat & AC_SRC_ALPHA) return ERROR_INVALID_PARAMETER; if (masks[0] != 0xff0000 || masks[1] != 0x00ff00 || masks[2] != 0x0000ff) goto update_format; } @@ -1027,8 +1028,11 @@ update_format: info->bmiHeader.biPlanes = 1; info->bmiHeader.biBitCount = 32; - info->bmiHeader.biCompression = BI_RGB; + info->bmiHeader.biCompression = BI_BITFIELDS; info->bmiHeader.biClrUsed = 0; + masks[0] = 0xff0000; + masks[1] = 0x00ff00; + masks[2] = 0x0000ff; return ERROR_BAD_FORMAT; } diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index 00e2229..0e68f80 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -760,6 +760,19 @@ static DWORD windrv_GetImage( PHYSDEV dev, BITMAPINFO *info, lock_surface( physdev->surface ); dev = GET_NEXT_PHYSDEV( dev, pGetImage ); ret = dev->funcs->pGetImage( dev, info, bits, src ); + + /* don't return alpha if original surface doesn't support it */ + if (info->bmiHeader.biBitCount == 32 && + info->bmiHeader.biCompression == BI_RGB && + physdev->dibdrv->dib.compression == BI_BITFIELDS) + { + DWORD *colors = (DWORD *)info->bmiColors; + colors[0] = 0xff0000; + colors[1] = 0x00ff00; + colors[2] = 0x0000ff; + info->bmiHeader.biCompression = BI_BITFIELDS; + } + if (!bits->is_copy) { /* use the freeing callback to unlock the surface */ diff --git a/dlls/winex11.drv/xrender.c b/dlls/winex11.drv/xrender.c index bdb8865..d7f0508 100644 --- a/dlls/winex11.drv/xrender.c +++ b/dlls/winex11.drv/xrender.c @@ -2077,7 +2077,7 @@ static DWORD xrenderdrv_BlendImage( PHYSDEV dev, BITMAPINFO *info, const struct format = get_xrender_format_from_bitmapinfo( info ); if (!(func.AlphaFormat & AC_SRC_ALPHA)) format = get_format_without_alpha( format ); - else if (format != WXR_FORMAT_A8R8G8B8) + else if (format != WXR_FORMAT_A8R8G8B8 || info->bmiHeader.biCompression != BI_RGB) return ERROR_INVALID_PARAMETER; if (!(pict_format = pict_formats[format])) goto update_format;

1 0

Alexandre Julliard : gdi32: Limit the allocation size to the useful data when converting DIB bits.
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 34b70728f386eda7b67eb77aa4ebae8c00c4a3d3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=34b70728f386eda7b67eb77aa… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 10 17:06:06 2012 +0200 gdi32: Limit the allocation size to the useful data when converting DIB bits. --- dlls/gdi32/bitblt.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/gdi32/bitblt.c b/dlls/gdi32/bitblt.c index 7a1f06f..151434f 100644 --- a/dlls/gdi32/bitblt.c +++ b/dlls/gdi32/bitblt.c @@ -162,9 +162,12 @@ DWORD convert_bits( const BITMAPINFO *src_info, struct bitblt_coords *src, { void *ptr; DWORD err; + BOOL top_down = dst_info->bmiHeader.biHeight < 0; dst_info->bmiHeader.biWidth = src->visrect.right - src->visrect.left; + dst_info->bmiHeader.biHeight = src->visrect.bottom - src->visrect.top; dst_info->bmiHeader.biSizeImage = get_dib_image_size( dst_info ); + if (top_down) dst_info->bmiHeader.biHeight = -dst_info->bmiHeader.biHeight; if (!(ptr = HeapAlloc( GetProcessHeap(), 0, dst_info->bmiHeader.biSizeImage ))) return ERROR_OUTOFMEMORY;

1 0

Alexandre Julliard : gdi32: Return the full DIB height in GetImage instead of clipping to the visible rect .
by Alexandre Julliard 10 Oct '12

10 Oct '12

Module: wine Branch: master Commit: 86cf1126a5cc6035e21db3df01088827f5b2245d URL: http://source.winehq.org/git/wine.git/?a=commit;h=86cf1126a5cc6035e21db3df0… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 10 17:05:41 2012 +0200 gdi32: Return the full DIB height in GetImage instead of clipping to the visible rect. --- dlls/gdi32/dibdrv/bitblt.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dlls/gdi32/dibdrv/bitblt.c b/dlls/gdi32/dibdrv/bitblt.c index bac5a60..bcaa585 100644 --- a/dlls/gdi32/dibdrv/bitblt.c +++ b/dlls/gdi32/dibdrv/bitblt.c @@ -812,12 +812,11 @@ static DWORD get_image_dib_info( dib_info *dib, BITMAPINFO *info, { info->bmiHeader.biSize = sizeof(info->bmiHeader); info->bmiHeader.biPlanes = 1; - info->bmiHeader.biCompression = BI_RGB; info->bmiHeader.biXPelsPerMeter = 0; info->bmiHeader.biYPelsPerMeter = 0; info->bmiHeader.biClrImportant = 0; info->bmiHeader.biWidth = dib->width; - info->bmiHeader.biHeight = dib->rect.bottom - dib->rect.top; + info->bmiHeader.biHeight = dib->height - dib->rect.top; info->bmiHeader.biBitCount = dib->bit_count; info->bmiHeader.biSizeImage = info->bmiHeader.biHeight * abs( dib->stride ); if (dib->stride > 0) info->bmiHeader.biHeight = -info->bmiHeader.biHeight;

1 0

← Newer
1
...
62
63
64
65
66
67
68
...
96
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits October 2012