December 2009 - Wine-Commits

Alexandre Julliard : winex11: Only check for fs register corruption on i386 .
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 10e12c81831891ac3f1de7b31b46641aa30c0973 URL: http://source.winehq.org/git/wine.git/?a=commit;h=10e12c81831891ac3f1de7b31… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 24 12:58:51 2009 +0100 winex11: Only check for fs register corruption on i386. --- dlls/winex11.drv/opengl.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/dlls/winex11.drv/opengl.c b/dlls/winex11.drv/opengl.c index 38f1dad..5bd0255 100644 --- a/dlls/winex11.drv/opengl.c +++ b/dlls/winex11.drv/opengl.c @@ -310,6 +310,7 @@ static BOOL X11DRV_WineGL_InitOpenglInfo(void) vis = pglXChooseVisual(gdi_display, screen, attribList); if (vis) { +#ifdef __i386__ WORD old_fs = wine_get_fs(); /* Create a GLX Context. Without one we can't query GL information */ ctx = pglXCreateContext(gdi_display, vis, None, GL_TRUE); @@ -321,6 +322,9 @@ static BOOL X11DRV_WineGL_InitOpenglInfo(void) ERR( "You need to set the \"UseFastTls\" option to \"2\" in your X config file.\n" ); return FALSE; } +#else + ctx = pglXCreateContext(gdi_display, vis, None, GL_TRUE); +#endif } if (ctx) {

1 0

Andrew Nguyen : kernel32: Ignore empty environment values from registry.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 9294882b947a509021db47410a752c73e57dee31 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9294882b947a509021db47410… Author: Andrew Nguyen <arethusa26(a)gmail.com> Date: Wed Dec 23 11:31:52 2009 -0600 kernel32: Ignore empty environment values from registry. --- dlls/kernel32/process.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 45488aa..eb39f5c 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -378,6 +378,7 @@ static void set_registry_variables( HANDLE hkey, ULONG type ) env_value.Length = env_value.MaximumLength = info->DataLength; if (env_value.Length && !env_value.Buffer[env_value.Length/sizeof(WCHAR)-1]) env_value.Length -= sizeof(WCHAR); /* don't count terminating null if any */ + if (!env_value.Length) continue; if (info->Type == REG_EXPAND_SZ) { WCHAR buf_expanded[1024];

1 0

Nikolay Sivov : comctl32/comboex: Fix silly typo in CBEM_SETITEM handler ( lParam wasn't set).
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 7deb2779007993a8031ce943486120d42d82c22f URL: http://source.winehq.org/git/wine.git/?a=commit;h=7deb2779007993a8031ce9434… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Wed Dec 23 23:08:32 2009 +0300 comctl32/comboex: Fix silly typo in CBEM_SETITEM handler (lParam wasn't set). --- dlls/comctl32/comboex.c | 2 +- dlls/comctl32/tests/comboex.c | 21 ++++++++++++++++++++- 2 files changed, 21 insertions(+), 2 deletions(-) diff --git a/dlls/comctl32/comboex.c b/dlls/comctl32/comboex.c index d423527..1609b90 100644 --- a/dlls/comctl32/comboex.c +++ b/dlls/comctl32/comboex.c @@ -806,7 +806,7 @@ static BOOL COMBOEX_SetItemW (const COMBOEX_INFO *infoPtr, COMBOBOXEXITEMW *cit) if (cit->mask & CBEIF_INDENT) item->iIndent = cit->iIndent; if (cit->mask & CBEIF_LPARAM) - cit->lParam = cit->lParam; + item->lParam = cit->lParam; if (TRACE_ON(comboex)) COMBOEX_DumpItem (item); diff --git a/dlls/comctl32/tests/comboex.c b/dlls/comctl32/tests/comboex.c index f732af5..d6bbce1 100644 --- a/dlls/comctl32/tests/comboex.c +++ b/dlls/comctl32/tests/comboex.c @@ -29,6 +29,8 @@ #define EDITBOX_ID 0 +#define expect(expected, got) ok(got == expected, "Expected %d, got %d\n", expected, got) + static struct msg_sequence *sequences[NUM_MSG_SEQUENCES]; static HWND hComboExParentWnd; @@ -491,10 +493,27 @@ static void test_get_set_item(void) item.pszText = textA; item.iItem = -1; ret = SendMessage(hComboEx, CBEM_SETITEMA, 0, (LPARAM)&item); - ok(ret, "CBEM_SETITEMA failed\n"); + expect(TRUE, ret); ok_sequence(sequences, EDITBOX_SEQ_INDEX, test_setitem_edit_seq, "set item data for edit", FALSE); + /* get/set lParam */ + item.mask = CBEIF_LPARAM; + item.iItem = -1; + item.lParam = 0xdeadbeef; + ret = SendMessage(hComboEx, CBEM_GETITEMA, 0, (LPARAM)&item); + expect(TRUE, ret); + ok(item.lParam == 0, "Expected zero, got %ld\n", item.lParam); + + item.lParam = 0xdeadbeef; + ret = SendMessage(hComboEx, CBEM_SETITEMA, 0, (LPARAM)&item); + expect(TRUE, ret); + + item.lParam = 0; + ret = SendMessage(hComboEx, CBEM_GETITEMA, 0, (LPARAM)&item); + expect(TRUE, ret); + ok(item.lParam == 0xdeadbeef, "Expected 0xdeadbeef, got %ld\n", item.lParam); + DestroyWindow(hComboEx); }

1 0

Nikolay Sivov : comctl32/comboex: Don' t invalidate all windows if Edit box isn't set up.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 7d748b6ba0f7da0f18e88de91e6117b8793af6c0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7d748b6ba0f7da0f18e88de91… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Wed Dec 23 23:32:21 2009 +0300 comctl32/comboex: Don't invalidate all windows if Edit box isn't set up. --- dlls/comctl32/comboex.c | 14 +++++--------- 1 files changed, 5 insertions(+), 9 deletions(-) diff --git a/dlls/comctl32/comboex.c b/dlls/comctl32/comboex.c index 1609b90..185f1e1 100644 --- a/dlls/comctl32/comboex.c +++ b/dlls/comctl32/comboex.c @@ -1149,7 +1149,7 @@ static LRESULT COMBOEX_Command (COMBOEX_INFO *infoPtr, WPARAM wParam) */ ShowWindow (infoPtr->hwndEdit, SW_SHOW); InvalidateRect (infoPtr->hwndCombo, 0, TRUE); - InvalidateRect (infoPtr->hwndEdit, 0, TRUE); + if (infoPtr->hwndEdit) InvalidateRect (infoPtr->hwndEdit, 0, TRUE); cursel = SendMessageW (infoPtr->hwndCombo, CB_GETCURSEL, 0, 0); if (cursel == -1) { cmp_func_t cmptext = get_cmp_func(infoPtr); @@ -1164,8 +1164,7 @@ static LRESULT COMBOEX_Command (COMBOEX_INFO *infoPtr, WPARAM wParam) if ((cursel == n) || ((INT_PTR)item == CB_ERR)) { TRACE("failed to find match??? item=%p cursel=%d\n", item, cursel); - if (infoPtr->hwndEdit) - SetFocus(infoPtr->hwndEdit); + if (infoPtr->hwndEdit) SetFocus(infoPtr->hwndEdit); return 0; } } @@ -1174,8 +1173,7 @@ static LRESULT COMBOEX_Command (COMBOEX_INFO *infoPtr, WPARAM wParam) if ((INT_PTR)item == CB_ERR) { TRACE("failed to find match??? item=%p cursel=%d\n", item, cursel); - if (infoPtr->hwndEdit) - SetFocus(infoPtr->hwndEdit); + if (infoPtr->hwndEdit) SetFocus(infoPtr->hwndEdit); return 0; } } @@ -1271,8 +1269,7 @@ static LRESULT COMBOEX_Command (COMBOEX_INFO *infoPtr, WPARAM wParam) * after passing the command to the parent of the ComboEx. */ lret = SendMessageW (parent, WM_COMMAND, wParam, (LPARAM)infoPtr->hwndSelf); - if (infoPtr->hwndEdit) - SetFocus(infoPtr->hwndEdit); + if (infoPtr->hwndEdit) SetFocus(infoPtr->hwndEdit); return lret; } return 0; @@ -1374,8 +1371,7 @@ static LRESULT COMBOEX_DrawItem (const COMBOEX_INFO *infoPtr, DRAWITEMSTRUCT con GetWindowRect (infoPtr->hwndSelf, &exrc); GetWindowRect (infoPtr->hwndCombo, &cbrc); edrc.left=edrc.top=edrc.right=edrc.bottom=-1; - if (infoPtr->hwndEdit) - GetWindowRect (infoPtr->hwndEdit, &edrc); + if (infoPtr->hwndEdit) GetWindowRect (infoPtr->hwndEdit, &edrc); TRACE("window rects ex=(%s), cb=(%s), ed=(%s)\n", wine_dbgstr_rect(&exrc), wine_dbgstr_rect(&cbrc), wine_dbgstr_rect(&edrc));

1 0

Nikolay Sivov : comctl32/rebar: Avoid empty band range when changing layout .
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 02c06f0d6662d5490761c7c43418b34e91228e95 URL: http://source.winehq.org/git/wine.git/?a=commit;h=02c06f0d6662d5490761c7c43… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Thu Dec 24 00:54:29 2009 +0300 comctl32/rebar: Avoid empty band range when changing layout. --- dlls/comctl32/rebar.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/comctl32/rebar.c b/dlls/comctl32/rebar.c index 90e79d5..021bc16 100644 --- a/dlls/comctl32/rebar.c +++ b/dlls/comctl32/rebar.c @@ -1377,7 +1377,8 @@ REBAR_Layout(REBAR_INFO *infoPtr) xMin += lpBand->cxMinBand; } - REBAR_LayoutRow(infoPtr, rowstart, infoPtr->uNumBands, adjcx, &row, &yPos); + if (rowstart < infoPtr->uNumBands) + REBAR_LayoutRow(infoPtr, rowstart, infoPtr->uNumBands, adjcx, &row, &yPos); if (!(infoPtr->dwStyle & RBS_VARHEIGHT)) yPos = REBAR_SetBandsHeight(infoPtr, first_visible(infoPtr), infoPtr->uNumBands, 0);

1 0

Nikolay Sivov : comctl32/comboex: Move debugging traces that require additional code into TRACE_ON() condition.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 6398448974136abb443356166a7e7009e3a9879e URL: http://source.winehq.org/git/wine.git/?a=commit;h=6398448974136abb443356166… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Thu Dec 24 00:00:46 2009 +0300 comctl32/comboex: Move debugging traces that require additional code into TRACE_ON() condition. --- dlls/comctl32/comboex.c | 53 +++++++++++++++++++++++++--------------------- 1 files changed, 29 insertions(+), 24 deletions(-) diff --git a/dlls/comctl32/comboex.c b/dlls/comctl32/comboex.c index 813504c..14079b2 100644 --- a/dlls/comctl32/comboex.c +++ b/dlls/comctl32/comboex.c @@ -544,7 +544,7 @@ static BOOL COMBOEX_GetItemW (const COMBOEX_INFO *infoPtr, COMBOBOXEXITEMW *cit) INT_PTR index = cit->iItem; CBE_ITEMDATA *item; - TRACE("(...)\n"); + TRACE("\n"); /* if item number requested does not exist then return failure */ if ((index >= infoPtr->nb_items) || (index < -1)) return FALSE; @@ -564,7 +564,7 @@ static BOOL COMBOEX_GetItemA (const COMBOEX_INFO *infoPtr, COMBOBOXEXITEMA *cit) { COMBOBOXEXITEMW tmpcit; - TRACE("(...)\n"); + TRACE("\n"); tmpcit.mask = cit->mask; tmpcit.iItem = cit->iItem; @@ -747,7 +747,7 @@ static HIMAGELIST COMBOEX_SetImageList (COMBOEX_INFO *infoPtr, HIMAGELIST himl) { HIMAGELIST himlTemp = infoPtr->himl; - TRACE("(...)\n"); + TRACE("\n"); infoPtr->himl = himl; @@ -967,7 +967,7 @@ static LRESULT COMBOEX_Create (HWND hwnd, CREATESTRUCTA const *cs) static const WCHAR NIL[] = { 0 }; COMBOEX_INFO *infoPtr; LOGFONTW mylogfont; - RECT wnrc1, clrc1, cmbwrc; + RECT win_rect; INT i; /* allocate memory for info structure */ @@ -992,11 +992,13 @@ static LRESULT COMBOEX_Create (HWND hwnd, CREATESTRUCTA const *cs) SetWindowLongPtrW (hwnd, 0, (DWORD_PTR)infoPtr); - /* create combo box */ - GetWindowRect(hwnd, &wnrc1); - GetClientRect(hwnd, &clrc1); - TRACE("EX window=(%s), client=(%s)\n", - wine_dbgstr_rect(&wnrc1), wine_dbgstr_rect(&clrc1)); + if (TRACE_ON(comboex)) { + RECT client, rect; + GetWindowRect(hwnd, &rect); + GetClientRect(hwnd, &client); + TRACE("EX window=(%s), client=(%s)\n", + wine_dbgstr_rect(&rect), wine_dbgstr_rect(&client)); + } /* Native version of ComboEx creates the ComboBox with DROPDOWNLIST */ /* specified. It then creates it's own version of the EDIT control */ @@ -1071,20 +1073,24 @@ static LRESULT COMBOEX_Create (HWND hwnd, CREATESTRUCTA const *cs) /* Above is fairly certain, below is much less certain. */ - GetWindowRect(hwnd, &wnrc1); - GetClientRect(hwnd, &clrc1); - GetWindowRect(infoPtr->hwndCombo, &cmbwrc); - TRACE("EX window=(%s) client=(%s) CB wnd=(%s)\n", - wine_dbgstr_rect(&wnrc1), wine_dbgstr_rect(&clrc1), - wine_dbgstr_rect(&cmbwrc)); - SetWindowPos(infoPtr->hwndCombo, HWND_TOP, - 0, 0, wnrc1.right-wnrc1.left, wnrc1.bottom-wnrc1.top, + GetWindowRect(hwnd, &win_rect); + + if (TRACE_ON(comboex)) { + RECT client, rect; + GetClientRect(hwnd, &client); + GetWindowRect(infoPtr->hwndCombo, &rect); + TRACE("EX window=(%s) client=(%s) CB wnd=(%s)\n", + wine_dbgstr_rect(&win_rect), wine_dbgstr_rect(&client), + wine_dbgstr_rect(&rect)); + } + SetWindowPos(infoPtr->hwndCombo, HWND_TOP, 0, 0, + win_rect.right - win_rect.left, win_rect.bottom - win_rect.top, SWP_NOACTIVATE | SWP_NOREDRAW); - GetWindowRect(infoPtr->hwndCombo, &cmbwrc); - TRACE("CB window=(%s)\n", wine_dbgstr_rect(&cmbwrc)); - SetWindowPos(hwnd, HWND_TOP, - 0, 0, cmbwrc.right-cmbwrc.left, cmbwrc.bottom-cmbwrc.top, + GetWindowRect(infoPtr->hwndCombo, &win_rect); + TRACE("CB window=(%s)\n", wine_dbgstr_rect(&win_rect)); + SetWindowPos(hwnd, HWND_TOP, 0, 0, + win_rect.right - win_rect.left, win_rect.bottom - win_rect.top, SWP_NOACTIVATE | SWP_NOZORDER | SWP_NOMOVE); COMBOEX_AdjustEditPos (infoPtr); @@ -1357,12 +1363,11 @@ static LRESULT COMBOEX_DrawItem (const COMBOEX_INFO *infoPtr, DRAWITEMSTRUCT con (dis->itemAction == ODA_DRAWENTIRE)) { /* draw of edit control data */ - /* testing */ - { + if (TRACE_ON(comboex)) { RECT exrc, cbrc, edrc; GetWindowRect (infoPtr->hwndSelf, &exrc); GetWindowRect (infoPtr->hwndCombo, &cbrc); - edrc.left=edrc.top=edrc.right=edrc.bottom=-1; + edrc.left = edrc.top = edrc.right = edrc.bottom = -1; if (infoPtr->hwndEdit) GetWindowRect (infoPtr->hwndEdit, &edrc); TRACE("window rects ex=(%s), cb=(%s), ed=(%s)\n", wine_dbgstr_rect(&exrc), wine_dbgstr_rect(&cbrc),

1 0

Nikolay Sivov : comctl32/comboex: Get rid of useless helper.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 253edf0fbbe64504fbcbf958c2a8d644eee2f11e URL: http://source.winehq.org/git/wine.git/?a=commit;h=253edf0fbbe64504fbcbf958c… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Wed Dec 23 23:35:08 2009 +0300 comctl32/comboex: Get rid of useless helper. --- dlls/comctl32/comboex.c | 14 +++----------- 1 files changed, 3 insertions(+), 11 deletions(-) diff --git a/dlls/comctl32/comboex.c b/dlls/comctl32/comboex.c index 185f1e1..813504c 100644 --- a/dlls/comctl32/comboex.c +++ b/dlls/comctl32/comboex.c @@ -484,13 +484,6 @@ static CBE_ITEMDATA * COMBOEX_FindItem(const COMBOEX_INFO *infoPtr, INT_PTR inde return item; } - -static inline BOOL COMBOEX_HasEdit(COMBOEX_INFO const *infoPtr) -{ - return infoPtr->hwndEdit ? TRUE : FALSE; -} - - /* *** CBEM_xxx message support *** */ static UINT COMBOEX_GetListboxText(const COMBOEX_INFO *infoPtr, INT_PTR n, LPWSTR buf) @@ -557,7 +550,7 @@ static BOOL COMBOEX_GetItemW (const COMBOEX_INFO *infoPtr, COMBOBOXEXITEMW *cit) if ((index >= infoPtr->nb_items) || (index < -1)) return FALSE; /* if the item is the edit control and there is no edit control, skip */ - if ((index == -1) && !COMBOEX_HasEdit(infoPtr)) return FALSE; + if ((index == -1) && !infoPtr->hwndEdit) return FALSE; if (!(item = COMBOEX_FindItem(infoPtr, index))) return FALSE; @@ -604,8 +597,7 @@ static BOOL COMBOEX_GetItemA (const COMBOEX_INFO *infoPtr, COMBOBOXEXITEMA *cit) static inline BOOL COMBOEX_HasEditChanged (COMBOEX_INFO const *infoPtr) { - return COMBOEX_HasEdit(infoPtr) && - (infoPtr->flags & WCBE_EDITHASCHANGED) == WCBE_EDITHASCHANGED; + return infoPtr->hwndEdit && (infoPtr->flags & WCBE_EDITHASCHANGED) == WCBE_EDITHASCHANGED; } @@ -778,7 +770,7 @@ static BOOL COMBOEX_SetItemW (const COMBOEX_INFO *infoPtr, COMBOBOXEXITEMW *cit) if ((index >= infoPtr->nb_items) || (index < -1)) return FALSE; /* if the item is the edit control and there is no edit control, skip */ - if ((index == -1) && !COMBOEX_HasEdit(infoPtr)) return FALSE; + if ((index == -1) && !infoPtr->hwndEdit) return FALSE; if (!(item = COMBOEX_FindItem(infoPtr, index))) return FALSE;

1 0

Eric Pouech : winedump: Add support for long section names ( at least used by MinGW).
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 1c238c77f7679fc2122c07f2e1a05ab0a40f8e6c URL: http://source.winehq.org/git/wine.git/?a=commit;h=1c238c77f7679fc2122c07f2e… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Wed Dec 23 22:05:44 2009 +0100 winedump: Add support for long section names (at least used by MinGW). --- tools/winedump/lib.c | 2 +- tools/winedump/pe.c | 22 ++++++++++++++++++---- tools/winedump/winedump.h | 2 +- 3 files changed, 20 insertions(+), 6 deletions(-) diff --git a/tools/winedump/lib.c b/tools/winedump/lib.c index 85f305d..de5184a 100644 --- a/tools/winedump/lib.c +++ b/tools/winedump/lib.c @@ -88,7 +88,7 @@ static void dump_long_import(const void *base, const IMAGE_SECTION_HEADER *ish, for (i = 0; i < num_sect; i++) { if (globals.do_dumpheader) - dump_section(&ish[i]); + dump_section(&ish[i], NULL); if (globals.do_dump_rawdata) { diff --git a/tools/winedump/pe.c b/tools/winedump/pe.c index a79593c..1f9c5a7 100644 --- a/tools/winedump/pe.c +++ b/tools/winedump/pe.c @@ -371,10 +371,14 @@ static void dump_pe_header(void) dump_optional_header((const IMAGE_OPTIONAL_HEADER32*)&PE_nt_headers->OptionalHeader, PE_nt_headers->FileHeader.SizeOfOptionalHeader); } -void dump_section(const IMAGE_SECTION_HEADER *sectHead) +void dump_section(const IMAGE_SECTION_HEADER *sectHead, const char* strtable) { - printf(" %-8.8s VirtSize: 0x%08x VirtAddr: 0x%08x\n", - sectHead->Name, sectHead->Misc.VirtualSize, sectHead->VirtualAddress); + if (strtable && sectHead->Name[0] == '/') /* long section name */ + printf(" %.8s (%s)", sectHead->Name, strtable + atoi((const char*)sectHead->Name + 1)); + else + printf(" %-8.8s", sectHead->Name); + printf(" VirtSize: 0x%08x VirtAddr: 0x%08x\n", + sectHead->Misc.VirtualSize, sectHead->VirtualAddress); printf(" raw data offs: 0x%08x raw data size: 0x%08x\n", sectHead->PointerToRawData, sectHead->SizeOfRawData); printf(" relocation offs: 0x%08x relocations: 0x%08x\n", @@ -448,11 +452,21 @@ static void dump_sections(const void *base, const void* addr, unsigned num_sect) { const IMAGE_SECTION_HEADER* sectHead = addr; unsigned i; + const char* strtable; + + if (PE_nt_headers->FileHeader.PointerToSymbolTable && PE_nt_headers->FileHeader.NumberOfSymbols) + { + /* FIXME: no way to get strtable size */ + strtable = (const char*)base + + PE_nt_headers->FileHeader.PointerToSymbolTable + + PE_nt_headers->FileHeader.NumberOfSymbols * sizeof(IMAGE_SYMBOL); + } + else strtable = NULL; printf("Section Table\n"); for (i = 0; i < num_sect; i++, sectHead++) { - dump_section(sectHead); + dump_section(sectHead, strtable); if (globals.do_dump_rawdata) { diff --git a/tools/winedump/winedump.h b/tools/winedump/winedump.h index 9ad20a5..1f49369 100644 --- a/tools/winedump/winedump.h +++ b/tools/winedump/winedump.h @@ -234,7 +234,7 @@ const char* get_guid_str(const GUID* guid); const char* get_symbol_str(const char* symname); void dump_file_header(const IMAGE_FILE_HEADER *); void dump_optional_header(const IMAGE_OPTIONAL_HEADER32 *, UINT); -void dump_section(const IMAGE_SECTION_HEADER *); +void dump_section(const IMAGE_SECTION_HEADER *, const char* strtable); enum FileSig get_kind_exec(void); void dos_dump( void );

1 0

Nikolay Sivov : comctl32/comboex: Add test for internally sent Edit messages, remove corresponding comments.
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: 96d965f29575c8fd6e7561236bbf1dcb9901fd9a URL: http://source.winehq.org/git/wine.git/?a=commit;h=96d965f29575c8fd6e7561236… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Wed Dec 23 22:51:54 2009 +0300 comctl32/comboex: Add test for internally sent Edit messages, remove corresponding comments. --- dlls/comctl32/comboex.c | 5 +-- dlls/comctl32/tests/comboex.c | 87 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 88 insertions(+), 4 deletions(-) diff --git a/dlls/comctl32/comboex.c b/dlls/comctl32/comboex.c index 62178f1..d423527 100644 --- a/dlls/comctl32/comboex.c +++ b/dlls/comctl32/comboex.c @@ -451,10 +451,7 @@ static void COMBOEX_ReSize (const COMBOEX_INFO *infoPtr) static void COMBOEX_SetEditText (const COMBOEX_INFO *infoPtr, CBE_ITEMDATA *item) { if (!infoPtr->hwndEdit) return; - /* native issues the following messages to the {Edit} control */ - /* WM_SETTEXT (0,addr) */ - /* EM_SETSEL32 (0,0) */ - /* EM_SETSEL32 (0,-1) */ + if (item->mask & CBEIF_TEXT) { SendMessageW (infoPtr->hwndEdit, WM_SETTEXT, 0, (LPARAM)COMBOEX_GetText(infoPtr, item)); SendMessageW (infoPtr->hwndEdit, EM_SETSEL, 0, 0); diff --git a/dlls/comctl32/tests/comboex.c b/dlls/comctl32/tests/comboex.c index a8d7f79..f732af5 100644 --- a/dlls/comctl32/tests/comboex.c +++ b/dlls/comctl32/tests/comboex.c @@ -22,6 +22,14 @@ #include <commctrl.h> #include "wine/test.h" +#include "msg.h" + +#define EDITBOX_SEQ_INDEX 0 +#define NUM_MSG_SEQUENCES 1 + +#define EDITBOX_ID 0 + +static struct msg_sequence *sequences[NUM_MSG_SEQUENCES]; static HWND hComboExParentWnd; static HINSTANCE hMainHinst; @@ -70,6 +78,50 @@ static LONG getItem(HWND cbex, int idx, COMBOBOXEXITEM *cbItem) { return (LONG)SendMessage(cbex, CBEM_GETITEM, 0, (LPARAM)cbItem); } +static LRESULT WINAPI editbox_subclass_proc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam) +{ + WNDPROC oldproc = (WNDPROC)GetWindowLongPtrA(hwnd, GWLP_USERDATA); + static LONG defwndproc_counter = 0; + LRESULT ret; + struct message msg; + + msg.message = message; + msg.flags = sent|wparam|lparam; + if (defwndproc_counter) msg.flags |= defwinproc; + msg.wParam = wParam; + msg.lParam = lParam; + msg.id = EDITBOX_ID; + + if (message != WM_PAINT && + message != WM_ERASEBKGND && + message != WM_NCPAINT && + message != WM_NCHITTEST && + message != WM_GETTEXT && + message != WM_GETICON && + message != WM_DEVICECHANGE) + { + add_message(sequences, EDITBOX_SEQ_INDEX, &msg); + } + + defwndproc_counter++; + ret = CallWindowProcA(oldproc, hwnd, message, wParam, lParam); + defwndproc_counter--; + return ret; +} + +static HWND subclass_editbox(HWND hwndComboEx) +{ + WNDPROC oldproc; + HWND hwnd; + + hwnd = (HWND)SendMessage(hwndComboEx, CBEM_GETEDITCONTROL, 0, 0); + oldproc = (WNDPROC)SetWindowLongPtrA(hwnd, GWLP_WNDPROC, + (LONG_PTR)editbox_subclass_proc); + SetWindowLongPtrA(hwnd, GWLP_USERDATA, (LONG_PTR)oldproc); + + return hwnd; +} + static void test_comboboxex(void) { HWND myHwnd = 0; LONG res = -1; @@ -414,15 +466,50 @@ static void test_comboboxex_subclass(void) DestroyWindow(hComboEx); } +static const struct message test_setitem_edit_seq[] = { + { WM_SETTEXT, sent|id, 0, 0, EDITBOX_ID }, + { EM_SETSEL, sent|id|wparam|lparam, 0, 0, EDITBOX_ID }, + { EM_SETSEL, sent|id|wparam|lparam, 0, -1, EDITBOX_ID }, + { 0 } +}; + +static void test_get_set_item(void) +{ + char textA[] = "test"; + HWND hComboEx; + COMBOBOXEXITEMA item; + BOOL ret; + + hComboEx = createComboEx(WS_BORDER | WS_VISIBLE | WS_CHILD | CBS_DROPDOWN); + + subclass_editbox(hComboEx); + + flush_sequences(sequences, NUM_MSG_SEQUENCES); + + memset(&item, 0, sizeof(item)); + item.mask = CBEIF_TEXT; + item.pszText = textA; + item.iItem = -1; + ret = SendMessage(hComboEx, CBEM_SETITEMA, 0, (LPARAM)&item); + ok(ret, "CBEM_SETITEMA failed\n"); + + ok_sequence(sequences, EDITBOX_SEQ_INDEX, test_setitem_edit_seq, "set item data for edit", FALSE); + + DestroyWindow(hComboEx); +} + START_TEST(comboex) { if (!init()) return; + init_msg_sequences(sequences, NUM_MSG_SEQUENCES); + test_comboboxex(); test_WM_LBUTTONDOWN(); test_CB_GETLBTEXT(); test_comboboxex_subclass(); + test_get_set_item(); cleanup(); }

1 0

André Hentschel : kernel32/nls: Fix German translation .
by Alexandre Julliard Dec. 24, 2009

Dec. 24, 2009

Module: wine Branch: master Commit: b67820cea6fa90ce82c060bd6f330bccadac11eb URL: http://source.winehq.org/git/wine.git/?a=commit;h=b67820cea6fa90ce82c060bd6… Author: André Hentschel <nerv(a)dawncrow.de> Date: Wed Dec 23 20:02:39 2009 +0100 kernel32/nls: Fix German translation. --- dlls/kernel32/nls/winerr_deu.mc | 22 +++++++++++----------- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git a/dlls/kernel32/nls/winerr_deu.mc b/dlls/kernel32/nls/winerr_deu.mc index e0d70d5..8b1d75a 100644 --- a/dlls/kernel32/nls/winerr_deu.mc +++ b/dlls/kernel32/nls/winerr_deu.mc @@ -227,7 +227,7 @@ Doppelter Netzwerkname MessageId=53 SymbolicName=ERROR_BAD_NETPATH Language=DEU -Fehlerhafter Netzpfad +Fehlerhafter Netzwerkpfad . MessageId=54 SymbolicName=ERROR_NETWORK_BUSY @@ -487,7 +487,7 @@ Ungültige Ebene MessageId=125 SymbolicName=ERROR_NO_VOLUME_LABEL Language=DEU -Keine Volumen-Bezeichnung +Keine Datenträgerbezeichnung . MessageId=126 SymbolicName=ERROR_MOD_NOT_FOUND @@ -502,12 +502,12 @@ Prozedur nicht gefunden MessageId=128 SymbolicName=ERROR_WAIT_NO_CHILDREN Language=DEU -Keine Kinder auf die gewartet werden kann +Keine Unterprogramme auf die gewartet werden kann . MessageId=129 SymbolicName=ERROR_CHILD_NOT_COMPLETE Language=DEU -Kind-Prozess ist nicht abgeschlossen +Unterprogramm ist nicht abgeschlossen . MessageId=130 SymbolicName=ERROR_DIRECT_ACCESS_HANDLE @@ -622,7 +622,7 @@ Event-Anzahl für DosMuxSemWait nicht korrekt MessageId=152 SymbolicName=ERROR_TOO_MANY_MUXWAITERS Language=DEU -Zu viele Warter für DosMuxSemWait +Zu viele Wartende für DosMuxSemWait . MessageId=153 SymbolicName=ERROR_INVALID_LIST_FORMAT @@ -632,7 +632,7 @@ DosSemMuxWait-Liste ungültig MessageId=154 SymbolicName=ERROR_LABEL_TOO_LONG Language=DEU -Volumen-Bezeichnung zu lang +Datenträgerbezeichnung zu lang . MessageId=155 SymbolicName=ERROR_TOO_MANY_TCBS @@ -687,7 +687,7 @@ Sperre fehlgeschlagen MessageId=170 SymbolicName=ERROR_BUSY Language=DEU -Resource benutzt +Resource in Benutzung . MessageId=173 SymbolicName=ERROR_CANCEL_VIOLATION @@ -1687,7 +1687,7 @@ Ungültiger Ereignisname MessageId=1212 SymbolicName=ERROR_INVALID_DOMAINNAME Language=DEU -Ungültiger Domain-Name +Ungültiger Domänenname . MessageId=1213 SymbolicName=ERROR_INVALID_SERVICENAME @@ -1702,7 +1702,7 @@ Ungültiger Netzwerkname MessageId=1215 SymbolicName=ERROR_INVALID_SHARENAME Language=DEU -Ungültiger Freigabenname +Ungültiger Freigabename . MessageId=1216 SymbolicName=ERROR_INVALID_PASSWORDNAME @@ -3492,7 +3492,7 @@ Nicht genügend Quota MessageId=1817 SymbolicName=RPC_S_NO_INTERFACES Language=DEU -Keine Interfaces +Keine Schnittstellen . MessageId=1818 SymbolicName=RPC_S_CALL_CANCELLED @@ -3527,7 +3527,7 @@ Kein RPC-Fehler MessageId=1824 SymbolicName=RPC_S_UUID_LOCAL_ONLY Language=DEU -UUID isr nur lokal +UUID ist nur lokal . MessageId=1825 SymbolicName=RPC_S_SEC_PKG_ERROR

1 0