wine-commits July 2010

wine-commits@winehq.org

2 participants
892 discussions

Alexandre Julliard : gdi32: Fix text positioning in mirrored DCs.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: 1da826a1498e68169a2f4f8e80470cb451e8f8a0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1da826a1498e68169a2f4f8e8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jul 29 13:37:20 2010 +0200 gdi32: Fix text positioning in mirrored DCs. --- dlls/gdi32/font.c | 16 ++++++++++++---- 1 files changed, 12 insertions(+), 4 deletions(-) diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 1b36374..29281f9 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -1715,6 +1715,7 @@ BOOL WINAPI ExtTextOutW( HDC hdc, INT x, INT y, UINT flags, LPWSTR reordered_str = (LPWSTR)str; WORD *glyphs = NULL; UINT align = GetTextAlign( hdc ); + DWORD layout = GetLayout( hdc ); POINT pt; TEXTMETRICW tm; LOGFONTW lf; @@ -1755,17 +1756,23 @@ BOOL WINAPI ExtTextOutW( HDC hdc, INT x, INT y, UINT flags, if (!lprect) flags &= ~ETO_CLIPPED; - + + if (flags & ETO_RTLREADING) align |= TA_RTLREADING; + if (layout & LAYOUT_RTL) + { + if ((align & TA_CENTER) != TA_CENTER) align ^= TA_RIGHT; + align ^= TA_RTLREADING; + } + if( !(flags & (ETO_GLYPH_INDEX | ETO_IGNORELANGUAGE)) && count > 0 ) { INT cGlyphs; reordered_str = HeapAlloc(GetProcessHeap(), 0, count*sizeof(WCHAR)); BIDI_Reorder( hdc, str, count, GCP_REORDER, - ((flags&ETO_RTLREADING)!=0 || (GetTextAlign(hdc)&TA_RTLREADING)!=0)? - WINE_GCPW_FORCE_RTL:WINE_GCPW_FORCE_LTR, + (align & TA_RTLREADING) ? WINE_GCPW_FORCE_RTL : WINE_GCPW_FORCE_LTR, reordered_str, count, NULL, &glyphs, &cGlyphs); - + flags |= ETO_IGNORELANGUAGE; if (glyphs) { @@ -1901,6 +1908,7 @@ BOOL WINAPI ExtTextOutW( HDC hdc, INT x, INT y, UINT flags, LPtoDP(hdc, desired, 2); desired[1].x -= desired[0].x; desired[1].y -= desired[0].y; + if (layout & LAYOUT_RTL) desired[1].x = -desired[1].x; deltas[i].x = desired[1].x - width.x; deltas[i].y = desired[1].y - width.y;

1 0

Alexandre Julliard : gdi32: Mirror the clip region when the DC is mirrored, with tests.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: 7d776a866ea6a0a290f3e60fe7592d29245c31bb URL: http://source.winehq.org/git/wine.git/?a=commit;h=7d776a866ea6a0a290f3e60fe… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jul 29 13:36:43 2010 +0200 gdi32: Mirror the clip region when the DC is mirrored, with tests. --- dlls/gdi32/clipping.c | 79 ++++++++++++++++++++++++++++++++------------ dlls/gdi32/gdi_private.h | 1 + dlls/gdi32/region.c | 57 +++++++++++++++++++++++++++++++ dlls/gdi32/tests/mapping.c | 55 ++++++++++++++++++++++++++++++ 4 files changed, 171 insertions(+), 21 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7d776a866ea6a0a290f3e…

1 0

Nikolay Sivov : shlwapi: Implement SKGetValueW over SHGetShellKey.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: 31da90fd9d4437f76271066c8cd0ddbf055997e8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=31da90fd9d4437f76271066c8… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Jul 29 10:03:29 2010 +0400 shlwapi: Implement SKGetValueW over SHGetShellKey. --- dlls/shlwapi/ordinal.c | 18 ++++++++++++++---- 1 files changed, 14 insertions(+), 4 deletions(-) diff --git a/dlls/shlwapi/ordinal.c b/dlls/shlwapi/ordinal.c index 3e47af6..3edb116 100644 --- a/dlls/shlwapi/ordinal.c +++ b/dlls/shlwapi/ordinal.c @@ -4505,12 +4505,22 @@ HRESULT WINAPI IUnknown_OnFocusChangeIS(LPUNKNOWN lpUnknown, LPUNKNOWN pFocusObj } /*********************************************************************** - * SHGetValueW (SHLWAPI.@) + * SKGetValueW (SHLWAPI.516) */ -HRESULT WINAPI SKGetValueW(DWORD a, LPWSTR b, LPWSTR c, DWORD d, DWORD e, DWORD f) +HRESULT WINAPI SKGetValueW(DWORD flags, LPCWSTR subkey, LPCWSTR value, DWORD *type, + void *data, DWORD *count) { - FIXME("(%x, %s, %s, %x, %x, %x): stub\n", a, debugstr_w(b), debugstr_w(c), d, e, f); - return E_FAIL; + DWORD ret; + HKEY hkey; + + TRACE("(0x%x, %s, %s, %p, %p, %p)\n", flags, debugstr_w(subkey), + debugstr_w(value), type, data, count); + + hkey = SHGetShellKey(flags, subkey, FALSE); + ret = SHQueryValueExW(hkey, value, NULL, type, data, count); + RegCloseKey(hkey); + + return HRESULT_FROM_WIN32(ret); } typedef HRESULT (WINAPI *DllGetVersion_func)(DLLVERSIONINFO *);

1 0

Nikolay Sivov : shlwapi: Partially implement SHGetShellKey.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: e7395e09e175ac635fa655ca9abb7c66bb549ab0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e7395e09e175ac635fa655ca9… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Jul 29 10:02:02 2010 +0400 shlwapi: Partially implement SHGetShellKey. --- dlls/shlwapi/ordinal.c | 36 ++++++++++++++++++++++-- dlls/shlwapi/tests/ordinal.c | 61 ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 94 insertions(+), 3 deletions(-) diff --git a/dlls/shlwapi/ordinal.c b/dlls/shlwapi/ordinal.c index 2bf0e79..3e47af6 100644 --- a/dlls/shlwapi/ordinal.c +++ b/dlls/shlwapi/ordinal.c @@ -4401,12 +4401,42 @@ BOOL WINAPI SHSkipJunction(IBindCtx *pbc, const CLSID *pclsid) } /*********************************************************************** - * SHGetShellKey (SHLWAPI.@) + * SHGetShellKey (SHLWAPI.491) */ HKEY WINAPI SHGetShellKey(DWORD flags, LPCWSTR sub_key, BOOL create) { - FIXME("(0x%08x, %s, %d): stub\n", flags, debugstr_w(sub_key), create); - return (HKEY)0x50; + enum _shellkey_flags { + SHKEY_Explorer = 0x00, + SHKEY_Root_HKCU = 0x01 + }; + + static const WCHAR explorerW[] = {'S','o','f','t','w','a','r','e','\\', + 'M','i','c','r','o','s','o','f','t','\\','W','i','n','d','o','w','s','\\', + 'C','u','r','r','e','n','t','V','e','r','s','i','o','n','\\', + 'E','x','p','l','o','r','e','r',0}; + HKEY hroot, hkey = NULL; + + TRACE("(0x%08x, %s, %d)\n", flags, debugstr_w(sub_key), create); + + switch (flags) + { + case SHKEY_Explorer | SHKEY_Root_HKCU: + RegOpenKeyExW(HKEY_CURRENT_USER, explorerW, 0, MAXIMUM_ALLOWED, &hroot); + break; + case 0: + return NULL; + default: + FIXME("unsupported flags (0x%08x)\n", flags); + return (HKEY)0xdeadbeef; + } + + if (create) + RegCreateKeyExW(hroot, sub_key, 0, NULL, 0, MAXIMUM_ALLOWED, NULL, &hkey, NULL); + else + RegOpenKeyExW(hroot, sub_key, 0, MAXIMUM_ALLOWED, &hkey); + + RegCloseKey(hroot); + return hkey; } /*********************************************************************** diff --git a/dlls/shlwapi/tests/ordinal.c b/dlls/shlwapi/tests/ordinal.c index c3c59c4..0089a43 100644 --- a/dlls/shlwapi/tests/ordinal.c +++ b/dlls/shlwapi/tests/ordinal.c @@ -36,6 +36,7 @@ /* Function ptrs for ordinal calls */ static HMODULE hShlwapi; static BOOL is_win2k_and_lower; +static BOOL is_win9x; static int (WINAPI *pSHSearchMapInt)(const int*,const int*,int,int); static HRESULT (WINAPI *pGetAcceptLanguagesA)(LPSTR,LPDWORD); @@ -60,6 +61,7 @@ static HWND (WINAPI *pSHCreateWorkerWindowA)(LONG, HWND, DWORD, DWORD, HMENU, static HRESULT (WINAPI *pSHIShellFolder_EnumObjects)(LPSHELLFOLDER, HWND, SHCONTF, IEnumIDList**); static DWORD (WINAPI *pSHGetIniStringW)(LPCWSTR, LPCWSTR, LPWSTR, DWORD, LPCWSTR); static BOOL (WINAPI *pSHSetIniStringW)(LPCWSTR, LPCWSTR, LPCWSTR, LPCWSTR); +static HKEY (WINAPI *pSHGetShellKey)(DWORD, LPWSTR, BOOL); static HMODULE hmlang; static HRESULT (WINAPI *pLcidToRfc1766A)(LCID, LPSTR, INT); @@ -2675,6 +2677,62 @@ static void test_SHSetIniString(void) DeleteFileW(TestIniW); } +enum _shellkey_flags { + SHKEY_Explorer = 0x00, + SHKEY_Root_HKCU = 0x01 +}; + +static void test_SHGetShellKey(void) +{ + void *pPathBuildRootW = GetProcAddress(hShlwapi, "PathBuildRootW"); + HKEY hkey, hkey2; + DWORD ret; + + if (!pSHGetShellKey) + { + win_skip("SHGetShellKey(ordinal 491) isn't available\n"); + return; + } + + /* some win2k */ + if (pPathBuildRootW && pPathBuildRootW == pSHGetShellKey) + { + win_skip("SHGetShellKey(ordinal 491) used for PathBuildRootW\n"); + return; + } + + if (is_win9x || is_win2k_and_lower) + { + win_skip("Ordinal 491 used for another call, skipping SHGetShellKey tests\n"); + return; + } + + /* marking broken cause latest Vista+ versions fail here */ + SetLastError(0xdeadbeef); + hkey = pSHGetShellKey(SHKEY_Explorer, NULL, FALSE); + ok(hkey == NULL || broken(hkey != NULL), "got %p\n", hkey); + if (hkey) + { + hkey2 = 0; + ret = RegOpenKeyExA(hkey, "Shell Folders", 0, KEY_READ, &hkey2); + ok(ret == ERROR_SUCCESS, "got %d\n", ret); + ok(hkey2 != NULL, "got %p\n", hkey2); + RegCloseKey( hkey2 ); + RegCloseKey( hkey ); + } + + hkey = pSHGetShellKey(SHKEY_Explorer | SHKEY_Root_HKCU, NULL, FALSE); + ok(hkey != NULL, "got %p\n", hkey); + + hkey2 = 0; + ret = RegOpenKeyExA(hkey, "Shell Folders", 0, KEY_READ, &hkey2); + ok(ret == ERROR_SUCCESS, "got %d\n", ret); + ok(hkey2 != NULL, "got %p\n", hkey2); + RegCloseKey( hkey2 ); + + RegCloseKey( hkey ); +} + static void init_pointers(void) { #define MAKEFUNC(f, ord) (p##f = (void*)GetProcAddress(hShlwapi, (LPSTR)(ord))) @@ -2698,6 +2756,7 @@ static void init_pointers(void) MAKEFUNC(SHIShellFolder_EnumObjects, 404); MAKEFUNC(SHGetObjectCompatFlags, 476); MAKEFUNC(IUnknown_QueryServiceExec, 484); + MAKEFUNC(SHGetShellKey, 491); MAKEFUNC(SHPropertyBag_ReadLONG, 496); MAKEFUNC(IUnknown_ProfferService, 514); #undef MAKEFUNC @@ -2707,6 +2766,7 @@ START_TEST(ordinal) { hShlwapi = GetModuleHandleA("shlwapi.dll"); is_win2k_and_lower = GetProcAddress(hShlwapi, "StrChrNW") == 0; + is_win9x = GetProcAddress(hShlwapi, (LPSTR)99) == 0; /* StrCpyNXA */ init_pointers(); @@ -2734,6 +2794,7 @@ START_TEST(ordinal) test_SHIShellFolder_EnumObjects(); test_SHGetIniString(); test_SHSetIniString(); + test_SHGetShellKey(); FreeLibrary(hshell32); FreeLibrary(hmlang);

1 0

Nikolay Sivov : shlwapi/tests: Fix some test failures on Win2000.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: b684c60a10c058d5e9d13dfc57d480978b54744d URL: http://source.winehq.org/git/wine.git/?a=commit;h=b684c60a10c058d5e9d13dfc5… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Jul 29 09:59:58 2010 +0400 shlwapi/tests: Fix some test failures on Win2000. --- dlls/shlwapi/tests/ordinal.c | 18 ++++++++++++++++-- 1 files changed, 16 insertions(+), 2 deletions(-) diff --git a/dlls/shlwapi/tests/ordinal.c b/dlls/shlwapi/tests/ordinal.c index d60effd..c3c59c4 100644 --- a/dlls/shlwapi/tests/ordinal.c +++ b/dlls/shlwapi/tests/ordinal.c @@ -590,6 +590,7 @@ static void test_GetShellSecurityDescriptor(void) }; SECURITY_DESCRIPTOR* psd; SECURITY_DESCRIPTOR* (WINAPI*pGetShellSecurityDescriptor)(PSHELL_USER_PERMISSION*,int); + void *pChrCmpIW = GetProcAddress(hShlwapi, "ChrCmpIW"); pGetShellSecurityDescriptor=(void*)GetProcAddress(hShlwapi,(char*)475); @@ -599,12 +600,18 @@ static void test_GetShellSecurityDescriptor(void) return; } + if(pChrCmpIW && pChrCmpIW == pGetShellSecurityDescriptor) /* win2k */ + { + win_skip("Skipping for GetShellSecurityDescriptor, same ordinal used for ChrCmpIW\n"); + return; + } + psd = pGetShellSecurityDescriptor(NULL, 2); ok(psd==NULL || broken(psd==INVALID_HANDLE_VALUE), /* IE5 */ "GetShellSecurityDescriptor should fail\n"); psd = pGetShellSecurityDescriptor(rgsup, 0); - ok(psd==NULL, "GetShellSecurityDescriptor should fail\n"); + ok(psd==NULL, "GetShellSecurityDescriptor should fail, got %p\n", psd); SetLastError(0xdeadbeef); psd = pGetShellSecurityDescriptor(rgsup, 2); @@ -614,7 +621,7 @@ static void test_GetShellSecurityDescriptor(void) win_skip("GetShellSecurityDescriptor is not implemented\n"); return; } - if (psd==INVALID_HANDLE_VALUE) + if (psd == INVALID_HANDLE_VALUE) { win_skip("GetShellSecurityDescriptor is broken on IE5\n"); return; @@ -1910,6 +1917,7 @@ static void test_SHGetObjectCompatFlags(void) }; static const char compat_path[] = "Software\\Microsoft\\Windows\\CurrentVersion\\ShellCompatibility\\Objects"; + void *pColorAdjustLuma = GetProcAddress(hShlwapi, "ColorAdjustLuma"); CHAR keyA[39]; /* {CLSID} */ HKEY root; DWORD ret; @@ -1921,6 +1929,12 @@ static void test_SHGetObjectCompatFlags(void) return; } + if (pColorAdjustLuma && pColorAdjustLuma == pSHGetObjectCompatFlags) /* win2k */ + { + win_skip("Skipping SHGetObjectCompatFlags, same ordinal used for ColorAdjustLuma\n"); + return; + } + /* null args */ ret = pSHGetObjectCompatFlags(NULL, NULL); ok(ret == 0, "got %d\n", ret);

1 0

Dylan Smith : richedit: Removed unused run flags values.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: 4c174fc84507ba1641fff263d4feece61ffb2bce URL: http://source.winehq.org/git/wine.git/?a=commit;h=4c174fc84507ba1641fff263d… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Wed Jul 28 21:18:03 2010 -0400 richedit: Removed unused run flags values. The only place MERF_SKIPPED was in debug code, and MERF_CALCBYWRAP was only used to clear this unused flag value, so I got rid of both of them. --- dlls/riched20/editstr.h | 4 ---- dlls/riched20/paint.c | 5 +---- dlls/riched20/wrap.c | 3 --- 3 files changed, 1 insertions(+), 11 deletions(-) diff --git a/dlls/riched20/editstr.h b/dlls/riched20/editstr.h index c9c55da..5b5afae 100644 --- a/dlls/riched20/editstr.h +++ b/dlls/riched20/editstr.h @@ -118,10 +118,6 @@ typedef enum { #define MERF_ENDWHITE 0x004000 /* run is completely made of whitespaces */ #define MERF_WHITESPACE 0x008000 -/* run is a last (dummy) run in the paragraph */ -#define MERF_SKIPPED 0x010000 -/* flags that are calculated during text wrapping */ -#define MERF_CALCBYWRAP 0x0F0000 /* the "end of paragraph" run, contains 1 character */ #define MERF_ENDPARA 0x100000 /* forcing the "end of row" run, contains 1 character */ diff --git a/dlls/riched20/paint.c b/dlls/riched20/paint.c index 5e52e51..2be034b 100644 --- a/dlls/riched20/paint.c +++ b/dlls/riched20/paint.c @@ -959,10 +959,7 @@ static void ME_DrawParagraph(ME_Context *c, ME_DisplayItem *paragraph) rc.top = c->pt.y + para->pt.y + run->pt.y; rc.bottom = rc.bottom + height; TRACE("rc = (%d, %d, %d, %d)\n", rc.left, rc.top, rc.right, rc.bottom); - if (run->nFlags & MERF_SKIPPED) - DrawFocusRect(c->hDC, &rc); - else - FrameRect(c->hDC, &rc, GetSysColorBrush(COLOR_GRAYTEXT)); + FrameRect(c->hDC, &rc, GetSysColorBrush(COLOR_GRAYTEXT)); } if (visible) ME_DrawRun(c, c->pt.x + run->pt.x, diff --git a/dlls/riched20/wrap.c b/dlls/riched20/wrap.c index d2ebd94..a7c7fd3 100644 --- a/dlls/riched20/wrap.c +++ b/dlls/riched20/wrap.c @@ -349,7 +349,6 @@ static ME_DisplayItem *ME_WrapHandleRun(ME_WrapContext *wc, ME_DisplayItem *p) return p->next; if (run->nFlags & MERF_WHITESPACE) { - p->member.run.nFlags |= MERF_SKIPPED; wc->pt.x += run->nWidth; /* skip runs consisting of only whitespaces */ return p->next; @@ -362,7 +361,6 @@ static ME_DisplayItem *ME_WrapHandleRun(ME_WrapContext *wc, ME_DisplayItem *p) if (black) { wc->bOverflown = FALSE; pp = ME_SplitRun(wc, p, black); - p->member.run.nFlags |= MERF_SKIPPED; ME_InsertRowStart(wc, pp); return pp; } @@ -588,7 +586,6 @@ static void ME_PrepareParagraphForWrapping(ME_Context *c, ME_DisplayItem *tp) { else break; } - p->member.run.nFlags &= ~MERF_CALCBYWRAP; break; default: break;

1 0

Dylan Smith : richedit: Removed redundant variable ptLastSplittableRun.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: 15251ad84b45c7bd51fbb94d9d73a6409b236d7f URL: http://source.winehq.org/git/wine.git/?a=commit;h=15251ad84b45c7bd51fbb94d9… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Wed Jul 28 21:17:53 2010 -0400 richedit: Removed redundant variable ptLastSplittableRun. --- dlls/riched20/editstr.h | 1 - dlls/riched20/wrap.c | 5 ++--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/dlls/riched20/editstr.h b/dlls/riched20/editstr.h index 9b8d645..c9c55da 100644 --- a/dlls/riched20/editstr.h +++ b/dlls/riched20/editstr.h @@ -414,7 +414,6 @@ typedef struct tagME_WrapContext ME_DisplayItem *pRowStart; ME_DisplayItem *pLastSplittableRun; - POINT ptLastSplittableRun; } ME_WrapContext; #endif diff --git a/dlls/riched20/wrap.c b/dlls/riched20/wrap.c index 8d75b7f..d2ebd94 100644 --- a/dlls/riched20/wrap.c +++ b/dlls/riched20/wrap.c @@ -271,7 +271,7 @@ static ME_DisplayItem *ME_SplitByBacktracking(ME_WrapContext *wc, ME_DisplayItem { if (wc->pLastSplittableRun->member.run.nFlags & (MERF_GRAPHICS|MERF_TAB)) { - wc->pt = wc->ptLastSplittableRun; + wc->pt = wc->pLastSplittableRun->member.run.pt; return wc->pLastSplittableRun; } else if (wc->pLastSplittableRun->member.run.nFlags & MERF_SPLITTABLE) @@ -299,7 +299,7 @@ static ME_DisplayItem *ME_SplitByBacktracking(ME_WrapContext *wc, ME_DisplayItem else { /* restart from the first run beginning with spaces */ - wc->pt = wc->ptLastSplittableRun; + wc->pt = wc->pLastSplittableRun->member.run.pt; return wc->pLastSplittableRun; } } @@ -451,7 +451,6 @@ static ME_DisplayItem *ME_WrapHandleRun(ME_WrapContext *wc, ME_DisplayItem *p) || ((run->nFlags & (MERF_GRAPHICS|MERF_TAB)) && (p != wc->pRowStart))) { wc->pLastSplittableRun = p; - wc->ptLastSplittableRun = wc->pt; } wc->pt.x += run->nWidth; return p->next;

1 0

Dylan Smith : richedit: Invalidate without calling UpdateWindow in many cases.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: dfa80b35b04692624378876a8fe0fe0fee3b7671 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dfa80b35b04692624378876a8… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Wed Jul 28 21:17:40 2010 -0400 richedit: Invalidate without calling UpdateWindow in many cases. --- dlls/riched20/editor.c | 32 ++++++++++++++++---------------- dlls/riched20/editor.h | 2 +- dlls/riched20/paint.c | 5 +++-- dlls/riched20/richole.c | 2 +- dlls/riched20/txtsrv.c | 2 +- dlls/riched20/undo.c | 4 ++-- 6 files changed, 24 insertions(+), 23 deletions(-) diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 959585a..0ed1959 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -1583,7 +1583,7 @@ static LRESULT ME_StreamIn(ME_TextEditor *editor, DWORD format, EDITSTREAM *stre ME_ReleaseStyle(style); editor->nEventMask = nEventMask; - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); if (!(format & SFF_SELECTION)) { ME_ClearTempStyle(editor); } @@ -2137,7 +2137,7 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) return TRUE; ME_MoveCursorFromTableRowStartParagraph(editor); ME_UpdateSelectionLinkAttribute(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); ME_SendRequestResize(editor, FALSE); return TRUE; case VK_RETURN: @@ -2193,7 +2193,7 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) editor->pCursors[1] = editor->pCursors[0]; ME_CommitUndo(editor); ME_CheckTablesForCorruption(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return TRUE; } else if (para == editor->pCursors[1].pPara && @@ -2218,7 +2218,7 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) para->member.para.next_para->member.para.nFlags |= MEPF_ROWSTART; ME_CommitCoalescingUndo(editor); ME_CheckTablesForCorruption(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return TRUE; } } else { /* v1.0 - 3.0 */ @@ -2235,7 +2235,7 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) editor->pCursors[0].nOffset = 0; editor->pCursors[1] = editor->pCursors[0]; ME_CommitCoalescingUndo(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return TRUE; } } else { @@ -2262,7 +2262,7 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) editor->pCursors[1] = editor->pCursors[0]; } ME_CommitCoalescingUndo(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return TRUE; } } @@ -2280,7 +2280,7 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) SetCursor(NULL); ME_UpdateSelectionLinkAttribute(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); } return TRUE; } @@ -2319,7 +2319,7 @@ ME_KeyDown(ME_TextEditor *editor, WORD nKey) { ME_InternalDeleteText(editor, selStart, nChars, FALSE); ME_CommitUndo(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, TRUE); } return result; } @@ -2449,7 +2449,7 @@ static LRESULT ME_Char(ME_TextEditor *editor, WPARAM charCode, } ME_UpdateSelectionLinkAttribute(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); } return 0; } @@ -3299,7 +3299,7 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, editor->nModifyStep = oldModify; ME_EmptyUndoStack(editor); } - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return len; } case EM_SETBKGNDCOLOR: @@ -3460,7 +3460,7 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, int nStartCursor = ME_GetSelectionOfs(editor, &from, &to); ME_InternalDeleteText(editor, &editor->pCursors[nStartCursor], to-from, FALSE); ME_CommitUndo(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, TRUE); return 0; } case EM_REPLACESEL: @@ -3488,7 +3488,7 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, ME_UpdateSelectionLinkAttribute(editor); if (!wParam) ME_EmptyUndoStack(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return len; } case EM_SCROLLCARET: @@ -3566,7 +3566,7 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, editor->nModifyStep = 0; ME_CommitUndo(editor); ME_EmptyUndoStack(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return 1; } case EM_CANPASTE: @@ -3592,7 +3592,7 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, { ME_InternalDeleteText(editor, selStart, nChars, FALSE); ME_CommitUndo(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, TRUE); } return 0; } @@ -4246,7 +4246,7 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, editor->imeStartIndex=ME_GetCursorOfs(&editor->pCursors[0]); ME_DeleteSelection(editor); ME_CommitUndo(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return 0; } case WM_IME_COMPOSITION: @@ -4278,7 +4278,7 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, } ME_ReleaseStyle(style); ME_CommitUndo(editor); - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return 0; } case WM_IME_ENDCOMPOSITION: diff --git a/dlls/riched20/editor.h b/dlls/riched20/editor.h index 9739eea..10a1f2c 100644 --- a/dlls/riched20/editor.h +++ b/dlls/riched20/editor.h @@ -210,7 +210,7 @@ void ME_SetDefaultParaFormat(PARAFORMAT2 *pFmt); void ME_PaintContent(ME_TextEditor *editor, HDC hDC, BOOL bOnlyNew, const RECT *rcUpdate); void ME_Repaint(ME_TextEditor *editor); void ME_RewrapRepaint(ME_TextEditor *editor); -void ME_UpdateRepaint(ME_TextEditor *editor); +void ME_UpdateRepaint(ME_TextEditor *editor, BOOL update_now); void ME_EnsureVisible(ME_TextEditor *editor, ME_Cursor *pCursor); void ME_InvalidateSelection(ME_TextEditor *editor); BOOL ME_SetZoom(ME_TextEditor *editor, int numerator, int denominator); diff --git a/dlls/riched20/paint.c b/dlls/riched20/paint.c index 0a32601..5e52e51 100644 --- a/dlls/riched20/paint.c +++ b/dlls/riched20/paint.c @@ -132,7 +132,7 @@ void ME_Repaint(ME_TextEditor *editor) ITextHost_TxViewChange(editor->texthost, TRUE); } -void ME_UpdateRepaint(ME_TextEditor *editor) +void ME_UpdateRepaint(ME_TextEditor *editor, BOOL update_now) { /* Should be called whenever the contents of the control have changed */ BOOL wrappedParagraphs; @@ -144,6 +144,8 @@ void ME_UpdateRepaint(ME_TextEditor *editor) /* Ensure that the cursor is visible */ ME_EnsureVisible(editor, &editor->pCursors[0]); + ITextHost_TxViewChange(editor->texthost, update_now); + ME_SendSelChange(editor); /* send EN_CHANGE if the event mask asks for it */ @@ -153,7 +155,6 @@ void ME_UpdateRepaint(ME_TextEditor *editor) ME_SendOldNotify(editor, EN_CHANGE); editor->nEventMask |= ENM_CHANGE; } - ME_Repaint(editor); } void diff --git a/dlls/riched20/richole.c b/dlls/riched20/richole.c index 5b6b2b6..0f7db26 100644 --- a/dlls/riched20/richole.c +++ b/dlls/riched20/richole.c @@ -391,7 +391,7 @@ IRichEditOle_fnInsertObject(IRichEditOle *me, REOBJECT *reo) ME_InsertOLEFromCursor(This->editor, reo, 0); ME_CommitUndo(This->editor); - ME_UpdateRepaint(This->editor); + ME_UpdateRepaint(This->editor, FALSE); return S_OK; } diff --git a/dlls/riched20/txtsrv.c b/dlls/riched20/txtsrv.c index 7ab2af4..b4625ad 100644 --- a/dlls/riched20/txtsrv.c +++ b/dlls/riched20/txtsrv.c @@ -325,7 +325,7 @@ HRESULT WINAPI fnTextSrv_TxSetText(ITextServices *iface, This->editor->nModifyStep = 0; OleFlushClipboard(); ME_EmptyUndoStack(This->editor); - ME_UpdateRepaint(This->editor); + ME_UpdateRepaint(This->editor, FALSE); return S_OK; } diff --git a/dlls/riched20/undo.c b/dlls/riched20/undo.c index 8dab8fe..be7f0e1 100644 --- a/dlls/riched20/undo.c +++ b/dlls/riched20/undo.c @@ -402,7 +402,7 @@ BOOL ME_Undo(ME_TextEditor *editor) { ME_CheckTablesForCorruption(editor); editor->nUndoStackSize--; editor->nUndoMode = nMode; - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return TRUE; } @@ -438,6 +438,6 @@ BOOL ME_Redo(ME_TextEditor *editor) { ME_AddUndoItem(editor, diUndoEndTransaction, NULL); ME_CheckTablesForCorruption(editor); editor->nUndoMode = nMode; - ME_UpdateRepaint(editor); + ME_UpdateRepaint(editor, FALSE); return TRUE; }

1 0

Dylan Smith : richedit: Remove redundant commit and repaint in ME_StreamInText.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: 1964e603fd7d8faa787438dae8781b30402a5a65 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1964e603fd7d8faa787438dae… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Wed Jul 28 21:17:29 2010 -0400 richedit: Remove redundant commit and repaint in ME_StreamInText. ME_StreamInText is only used in one function, ME_StreamIn, which always calls ME_CommitUndo and ME_UpdateRepaint after calling this function. --- dlls/riched20/editor.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 7e3dfda..959585a 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -317,8 +317,6 @@ static LRESULT ME_StreamInText(ME_TextEditor *editor, DWORD dwFormat, ME_InStrea break; stream->dwSize = 0; } while(1); - ME_CommitUndo(editor); - ME_UpdateRepaint(editor); return 0; }

1 0

Dylan Smith : richedit: EN_UPDATE notification is sent on WM_PAINT.
by Alexandre Julliard 29 Jul '10

29 Jul '10

Module: wine Branch: master Commit: 328a3f1d8c7e254ab003bd22ad52251d8591ed8e URL: http://source.winehq.org/git/wine.git/?a=commit;h=328a3f1d8c7e254ab003bd22a… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Wed Jul 28 21:17:13 2010 -0400 richedit: EN_UPDATE notification is sent on WM_PAINT. --- dlls/riched20/editor.c | 2 ++ dlls/riched20/paint.c | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 14ae16b..7e3dfda 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -4421,6 +4421,8 @@ static LRESULT RichEditWndProc_common(HWND hWnd, UINT msg, WPARAM wParam, PAINTSTRUCT ps; hDC = BeginPaint(editor->hWnd, &ps); + if (!editor->bEmulateVersion10 || (editor->nEventMask & ENM_UPDATE)) + ME_SendOldNotify(editor, EN_UPDATE); /* Erase area outside of the formatting rectangle */ if (ps.rcPaint.top < editor->rcFormat.top) { diff --git a/dlls/riched20/paint.c b/dlls/riched20/paint.c index fc52afb..0a32601 100644 --- a/dlls/riched20/paint.c +++ b/dlls/riched20/paint.c @@ -129,8 +129,6 @@ void ME_Repaint(ME_TextEditor *editor) ME_UpdateScrollBar(editor); FIXME("ME_Repaint had to call ME_WrapMarkedParagraphs\n"); } - if (!editor->bEmulateVersion10 || (editor->nEventMask & ENM_UPDATE)) - ME_SendOldNotify(editor, EN_UPDATE); ITextHost_TxViewChange(editor->texthost, TRUE); }

1 0

← Newer
1
...
5
6
7
8
9
10
11
...
90
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 July 2010