December 2012 - Wine-Commits

Stefan Dösinger : wined3d: Add GL_ALPHA to the atifs argument replicator debug function.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: 8e05ee699475677e3c254f342c62396fc04e6267 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8e05ee699475677e3c254f342… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Wed Dec 5 09:01:37 2012 +0100 wined3d: Add GL_ALPHA to the atifs argument replicator debug function. --- dlls/wined3d/ati_fragment_shader.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/wined3d/ati_fragment_shader.c b/dlls/wined3d/ati_fragment_shader.c index 6aa91a6..4994d96 100644 --- a/dlls/wined3d/ati_fragment_shader.c +++ b/dlls/wined3d/ati_fragment_shader.c @@ -157,6 +157,7 @@ static const char *debug_rep(GLuint rep) { case GL_RED: return "GL_RED"; case GL_GREEN: return "GL_GREEN"; case GL_BLUE: return "GL_BLUE"; + case GL_ALPHA: return "GL_ALPHA"; default: return "unknown argrep"; } }

1 0

Alexandre Julliard : wineps: Support getting character codes instead of glyphs in ExtTextOut.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: 55353027581472ea0ce1add7940e5fa43373236d URL: http://source.winehq.org/git/wine.git/?a=commit;h=55353027581472ea0ce1add79… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 5 17:23:49 2012 +0100 wineps: Support getting character codes instead of glyphs in ExtTextOut. --- dlls/wineps.drv/download.c | 2 +- dlls/wineps.drv/psdrv.h | 2 +- dlls/wineps.drv/text.c | 15 ++++++++++----- 3 files changed, 12 insertions(+), 7 deletions(-) diff --git a/dlls/wineps.drv/download.c b/dlls/wineps.drv/download.c index f984225..5163cef 100644 --- a/dlls/wineps.drv/download.c +++ b/dlls/wineps.drv/download.c @@ -366,7 +366,7 @@ void get_glyph_name(HDC hdc, WORD index, char *name) * Download and write out a number of glyphs * */ -BOOL PSDRV_WriteDownloadGlyphShow(PHYSDEV dev, WORD *glyphs, +BOOL PSDRV_WriteDownloadGlyphShow(PHYSDEV dev, const WORD *glyphs, UINT count) { PSDRV_PDEVICE *physDev = get_psdrv_dev( dev ); diff --git a/dlls/wineps.drv/psdrv.h b/dlls/wineps.drv/psdrv.h index 1e76bef..bfc2ad6 100644 --- a/dlls/wineps.drv/psdrv.h +++ b/dlls/wineps.drv/psdrv.h @@ -539,7 +539,7 @@ extern BOOL PSDRV_WriteBuiltinGlyphShow(PHYSDEV dev, LPCWSTR str, INT count) DEC extern BOOL PSDRV_SelectDownloadFont(PHYSDEV dev) DECLSPEC_HIDDEN; extern BOOL PSDRV_WriteSetDownloadFont(PHYSDEV dev) DECLSPEC_HIDDEN; -extern BOOL PSDRV_WriteDownloadGlyphShow(PHYSDEV dev, WORD *glpyhs, UINT count) DECLSPEC_HIDDEN; +extern BOOL PSDRV_WriteDownloadGlyphShow(PHYSDEV dev, const WORD *glpyhs, UINT count) DECLSPEC_HIDDEN; extern BOOL PSDRV_EmptyDownloadList(PHYSDEV dev, BOOL write_undef) DECLSPEC_HIDDEN; extern DWORD write_spool( PHYSDEV dev, const void *data, DWORD num ) DECLSPEC_HIDDEN; diff --git a/dlls/wineps.drv/text.c b/dlls/wineps.drv/text.c index a35945a..6b5fe45 100644 --- a/dlls/wineps.drv/text.c +++ b/dlls/wineps.drv/text.c @@ -100,14 +100,18 @@ static BOOL PSDRV_Text(PHYSDEV dev, INT x, INT y, UINT flags, LPCWSTR str, if (!count) return TRUE; - if(physDev->font.fontloc == Download) - glyphs = (LPWORD)str; + if(physDev->font.fontloc == Download && !(flags & ETO_GLYPH_INDEX)) + { + glyphs = HeapAlloc( GetProcessHeap(), 0, count * sizeof(WORD) ); + GetGlyphIndicesW( dev->hdc, str, count, glyphs, 0 ); + str = glyphs; + } PSDRV_WriteMoveTo(dev, x, y); if(!lpDx) { if(physDev->font.fontloc == Download) - PSDRV_WriteDownloadGlyphShow(dev, glyphs, count); + PSDRV_WriteDownloadGlyphShow(dev, str, count); else PSDRV_WriteBuiltinGlyphShow(dev, str, count); } @@ -117,7 +121,7 @@ static BOOL PSDRV_Text(PHYSDEV dev, INT x, INT y, UINT flags, LPCWSTR str, for(i = 0; i < count-1; i++) { if(physDev->font.fontloc == Download) - PSDRV_WriteDownloadGlyphShow(dev, glyphs + i, 1); + PSDRV_WriteDownloadGlyphShow(dev, str + i, 1); else PSDRV_WriteBuiltinGlyphShow(dev, str + i, 1); if(flags & ETO_PDY) @@ -130,10 +134,11 @@ static BOOL PSDRV_Text(PHYSDEV dev, INT x, INT y, UINT flags, LPCWSTR str, PSDRV_WriteMoveTo(dev, x + offset.x, y + offset.y); } if(physDev->font.fontloc == Download) - PSDRV_WriteDownloadGlyphShow(dev, glyphs + i, 1); + PSDRV_WriteDownloadGlyphShow(dev, str + i, 1); else PSDRV_WriteBuiltinGlyphShow(dev, str + i, 1); } + HeapFree( GetProcessHeap(), 0, glyphs ); return TRUE; }

1 0

Jacek Caban : jscript: Treat VT_UNKNOWN VARIANTs supporting IDispatch interface as VT_DISPATCH.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: 11f41aa05a993bc1c6b6d6b7a672632cb906b649 URL: http://source.winehq.org/git/wine.git/?a=commit;h=11f41aa05a993bc1c6b6d6b7a… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Dec 5 14:49:42 2012 +0100 jscript: Treat VT_UNKNOWN VARIANTs supporting IDispatch interface as VT_DISPATCH. --- dlls/jscript/jsutils.c | 12 ++++++++++++ dlls/jscript/tests/run.c | 23 ++++++++++++++++++++--- 2 files changed, 32 insertions(+), 3 deletions(-) diff --git a/dlls/jscript/jsutils.c b/dlls/jscript/jsutils.c index ac26ef3..91e0397 100644 --- a/dlls/jscript/jsutils.c +++ b/dlls/jscript/jsutils.c @@ -315,6 +315,18 @@ HRESULT variant_to_jsval(VARIANT *var, jsval_t *r) case VT_INT: *r = jsval_number(V_INT(var)); return S_OK; + case VT_UNKNOWN: + if(V_UNKNOWN(var)) { + IDispatch *disp; + HRESULT hres; + + hres = IUnknown_QueryInterface(V_UNKNOWN(var), &IID_IDispatch, (void**)&disp); + if(SUCCEEDED(hres)) { + *r = jsval_disp(disp); + return S_OK; + } + } + /* fall through */ default: return jsval_variant(r, var); } diff --git a/dlls/jscript/tests/run.c b/dlls/jscript/tests/run.c index cacad25..3c5422c 100644 --- a/dlls/jscript/tests/run.c +++ b/dlls/jscript/tests/run.c @@ -125,6 +125,7 @@ DEFINE_EXPECT(DeleteMemberByDispID); #define DISPID_GLOBAL_GETSHORT 0x1014 #define DISPID_GLOBAL_TESTARGTYPES 0x1015 #define DISPID_GLOBAL_INTPROP 0x1016 +#define DISPID_GLOBAL_DISPUNK 0x1017 #define DISPID_GLOBAL_TESTPROPDELETE 0x2000 @@ -592,6 +593,11 @@ static HRESULT WINAPI Global_GetDispID(IDispatchEx *iface, BSTR bstrName, DWORD return S_OK; } + if(!strcmp_wa(bstrName, "dispUnk")) { + *pid = DISPID_GLOBAL_DISPUNK; + return S_OK; + } + if(strict_dispid_check && strcmp_wa(bstrName, "t")) ok(0, "unexpected call %s\n", wine_dbgstr_w(bstrName)); return DISP_E_UNKNOWNNAME; @@ -987,8 +993,13 @@ static HRESULT WINAPI Global_InvokeEx(IDispatchEx *iface, DISPID id, LCID lcid, V_INT(pvarRes) = 22; return S_OK; + case DISPID_GLOBAL_DISPUNK: + V_VT(pvarRes) = VT_UNKNOWN; + V_UNKNOWN(pvarRes) = (IUnknown*)&testObj; + return S_OK; + case DISPID_GLOBAL_TESTARGTYPES: { - VARIANT args[2]; + VARIANT args[3]; DISPPARAMS dp = {args, NULL, sizeof(args)/sizeof(*args), 0}; HRESULT hres; @@ -996,7 +1007,7 @@ static HRESULT WINAPI Global_InvokeEx(IDispatchEx *iface, DISPID id, LCID lcid, ok(wFlags == DISPATCH_METHOD, "wFlags = %x\n", wFlags); ok(pdp != NULL, "pdp == NULL\n"); ok(pdp->rgvarg != NULL, "rgvarg != NULL\n"); - ok(pdp->cArgs == 5, "cArgs = %d\n", pdp->cArgs); + ok(pdp->cArgs == 6, "cArgs = %d\n", pdp->cArgs); ok(!pvarRes, "pvarRes != NULL\n"); ok(V_VT(pdp->rgvarg+1) == VT_I4, "V_VT(pdp->rgvarg+1) = %d\n", V_VT(pdp->rgvarg+1)); @@ -1011,12 +1022,17 @@ static HRESULT WINAPI Global_InvokeEx(IDispatchEx *iface, DISPID id, LCID lcid, ok(V_VT(pdp->rgvarg+4) == VT_I4, "V_VT(pdp->rgvarg+4) = %d\n", V_VT(pdp->rgvarg+4)); ok(V_I4(pdp->rgvarg+4) == 22, "V_I4(pdp->rgvarg+4) = %d\n", V_I4(pdp->rgvarg+4)); + ok(V_VT(pdp->rgvarg+5) == VT_DISPATCH, "V_VT(pdp->rgvarg+5) = %d\n", V_VT(pdp->rgvarg+5)); + ok(V_DISPATCH(pdp->rgvarg+5) == (IDispatch*)&testObj, "V_DISPATCH(pdp->rgvarg+5) != testObj\n"); + ok(V_VT(pdp->rgvarg) == VT_DISPATCH, "V_VT(pdp->rgvarg) = %d\n", V_VT(pdp->rgvarg)); V_VT(args) = VT_I2; V_I2(args) = 2; V_VT(args+1) = VT_INT; V_INT(args+1) = 22; + V_VT(args+2) = VT_UNKNOWN; + V_UNKNOWN(args+2) = (IUnknown*)&testObj; hres = IDispatch_Invoke(V_DISPATCH(pdp->rgvarg), DISPID_VALUE, &IID_NULL, 0, DISPATCH_METHOD, &dp, NULL, NULL, NULL); ok(hres == S_OK, "Invoke failed: %08x\n", hres); @@ -1839,9 +1855,10 @@ static BOOL run_tests(void) CHECK_CALLED(global_propargput_i); SET_EXPECT(global_testargtypes_i); - parse_script_a("testArgTypes(intProp(), intProp, getShort(), shortProp, function(i,s) {" + parse_script_a("testArgTypes(dispUnk, intProp(), intProp, getShort(), shortProp, function(d,i,s) {" " ok(getVT(i) === 'VT_I4', 'getVT(i) = ' + getVT(i));" " ok(getVT(s) === 'VT_I4', 'getVT(s) = ' + getVT(s));" + " ok(getVT(d) === 'VT_DISPATCH', 'getVT(d) = ' + getVT(d));" "});"); CHECK_CALLED(global_testargtypes_i);

1 0

Alexandre Julliard : gdi32: Remove special handling for linked fonts in ExtTextOut.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: 6144824965ec6e6a323979f18df7b15feaf3f0a4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6144824965ec6e6a323979f18… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 5 16:44:15 2012 +0100 gdi32: Remove special handling for linked fonts in ExtTextOut. --- dlls/gdi32/font.c | 75 +-------------------------------------------- dlls/gdi32/freetype.c | 26 ---------------- dlls/gdi32/gdi_private.h | 1 - 3 files changed, 2 insertions(+), 100 deletions(-) diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 8580fcb..f84e707 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -2371,79 +2371,8 @@ BOOL WINAPI ExtTextOutW( HDC hdc, INT x, INT y, UINT flags, } } - if(FontIsLinked(hdc) && !(flags & ETO_GLYPH_INDEX)) - { - HFONT orig_font = dc->hFont, cur_font; - UINT glyph; - INT span = 0; - POINT *offsets = NULL; - unsigned int i; - - glyphs = HeapAlloc(GetProcessHeap(), 0, count * sizeof(WORD)); - for(i = 0; i < count; i++) - { - WineEngGetLinkedHFont(dc, reordered_str[i], &cur_font, &glyph); - if(cur_font != dc->hFont) - { - if(!offsets) - { - unsigned int j; - offsets = HeapAlloc(GetProcessHeap(), 0, count * sizeof(*deltas)); - offsets[0].x = offsets[0].y = 0; - - if(!deltas) - { - SIZE tmpsz; - for(j = 1; j < count; j++) - { - GetTextExtentPointW(hdc, reordered_str + j - 1, 1, &tmpsz); - offsets[j].x = offsets[j - 1].x + abs(INTERNAL_XWSTODS(dc, tmpsz.cx)); - offsets[j].y = 0; - } - } - else - { - for(j = 1; j < count; j++) - { - offsets[j].x = offsets[j - 1].x + deltas[j].x; - offsets[j].y = offsets[j - 1].y + deltas[j].y; - } - } - } - if(span) - { - physdev->funcs->pExtTextOut( physdev, x + offsets[i - span].x, - y + offsets[i - span].y, - (flags & ~ETO_OPAQUE) | ETO_GLYPH_INDEX, &rc, glyphs, - span, deltas ? (INT*)(deltas + (i - span)) : NULL); - span = 0; - } - SelectObject(hdc, cur_font); - } - glyphs[span++] = glyph; - - if(i == count - 1) - { - ret = physdev->funcs->pExtTextOut(physdev, x + (offsets ? offsets[count - span].x : 0), - y + (offsets ? offsets[count - span].y : 0), - (flags & ~ETO_OPAQUE) | ETO_GLYPH_INDEX, &rc, glyphs, - span, deltas ? (INT*)(deltas + (count - span)) : NULL); - SelectObject(hdc, orig_font); - HeapFree(GetProcessHeap(), 0, offsets); - } - } - } - else - { - if(!(flags & ETO_GLYPH_INDEX) && dc->gdiFont) - { - glyphs = HeapAlloc(GetProcessHeap(), 0, count * sizeof(WORD)); - GetGlyphIndicesW(hdc, reordered_str, count, glyphs, 0); - flags |= ETO_GLYPH_INDEX; - } - ret = physdev->funcs->pExtTextOut( physdev, x, y, (flags & ~ETO_OPAQUE), &rc, - glyphs ? glyphs : reordered_str, count, (INT*)deltas ); - } + ret = physdev->funcs->pExtTextOut( physdev, x, y, (flags & ~ETO_OPAQUE), &rc, + glyphs ? glyphs : reordered_str, count, (INT*)deltas ); done: HeapFree(GetProcessHeap(), 0, deltas); diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index c5543e1..f9db1d7 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -7431,27 +7431,6 @@ static UINT freetype_GetTextCharsetInfo( PHYSDEV dev, LPFONTSIGNATURE fs, DWORD return physdev->font->charset; } -BOOL WineEngGetLinkedHFont(DC *dc, WCHAR c, HFONT *new_hfont, UINT *glyph) -{ - GdiFont *font = dc->gdiFont, *linked_font; - struct list *first_hfont; - BOOL ret; - - GDI_CheckNotLock(); - EnterCriticalSection( &freetype_cs ); - ret = get_glyph_index_linked(font, c, &linked_font, glyph); - TRACE("get_glyph_index_linked glyph %d font %p\n", *glyph, linked_font); - if(font == linked_font) - *new_hfont = dc->hFont; - else - { - first_hfont = list_head(&linked_font->hfontlist); - *new_hfont = LIST_ENTRY(first_hfont, struct tagHFONTLIST, entry)->hfont; - } - LeaveCriticalSection( &freetype_cs ); - return ret; -} - /* Retrieve a list of supported Unicode ranges for a given font. * Can be called with NULL gs to calculate the buffer size. Returns * the number of ranges found. @@ -8020,11 +7999,6 @@ BOOL WineEngCreateScalableFontResource( DWORD hidden, LPCWSTR resource, return FALSE; } -BOOL WineEngGetLinkedHFont(DC *dc, WCHAR c, HFONT *new_hfont, UINT *glyph) -{ - return FALSE; -} - /************************************************************************* * GetRasterizerCaps (GDI32.@) */ diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index e3fe577..2408c97 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -294,7 +294,6 @@ extern INT WineEngAddFontResourceEx(LPCWSTR, DWORD, PVOID) DECLSPEC_HIDDEN; extern HANDLE WineEngAddFontMemResourceEx(PVOID, DWORD, PVOID, LPDWORD) DECLSPEC_HIDDEN; extern BOOL WineEngCreateScalableFontResource(DWORD, LPCWSTR, LPCWSTR, LPCWSTR) DECLSPEC_HIDDEN; extern BOOL WineEngDestroyFontInstance(HFONT handle) DECLSPEC_HIDDEN; -extern BOOL WineEngGetLinkedHFont(DC *dc, WCHAR c, HFONT *new_hfont, UINT *glyph) DECLSPEC_HIDDEN; extern BOOL WineEngInit(void) DECLSPEC_HIDDEN; extern BOOL WineEngRemoveFontResourceEx(LPCWSTR, DWORD, PVOID) DECLSPEC_HIDDEN;

1 0

Alexandre Julliard : gdi32: Support caching glyphs by character code and not only by glyph index.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: 78626cdbaa0419e78d5bf5537e084b0e8cef9234 URL: http://source.winehq.org/git/wine.git/?a=commit;h=78626cdbaa0419e78d5bf5537… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 5 16:41:08 2012 +0100 gdi32: Support caching glyphs by character code and not only by glyph index. --- dlls/gdi32/dibdrv/graphics.c | 61 ++++++++++++++++++++++++++--------------- dlls/gdi32/font.c | 12 ++++---- 2 files changed, 45 insertions(+), 28 deletions(-) diff --git a/dlls/gdi32/dibdrv/graphics.c b/dlls/gdi32/dibdrv/graphics.c index 0b843fe..d0b7a39 100644 --- a/dlls/gdi32/dibdrv/graphics.c +++ b/dlls/gdi32/dibdrv/graphics.c @@ -33,6 +33,13 @@ struct cached_glyph BYTE bits[1]; }; +enum glyph_type +{ + GLYPH_INDEX, + GLYPH_WCHAR, + GLYPH_NBTYPES +}; + struct cached_font { struct list entry; @@ -41,8 +48,8 @@ struct cached_font LOGFONTW lf; XFORM xform; UINT aa_flags; - UINT nb_glyphs; - struct cached_glyph **glyphs; + UINT nb_glyphs[GLYPH_NBTYPES]; + struct cached_glyph **glyphs[GLYPH_NBTYPES]; }; static struct list font_cache = LIST_INIT( font_cache ); @@ -487,7 +494,7 @@ static int font_cache_cmp( const struct cached_font *p1, const struct cached_fon static struct cached_font *add_cached_font( HDC hdc, HFONT hfont, UINT aa_flags ) { struct cached_font font, *ptr, *last_unused = NULL; - UINT i = 0; + UINT i = 0, j; GetObjectW( hfont, sizeof(font.lf), &font.lf ); GetTransform( hdc, 0x204, &font.xform ); @@ -517,8 +524,11 @@ static struct cached_font *add_cached_font( HDC hdc, HFONT hfont, UINT aa_flags if (i > 5) /* keep at least 5 of the most-recently used fonts around */ { ptr = last_unused; - for (i = 0; i < ptr->nb_glyphs; i++) HeapFree( GetProcessHeap(), 0, ptr->glyphs[i] ); - HeapFree( GetProcessHeap(), 0, ptr->glyphs ); + for (i = 0; i < GLYPH_NBTYPES; i++) + { + for (j = 0; j < ptr->nb_glyphs[i]; j++) HeapFree( GetProcessHeap(), 0, ptr->glyphs[i][j] ); + HeapFree( GetProcessHeap(), 0, ptr->glyphs[i] ); + } list_remove( &ptr->entry ); } else if (!(ptr = HeapAlloc( GetProcessHeap(), 0, sizeof(*ptr) ))) @@ -529,9 +539,11 @@ static struct cached_font *add_cached_font( HDC hdc, HFONT hfont, UINT aa_flags *ptr = font; ptr->ref = 1; - ptr->glyphs = NULL; - ptr->nb_glyphs = 0; - + for (i = 0; i < GLYPH_NBTYPES; i++) + { + ptr->glyphs[i] = NULL; + ptr->nb_glyphs[i] = 0; + } done: list_add_head( &font_cache, &ptr->entry ); LeaveCriticalSection( &font_cache_cs ); @@ -544,28 +556,32 @@ void release_cached_font( struct cached_font *font ) if (font) InterlockedDecrement( &font->ref ); } -static void add_cached_glyph( struct cached_font *font, UINT index, struct cached_glyph *glyph ) +static void add_cached_glyph( struct cached_font *font, UINT index, UINT flags, struct cached_glyph *glyph ) { - if (index >= font->nb_glyphs) + enum glyph_type type = (flags & ETO_GLYPH_INDEX) ? GLYPH_INDEX : GLYPH_WCHAR; + + if (index >= font->nb_glyphs[type]) { UINT new_count = (index + 128) & ~127; struct cached_glyph **new; - if (font->glyphs) + if (font->glyphs[type]) new = HeapReAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, - font->glyphs, new_count * sizeof(*new) ); + font->glyphs[type], new_count * sizeof(*new) ); else new = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, new_count * sizeof(*new) ); if (!new) return; - font->glyphs = new; - font->nb_glyphs = new_count; + font->glyphs[type] = new; + font->nb_glyphs[type] = new_count; } - font->glyphs[index] = glyph; + font->glyphs[type][index] = glyph; } -static struct cached_glyph *get_cached_glyph( struct cached_font *font, UINT index ) +static struct cached_glyph *get_cached_glyph( struct cached_font *font, UINT index, UINT flags ) { - if (index < font->nb_glyphs) return font->glyphs[index]; + enum glyph_type type = (flags & ETO_GLYPH_INDEX) ? GLYPH_INDEX : GLYPH_WCHAR; + + if (index < font->nb_glyphs[type]) return font->glyphs[type][index]; return NULL; } @@ -654,9 +670,9 @@ static const int padding[4] = {0, 3, 2, 1}; * For non-antialiased bitmaps convert them to the 17-level format * using only values 0 or 16. */ -static struct cached_glyph *cache_glyph_bitmap( HDC hdc, struct cached_font *font, UINT index ) +static struct cached_glyph *cache_glyph_bitmap( HDC hdc, struct cached_font *font, UINT index, UINT flags ) { - UINT ggo_flags = font->aa_flags | GGO_GLYPH_INDEX; + UINT ggo_flags = font->aa_flags; static const MAT2 identity = { {0,1}, {0,0}, {0,0}, {0,1} }; UINT indices[3] = {0, 0, 0x20}; int i, x, y; @@ -666,6 +682,7 @@ static struct cached_glyph *cache_glyph_bitmap( HDC hdc, struct cached_font *fon GLYPHMETRICS metrics; struct cached_glyph *glyph; + if (flags & ETO_GLYPH_INDEX) ggo_flags |= GGO_GLYPH_INDEX; indices[0] = index; for (i = 0; i < sizeof(indices) / sizeof(indices[0]); i++) { @@ -712,7 +729,7 @@ static struct cached_glyph *cache_glyph_bitmap( HDC hdc, struct cached_font *fon done: glyph->metrics = metrics; - add_cached_glyph( font, index, glyph ); + add_cached_glyph( font, index, flags, glyph ); return glyph; } @@ -740,8 +757,8 @@ static void render_string( HDC hdc, dib_info *dib, struct cached_font *font, INT EnterCriticalSection( &font_cache_cs ); for (i = 0; i < count; i++) { - if (!(glyph = get_cached_glyph( font, str[i] )) && - !(glyph = cache_glyph_bitmap( hdc, font, str[i] ))) continue; + if (!(glyph = get_cached_glyph( font, str[i], flags )) && + !(glyph = cache_glyph_bitmap( hdc, font, str[i], flags ))) continue; glyph_dib.width = glyph->metrics.gmBlackBoxX; glyph_dib.height = glyph->metrics.gmBlackBoxY; diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 027c392..8580fcb 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -1710,10 +1710,9 @@ BOOL WINAPI GetCharWidth32A( HDC hdc, UINT firstChar, UINT lastChar, /* helper for nulldrv_ExtTextOut */ -static DWORD get_glyph_bitmap( HDC hdc, UINT index, UINT aa_flags, +static DWORD get_glyph_bitmap( HDC hdc, UINT index, UINT flags, UINT aa_flags, GLYPHMETRICS *metrics, struct gdi_image_bits *image ) { - UINT ggo_flags = aa_flags | GGO_GLYPH_INDEX; static const MAT2 identity = { {0,1}, {0,0}, {0,0}, {0,1} }; UINT indices[3] = {0, 0, 0x20}; int i; @@ -1721,11 +1720,12 @@ static DWORD get_glyph_bitmap( HDC hdc, UINT index, UINT aa_flags, int stride; indices[0] = index; + if (flags & ETO_GLYPH_INDEX) aa_flags |= GGO_GLYPH_INDEX; for (i = 0; i < sizeof(indices) / sizeof(indices[0]); i++) { index = indices[i]; - ret = GetGlyphOutlineW( hdc, index, ggo_flags, metrics, 0, NULL, &identity ); + ret = GetGlyphOutlineW( hdc, index, aa_flags, metrics, 0, NULL, &identity ); if (ret != GDI_ERROR) break; } @@ -1743,7 +1743,7 @@ static DWORD get_glyph_bitmap( HDC hdc, UINT index, UINT aa_flags, image->is_copy = TRUE; image->free = free_heap_bits; - ret = GetGlyphOutlineW( hdc, index, ggo_flags, metrics, size, image->ptr, &identity ); + ret = GetGlyphOutlineW( hdc, index, aa_flags, metrics, size, image->ptr, &identity ); if (ret == GDI_ERROR) { HeapFree( GetProcessHeap(), 0, image->ptr ); @@ -1764,7 +1764,7 @@ static RECT get_total_extents( HDC hdc, INT x, INT y, UINT flags, UINT aa_flags, { GLYPHMETRICS metrics; - if (get_glyph_bitmap( hdc, (UINT)str[i], aa_flags, &metrics, NULL )) continue; + if (get_glyph_bitmap( hdc, str[i], flags, aa_flags, &metrics, NULL )) continue; rect.left = x + metrics.gmptGlyphOrigin.x; rect.top = y - metrics.gmptGlyphOrigin.y; @@ -1949,7 +1949,7 @@ BOOL nulldrv_ExtTextOut( PHYSDEV dev, INT x, INT y, UINT flags, const RECT *rect GLYPHMETRICS metrics; struct gdi_image_bits image; - err = get_glyph_bitmap( dev->hdc, (UINT)str[i], GGO_BITMAP, &metrics, &image ); + err = get_glyph_bitmap( dev->hdc, str[i], flags, GGO_BITMAP, &metrics, &image ); if (err) continue; if (image.ptr) draw_glyph( dev->hdc, x, y, &metrics, &image, (flags & ETO_CLIPPED) ? rect : NULL );

1 0

Alexandre Julliard : winex11: Support caching glyphs by character code and not only by glyph index.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: b4ef8a0ee127b043a9954ecf388ba89ff0aa5b93 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b4ef8a0ee127b043a9954ecf3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 5 16:41:19 2012 +0100 winex11: Support caching glyphs by character code and not only by glyph index. --- dlls/winex11.drv/xrender.c | 72 ++++++++++++++++++++++--------------------- 1 files changed, 37 insertions(+), 35 deletions(-) diff --git a/dlls/winex11.drv/xrender.c b/dlls/winex11.drv/xrender.c index d1d1ccc..4da2eb4 100644 --- a/dlls/winex11.drv/xrender.c +++ b/dlls/winex11.drv/xrender.c @@ -132,6 +132,8 @@ typedef struct #define INITIAL_REALIZED_BUF_SIZE 128 +enum glyph_type { GLYPH_INDEX, GLYPH_WCHAR, GLYPH_NBTYPES }; + typedef enum { AA_None = 0, AA_Grey, AA_RGB, AA_BGR, AA_VRGB, AA_VBGR, AA_MAXVALUE } AA_Type; typedef struct @@ -146,7 +148,7 @@ typedef struct typedef struct { LFANDSIZE lfsz; - gsCacheEntryFormat * format[AA_MAXVALUE]; + gsCacheEntryFormat *format[GLYPH_NBTYPES][AA_MAXVALUE]; INT count; INT next; } gsCacheEntry; @@ -639,30 +641,33 @@ static int LookupEntry(LFANDSIZE *plfsz) static void FreeEntry(int entry) { - int format; + int type, format; - for(format = 0; format < AA_MAXVALUE; format++) { - gsCacheEntryFormat * formatEntry; + for (type = 0; type < GLYPH_NBTYPES; type++) + { + for(format = 0; format < AA_MAXVALUE; format++) { + gsCacheEntryFormat * formatEntry; - if( !glyphsetCache[entry].format[format] ) - continue; + if( !glyphsetCache[entry].format[type][format] ) + continue; - formatEntry = glyphsetCache[entry].format[format]; + formatEntry = glyphsetCache[entry].format[type][format]; - if(formatEntry->glyphset) { - pXRenderFreeGlyphSet(gdi_display, formatEntry->glyphset); - formatEntry->glyphset = 0; - } - if(formatEntry->nrealized) { - HeapFree(GetProcessHeap(), 0, formatEntry->realized); - formatEntry->realized = NULL; - HeapFree(GetProcessHeap(), 0, formatEntry->gis); - formatEntry->gis = NULL; - formatEntry->nrealized = 0; - } + if(formatEntry->glyphset) { + pXRenderFreeGlyphSet(gdi_display, formatEntry->glyphset); + formatEntry->glyphset = 0; + } + if(formatEntry->nrealized) { + HeapFree(GetProcessHeap(), 0, formatEntry->realized); + formatEntry->realized = NULL; + HeapFree(GetProcessHeap(), 0, formatEntry->gis); + formatEntry->gis = NULL; + formatEntry->nrealized = 0; + } - HeapFree(GetProcessHeap(), 0, formatEntry); - glyphsetCache[entry].format[format] = NULL; + HeapFree(GetProcessHeap(), 0, formatEntry); + glyphsetCache[entry].format[type][format] = NULL; + } } } @@ -736,7 +741,6 @@ static int AllocEntry(void) static int GetCacheEntry( LFANDSIZE *plfsz ) { int ret; - int format; gsCacheEntry *entry; if((ret = LookupEntry(plfsz)) != -1) return ret; @@ -744,10 +748,6 @@ static int GetCacheEntry( LFANDSIZE *plfsz ) ret = AllocEntry(); entry = glyphsetCache + ret; entry->lfsz = *plfsz; - for( format = 0; format < AA_MAXVALUE; format++ ) { - assert( !entry->format[format] ); - } - return ret; } @@ -1026,7 +1026,7 @@ static void xrenderdrv_SetDeviceClipping( PHYSDEV dev, HRGN rgn ) * * Helper to ExtTextOut. Must be called inside xrender_cs */ -static void UploadGlyph(struct xrender_physdev *physDev, int glyph) +static void UploadGlyph(struct xrender_physdev *physDev, UINT glyph, enum glyph_type type) { unsigned int buflen; char *buf; @@ -1035,18 +1035,19 @@ static void UploadGlyph(struct xrender_physdev *physDev, int glyph) XGlyphInfo gi; gsCacheEntry *entry = glyphsetCache + physDev->cache_index; gsCacheEntryFormat *formatEntry; - UINT ggo_format = GGO_GLYPH_INDEX | physDev->aa_flags; + UINT ggo_format = physDev->aa_flags; AA_Type format = aa_type_from_flags( physDev->aa_flags ); enum wxr_format wxr_format; static const char zero[4]; static const MAT2 identity = { {0,1},{0,0},{0,0},{0,1} }; + if (type == GLYPH_INDEX) ggo_format |= GGO_GLYPH_INDEX; buflen = GetGlyphOutlineW(physDev->dev.hdc, glyph, ggo_format, &gm, 0, NULL, &identity); if(buflen == GDI_ERROR) { if(format != AA_None) { format = AA_None; physDev->aa_flags = GGO_BITMAP; - ggo_format = GGO_GLYPH_INDEX | GGO_BITMAP; + ggo_format = (ggo_format & GGO_GLYPH_INDEX) | GGO_BITMAP; buflen = GetGlyphOutlineW(physDev->dev.hdc, glyph, ggo_format, &gm, 0, NULL, &identity); } if(buflen == GDI_ERROR) { @@ -1065,12 +1066,12 @@ static void UploadGlyph(struct xrender_physdev *physDev, int glyph) } /* If there is nothing for the current type, we create the entry. */ - if( !entry->format[format] ) { - entry->format[format] = HeapAlloc(GetProcessHeap(), + if( !entry->format[type][format] ) { + entry->format[type][format] = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(gsCacheEntryFormat)); } - formatEntry = entry->format[format]; + formatEntry = entry->format[type][format]; if(formatEntry->nrealized <= glyph) { formatEntry->nrealized = (glyph / 128 + 1) * 128; @@ -1315,6 +1316,7 @@ static BOOL xrenderdrv_ExtTextOut( PHYSDEV dev, INT x, INT y, UINT flags, int render_op = PictOpOver; XRenderColor col; RECT rect, bounds; + enum glyph_type type = (flags & ETO_GLYPH_INDEX) ? GLYPH_INDEX : GLYPH_WCHAR; get_xrender_color( physdev, GetTextColor( physdev->dev.hdc ), &col ); pict = get_xrender_picture( physdev, 0, (flags & ETO_CLIPPED) ? lprect : NULL ); @@ -1343,15 +1345,15 @@ static BOOL xrenderdrv_ExtTextOut( PHYSDEV dev, INT x, INT y, UINT flags, EnterCriticalSection(&xrender_cs); entry = glyphsetCache + physdev->cache_index; - formatEntry = entry->format[aa_type_from_flags( physdev->aa_flags )]; + formatEntry = entry->format[type][aa_type_from_flags( physdev->aa_flags )]; for(idx = 0; idx < count; idx++) { if( !formatEntry ) { - UploadGlyph(physdev, wstr[idx]); + UploadGlyph(physdev, wstr[idx], type); /* re-evaluate format entry since aa_flags may have changed */ - formatEntry = entry->format[aa_type_from_flags( physdev->aa_flags )]; + formatEntry = entry->format[type][aa_type_from_flags( physdev->aa_flags )]; } else if( wstr[idx] >= formatEntry->nrealized || formatEntry->realized[wstr[idx]] == FALSE) { - UploadGlyph(physdev, wstr[idx]); + UploadGlyph(physdev, wstr[idx], type); } } if (!formatEntry)

1 0

Detlef Riekenberg : shell32/tests: Add tests for DoEnvironmentSubstA/W.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: fc3868a07f18063ee2397aea7fbbd4af9e0e04c8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fc3868a07f18063ee2397aea7… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Tue Dec 4 11:38:02 2012 +0100 shell32/tests: Add tests for DoEnvironmentSubstA/W. --- dlls/shell32/tests/shellpath.c | 152 ++++++++++++++++++++++++++++++++++++++++ 1 files changed, 152 insertions(+), 0 deletions(-) diff --git a/dlls/shell32/tests/shellpath.c b/dlls/shell32/tests/shellpath.c index 1798ada..729a341 100644 --- a/dlls/shell32/tests/shellpath.c +++ b/dlls/shell32/tests/shellpath.c @@ -2495,6 +2495,157 @@ static void test_knownFolders(void) CoUninitialize(); } + +static void test_DoEnvironmentSubst(void) +{ + WCHAR expectedW[MAX_PATH]; + WCHAR bufferW[MAX_PATH]; + CHAR expectedA[MAX_PATH]; + CHAR bufferA[MAX_PATH]; + DWORD res; + DWORD res2; + DWORD len; + INT i; + static const WCHAR does_not_existW[] = {'%','D','O','E','S','_','N','O','T','_','E','X','I','S','T','%',0}; + static const CHAR does_not_existA[] = "%DOES_NOT_EXIST%"; + static const CHAR *names[] = { + /* interactive apps and services (works on all windows versions) */ + "%ALLUSERSPROFILE%", "%APPDATA%", "%LOCALAPPDATA%", + "%NUMBER_OF_PROCESSORS%", "%OS%", "%PROCESSOR_ARCHITECTURE%", + "%PROCESSOR_IDENTIFIER%", "%PROCESSOR_LEVEL%", "%PROCESSOR_REVISION%", + "%ProgramFiles%", "%SystemDrive%", + "%SystemRoot%", "%USERPROFILE%", "%windir%", + /* todo_wine: "%COMPUTERNAME%", "%ProgramData%", "%PUBLIC%", */ + + /* replace more than one var is allowed */ + "%HOMEDRIVE%%HOMEPATH%", + "%OS% %windir%"}; /* always the last entry in the table */ + + for (i = 0; i < (sizeof(names)/sizeof(LPSTR)); i++) + { + memset(bufferA, '#', MAX_PATH - 1); + bufferA[MAX_PATH - 1] = 0; + lstrcpyA(bufferA, names[i]); + MultiByteToWideChar(CP_ACP, 0, bufferA, MAX_PATH, bufferW, sizeof(bufferW)/sizeof(WCHAR)); + + res2 = ExpandEnvironmentStringsA(names[i], expectedA, MAX_PATH); + res = DoEnvironmentSubstA(bufferA, MAX_PATH); + + /* is the space for the terminating 0 included? */ + if (!i && HIWORD(res) && (LOWORD(res) == (lstrlenA(bufferA)))) + { + win_skip("DoEnvironmentSubstA/W are broken on NT 4\n"); + return; + } + ok(HIWORD(res) && (LOWORD(res) == res2), + "%d: got %d/%d (expected TRUE/%d)\n", i, HIWORD(res), LOWORD(res), res2); + ok(!lstrcmpA(bufferA, expectedA), + "%d: got %s (expected %s)\n", i, bufferA, expectedA); + + res2 = ExpandEnvironmentStringsW(bufferW, expectedW, MAX_PATH); + res = DoEnvironmentSubstW(bufferW, MAX_PATH); + ok(HIWORD(res) && (LOWORD(res) == res2), + "%d: got %d/%d (expected TRUE/%d)\n", i, HIWORD(res), LOWORD(res), res2); + ok(!lstrcmpW(bufferW, expectedW), + "%d: got %s (expected %s)\n", i, wine_dbgstr_w(bufferW), wine_dbgstr_w(expectedW)); + } + + i--; /* reuse data in the last table entry */ + len = LOWORD(res); /* needed length */ + + /* one character extra is fine */ + memset(bufferA, '#', MAX_PATH - 1); + bufferA[len + 2] = 0; + lstrcpyA(bufferA, names[i]); + MultiByteToWideChar(CP_ACP, 0, bufferA, MAX_PATH, bufferW, sizeof(bufferW)/sizeof(WCHAR)); + + res2 = ExpandEnvironmentStringsA(bufferA, expectedA, MAX_PATH); + res = DoEnvironmentSubstA(bufferA, len + 1); + ok(HIWORD(res) && (LOWORD(res) == res2), + "+1: got %d/%d (expected TRUE/%d)\n", HIWORD(res), LOWORD(res), res2); + ok(!lstrcmpA(bufferA, expectedA), + "+1: got %s (expected %s)\n", bufferA, expectedA); + + res2 = ExpandEnvironmentStringsW(bufferW, expectedW, MAX_PATH); + res = DoEnvironmentSubstW(bufferW, len + 1); + ok(HIWORD(res) && (LOWORD(res) == res2), + "+1: got %d/%d (expected TRUE/%d)\n", HIWORD(res), LOWORD(res), res2); + ok(!lstrcmpW(bufferW, expectedW), + "+1: got %s (expected %s)\n", wine_dbgstr_w(bufferW), wine_dbgstr_w(expectedW)); + + + /* minimal buffer length (result string and terminating 0) */ + memset(bufferA, '#', MAX_PATH - 1); + bufferA[len + 2] = 0; + lstrcpyA(bufferA, names[i]); + MultiByteToWideChar(CP_ACP, 0, bufferA, MAX_PATH, bufferW, sizeof(bufferW)/sizeof(WCHAR)); + + /* ANSI version failed without an extra byte, as documented on msdn */ + res = DoEnvironmentSubstA(bufferA, len); + ok(!HIWORD(res) && (LOWORD(res) == len), + " 0: got %d/%d (expected FALSE/%d)\n", HIWORD(res), LOWORD(res), len); + ok(!lstrcmpA(bufferA, names[i]), + " 0: got %s (expected %s)\n", bufferA, names[i]); + + /* DoEnvironmentSubstW works as expected */ + res2 = ExpandEnvironmentStringsW(bufferW, expectedW, MAX_PATH); + res = DoEnvironmentSubstW(bufferW, len); + ok(HIWORD(res) && (LOWORD(res) == res2), + " 0: got %d/%d (expected TRUE/%d)\n", HIWORD(res), LOWORD(res), res2); + ok(!lstrcmpW(bufferW, expectedW), + " 0: got %s (expected %s)\n", wine_dbgstr_w(bufferW), wine_dbgstr_w(expectedW)); + + + /* buffer to small */ + /* result: FALSE / provided buffer length / the buffer is untouched */ + memset(bufferA, '#', MAX_PATH - 1); + bufferA[len + 2] = 0; + lstrcpyA(bufferA, names[i]); + MultiByteToWideChar(CP_ACP, 0, bufferA, MAX_PATH, bufferW, sizeof(bufferW)/sizeof(WCHAR)); + + res = DoEnvironmentSubstA(bufferA, len - 1); + ok(!HIWORD(res) && (LOWORD(res) == (len - 1)), + "-1: got %d/%d (expected FALSE/%d)\n", HIWORD(res), LOWORD(res), len - 1); + ok(!lstrcmpA(bufferA, names[i]), + "-1: got %s (expected %s)\n", bufferA, names[i]); + + lstrcpyW(expectedW, bufferW); + res = DoEnvironmentSubstW(bufferW, len - 1); + ok(!HIWORD(res) && (LOWORD(res) == (len - 1)), + "-1: got %d/%d (expected FALSE/%d)\n", HIWORD(res), LOWORD(res), len - 1); + ok(!lstrcmpW(bufferW, expectedW), + "-1: got %s (expected %s)\n", wine_dbgstr_w(bufferW), wine_dbgstr_w(expectedW)); + + + /* unknown variable */ + /* result: TRUE / string length including terminating 0 / the buffer is untouched */ + memset(bufferA, '#', MAX_PATH - 1); + bufferA[MAX_PATH - 1] = 0; + lstrcpyA(bufferA, does_not_existA); + MultiByteToWideChar(CP_ACP, 0, bufferA, MAX_PATH, bufferW, sizeof(bufferW)/sizeof(WCHAR)); + + res2 = lstrlenA(does_not_existA) + 1; + res = DoEnvironmentSubstA(bufferA, MAX_PATH); + ok(HIWORD(res) && (LOWORD(res) == res2), + "%d: got %d/%d (expected TRUE/%d)\n", i, HIWORD(res), LOWORD(res), res2); + ok(!lstrcmpA(bufferA, does_not_existA), + "%d: got %s (expected %s)\n", i, bufferA, does_not_existA); + + res = DoEnvironmentSubstW(bufferW, MAX_PATH); + ok(HIWORD(res) && (LOWORD(res) == res2), + "%d: got %d/%d (expected TRUE/%d)\n", i, HIWORD(res), LOWORD(res), res2); + ok(!lstrcmpW(bufferW, does_not_existW), + "%d: got %s (expected %s)\n", i, wine_dbgstr_w(bufferW), wine_dbgstr_w(does_not_existW)); + + + if (0) + { + /* NULL crashes on windows */ + res = DoEnvironmentSubstA(NULL, MAX_PATH); + res = DoEnvironmentSubstW(NULL, MAX_PATH); + } +} + START_TEST(shellpath) { if (!init()) return; @@ -2522,5 +2673,6 @@ START_TEST(shellpath) test_NonExistentPath(); test_SHGetFolderPathEx(); test_knownFolders(); + test_DoEnvironmentSubst(); } }

1 0

Detlef Riekenberg : shell32: Implement DoEnvironmentSubstW.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: ff2f4941951b99acdb46fa67d64c8069431171c0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ff2f4941951b99acdb46fa67d… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Tue Dec 4 11:38:00 2012 +0100 shell32: Implement DoEnvironmentSubstW. --- dlls/shell32/shellord.c | 41 +++++++++++++++++++++++++++++++++++++---- 1 files changed, 37 insertions(+), 4 deletions(-) diff --git a/dlls/shell32/shellord.c b/dlls/shell32/shellord.c index c1fb382..b71d213 100644 --- a/dlls/shell32/shellord.c +++ b/dlls/shell32/shellord.c @@ -1626,14 +1626,47 @@ DWORD WINAPI DoEnvironmentSubstA(LPSTR pszString, UINT cchString) } /************************************************************************ - * DoEnvironmentSubstW [SHELL32.@] + * DoEnvironmentSubstW [SHELL32.@] * - * See DoEnvironmentSubstA. + * Replace all %KEYWORD% in the string with the value of the named + * environment variable. If the buffer is too small, the string is not modified. + * + * PARAMS + * pszString [I] '\0' terminated string with %keyword%. + * [O] '\0' terminated string with %keyword% substituted. + * cchString [I] size of str. + * + * RETURNS + * Success: The string in the buffer is updated + * HIWORD: TRUE + * LOWORD: characters used in the buffer, including space for the terminating 0 + * Failure: buffer too small. The string is not modified. + * HIWORD: FALSE + * LOWORD: provided size of the buffer in characters */ DWORD WINAPI DoEnvironmentSubstW(LPWSTR pszString, UINT cchString) { - FIXME("(%s, %d): stub\n", debugstr_w(pszString), cchString); - return MAKELONG(FALSE,cchString); + LPWSTR dst; + BOOL res = FALSE; + DWORD len = cchString; + + TRACE("(%s, %d)\n", debugstr_w(pszString), cchString); + + if ((cchString < MAXLONG) && (dst = HeapAlloc(GetProcessHeap(), 0, cchString * sizeof(WCHAR)))) + { + len = ExpandEnvironmentStringsW(pszString, dst, cchString); + /* len includes the terminating 0 */ + if (len && len <= cchString) + { + res = TRUE; + memcpy(pszString, dst, len * sizeof(WCHAR)); + } + else + len = cchString; + + HeapFree(GetProcessHeap(), 0, dst); + } + return MAKELONG(len, res); } /************************************************************************

1 0

Alexandre Julliard : user32: Ignore the PM_NOYIELD flag on the client side.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: 1cf2c9c179ddf756a89d0a746414542200fdd47a URL: http://source.winehq.org/git/wine.git/?a=commit;h=1cf2c9c179ddf756a89d0a746… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 5 12:48:30 2012 +0100 user32: Ignore the PM_NOYIELD flag on the client side. --- dlls/user32/message.c | 9 ++------- 1 files changed, 2 insertions(+), 7 deletions(-) diff --git a/dlls/user32/message.c b/dlls/user32/message.c index 8ca259b..3f0312a 100644 --- a/dlls/user32/message.c +++ b/dlls/user32/message.c @@ -3685,13 +3685,8 @@ BOOL WINAPI DECLSPEC_HOTPATCH PeekMessageW( MSG *msg_out, HWND hwnd, UINT first, { DWORD ret; - flush_window_surfaces( !(flags & PM_NOYIELD) ); - - if (flags & PM_NOYIELD) - ret = USER_Driver->pMsgWaitForMultipleObjectsEx( 0, NULL, 0, QS_ALLINPUT, 0 ); - else - ret = wow_handlers.wait_message( 0, NULL, 0, QS_ALLINPUT, 0 ); - + flush_window_surfaces( TRUE ); + ret = wow_handlers.wait_message( 0, NULL, 0, QS_ALLINPUT, 0 ); /* if we received driver events, check again for a pending message */ if (ret == WAIT_TIMEOUT || !peek_message( &msg, hwnd, first, last, flags, 0 )) return FALSE; }

1 0

Stefan Dösinger : wined3d: Correctly count used stages in the atifs pipeline.
by Alexandre Julliard Dec. 5, 2012

Dec. 5, 2012

Module: wine Branch: master Commit: 33802da4f3fcf53960d2d5c628376eb1dda22805 URL: http://source.winehq.org/git/wine.git/?a=commit;h=33802da4f3fcf53960d2d5c62… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Wed Dec 5 09:01:34 2012 +0100 wined3d: Correctly count used stages in the atifs pipeline. --- dlls/wined3d/ati_fragment_shader.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/ati_fragment_shader.c b/dlls/wined3d/ati_fragment_shader.c index fba55ea..1b9b969 100644 --- a/dlls/wined3d/ati_fragment_shader.c +++ b/dlls/wined3d/ati_fragment_shader.c @@ -857,7 +857,7 @@ static void set_tex_op_atifs(struct wined3d_context *context, const struct wined { if (settings.op[i].cop == WINED3D_TOP_DISABLE) break; - new_desc->num_textures_used = i; + new_desc->num_textures_used = i + 1; } memcpy(&new_desc->parent.settings, &settings, sizeof(settings));

1 0