18 Jan
2009
18 Jan
'09
2:28 a.m.
Author: tkreuzer
Date: Sat Jan 17 20:28:12 2009
New Revision: 38865
URL: http://svn.reactos.org/svn/reactos?rev=38865&view=rev
Log:
user32: sync listbox.c to wine 1.1.13
Modified:
trunk/reactos/dll/win32/user32/controls/listbox.c
Modified: trunk/reactos/dll/win32/user32/controls/listbox.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/user32/controls/…
==============================================================================
--- trunk/reactos/dll/win32/user32/controls/listbox.c [iso-8859-1] (original)
+++ trunk/reactos/dll/win32/user32/controls/listbox.c [iso-8859-1] Sat Jan 17 20:28:12
2009
@@ -37,7 +37,6 @@
#include <wine/debug.h>
WINE_DEFAULT_DEBUG_CHANNEL(listbox);
-WINE_DECLARE_DEBUG_CHANNEL(combo);
/* Start of hack section -------------------------------- */
@@ -147,9 +146,10 @@
/*********************************************************************
* listbox class descriptor
*/
+static const WCHAR listboxW[] =
{'L','i','s','t','B','o','x',0};
const struct builtin_class_descr LISTBOX_builtin_class =
{
- L"ListBox", /* name */
+ listboxW, /* name */
CS_DBLCLKS /*| CS_PARENTDC*/, /* style */
ListBoxWndProcA, /* procA */
ListBoxWndProcW, /* procW */
@@ -162,9 +162,10 @@
/*********************************************************************
* combolbox class descriptor
*/
+static const WCHAR combolboxW[] =
{'C','o','m','b','o','L','B','o','x',0};
const struct builtin_class_descr COMBOLBOX_builtin_class =
{
- L"ComboLBox", /* name */
+ combolboxW, /* name */
CS_DBLCLKS | CS_SAVEBITS, /* style */
ListBoxWndProcA, /* procA */
ListBoxWndProcW, /* procW */
@@ -175,31 +176,11 @@
#ifndef __REACTOS__
/* check whether app is a Win 3.1 app */
-inline static BOOL is_old_app( HWND hwnd )
-{
- return (GetExpWinVer16( GetWindowLongA(hwnd,GWL_HINSTANCE) ) & 0xFF00 ) ==
0x0300;
-}
-#endif
-
-
-/***********************************************************************
- * LISTBOX_Dump
- */
-void LISTBOX_Dump( HWND hwnd )
-{
- INT i;
- LB_ITEMDATA *item;
- LB_DESCR *descr = (LB_DESCR *)GetWindowLongA( hwnd, 0 );
-
- TRACE( "Listbox:\n" );
- TRACE( "hwnd=%p descr=%08x items=%d top=%d\n",
- hwnd, (UINT)descr, descr->nb_items, descr->top_item );
- for (i = 0, item = descr->items; i < descr->nb_items; i++, item++)
- {
- TRACE( "%4d: %-40s %d %08lx %3d\n",
- i, debugstr_w(item->str), item->selected, item->data,
item->height );
- }
-}
+static inline BOOL is_old_app( LB_DESCR *descr )
+{
+ return (GetExpWinVer16( GetWindowLongPtrW(descr->self, GWLP_HINSTANCE) ) &
0xFF00 ) == 0x0300;
+}
+#endif
/***********************************************************************
@@ -429,7 +410,7 @@
if ((descr->height > descr->item_height) && remaining)
{
#ifndef __REACTOS__
- if (is_old_app(hwnd))
+ if (is_old_app(descr))
{ /* give a margin for error to 16 bits programs - if we need
less than the height of the nonclient area, round to the
*next* number of items */
@@ -569,8 +550,8 @@
*
* Paint an item.
*/
-static void LISTBOX_PaintItem( LB_DESCR *descr, HDC hdc,
- const RECT *rect, INT index, UINT action, BOOL ignoreFocus
)
+static void LISTBOX_PaintItem( LB_DESCR *descr, HDC hdc, const RECT *rect,
+ INT index, UINT action, BOOL ignoreFocus )
{
LB_ITEMDATA *item = NULL;
if (index < descr->nb_items) item = &descr->items[index];
@@ -589,7 +570,7 @@
DrawFocusRect( hdc, rect );
}
else
- FIXME("called with an out of bounds index %d(%d) in owner draw, Not
good.\n",index,descr->nb_items);
+ ERR("called with an out of bounds index %d(%d) in owner draw, Not
good.\n",index,descr->nb_items);
return;
}
@@ -616,9 +597,9 @@
if (!IsWindowEnabled(descr->self)) dis.itemState |= ODS_DISABLED;
dis.itemData = item->data;
dis.rcItem = *rect;
- TRACE("[%p]: drawitem %d (%s) action=%02x state=%02x
rect=%ld,%ld-%ld,%ld\n",
+ TRACE("[%p]: drawitem %d (%s) action=%02x state=%02x rect=%s\n",
descr->self, index, item ? debugstr_w(item->str) : "",
action,
- dis.itemState, rect->left, rect->top, rect->right, rect->bottom
);
+ dis.itemState, wine_dbgstr_rect(rect) );
SendMessageW(descr->owner, WM_DRAWITEM, dis.CtlID, (LPARAM)&dis);
}
else
@@ -637,9 +618,9 @@
oldText = SetTextColor( hdc, GetSysColor(COLOR_HIGHLIGHTTEXT));
}
- TRACE("[%p]: painting %d (%s) action=%02x rect=%ld,%ld-%ld,%ld\n",
+ TRACE("[%p]: painting %d (%s) action=%02x rect=%s\n",
descr->self, index, item ? debugstr_w(item->str) : "",
action,
- rect->left, rect->top, rect->right, rect->bottom );
+ wine_dbgstr_rect(rect) );
if (!item)
ExtTextOutW( hdc, rect->left + 1, rect->top,
ETO_OPAQUE | ETO_CLIPPED, rect, NULL, 0, NULL );
@@ -724,7 +705,7 @@
if (!IsWindowEnabled(descr->self))
SetTextColor( hdc, GetSysColor( COLOR_GRAYTEXT ) );
SetWindowOrgEx( hdc, descr->horz_pos, 0, NULL );
- LISTBOX_PaintItem( descr, hdc, &rect, index, action, FALSE );
+ LISTBOX_PaintItem( descr, hdc, &rect, index, action, TRUE );
if (oldFont) SelectObject( hdc, oldFont );
if (oldBrush) SelectObject( hdc, oldBrush );
ReleaseDC( descr->self, hdc );
@@ -815,7 +796,7 @@
INT i;
LPINT16 p = (LPINT16)tabs;
- TRACE("[%p]: settabstops ", hwnd );
+ TRACE("[%p]: settabstops ", descr->self );
for (i = 0; i < descr->nb_tabs; i++) {
descr->tabs[i] = *p++<<1; /* FIXME */
TRACE("%hd ", descr->tabs[i]);
@@ -840,6 +821,8 @@
*/
static LRESULT LISTBOX_GetText( LB_DESCR *descr, INT index, LPWSTR buffer, BOOL unicode
)
{
+ DWORD len;
+
if ((index < 0) || (index >= descr->nb_items))
{
SetLastError(ERROR_INVALID_INDEX);
@@ -849,7 +832,7 @@
{
if (!buffer)
{
- DWORD len = strlenW(descr->items[index].str);
+ len = strlenW(descr->items[index].str);
if( unicode )
return len;
return WideCharToMultiByte( CP_ACP, 0, descr->items[index].str, len,
@@ -858,23 +841,35 @@
TRACE("index %d (0x%04x) %s\n", index, index,
debugstr_w(descr->items[index].str));
- if(unicode)
- {
- strcpyW( buffer, descr->items[index].str );
- return strlenW(buffer);
- }
- else
- {
- return WideCharToMultiByte(CP_ACP, 0, descr->items[index].str, -1,
(LPSTR)buffer, 0x7FFFFFFF, NULL, NULL) - 1;
- }
+ __TRY /* hide a Delphi bug that passes a read-only buffer */
+ {
+ if(unicode)
+ {
+ strcpyW( buffer, descr->items[index].str );
+ len = strlenW(buffer);
+ }
+ else
+ {
+ len = WideCharToMultiByte(CP_ACP, 0, descr->items[index].str, -1,
+ (LPSTR)buffer, 0x7FFFFFFF, NULL, NULL) - 1;
+ }
+ }
+ __EXCEPT_PAGE_FAULT
+ {
+ WARN( "got an invalid buffer (Delphi bug?)\n" );
+ SetLastError( ERROR_INVALID_PARAMETER );
+ return LB_ERR;
+ }
+ __ENDTRY
} else {
if (buffer)
*((LPDWORD)buffer)=*(LPDWORD)(&descr->items[index].data);
- return sizeof(DWORD);
- }
-}
-
-static __inline INT LISTBOX_lstrcmpiW( LCID lcid, LPCWSTR str1, LPCWSTR str2 )
+ len = sizeof(DWORD);
+ }
+ return len;
+}
+
+static inline INT LISTBOX_lstrcmpiW( LCID lcid, LPCWSTR str1, LPCWSTR str2 )
{
INT ret = CompareStringW( lcid, NORM_IGNORECASE, str1, -1, str2, -1 );
if (ret == CSTR_LESS_THAN)
@@ -1509,7 +1504,7 @@
if (descr->style & LBS_MULTIPLESEL)
{
if (index == -1) /* Select all items */
- return LISTBOX_SelectItemRange( descr, 0, -1, on );
+ return LISTBOX_SelectItemRange( descr, 0, descr->nb_items, on );
else /* Only one item */
return LISTBOX_SelectItemRange( descr, index, index, on );
}
@@ -1669,8 +1664,8 @@
{
if (index <= descr->selected_item)
{
- descr->selected_item++;
- descr->focus_item = oldfocus; /* focus not changed */
+ descr->selected_item++;
+ descr->focus_item = oldfocus; /* focus not changed */
}
}
return LB_OKAY;
@@ -1680,8 +1675,7 @@
/***********************************************************************
* LISTBOX_InsertString
*/
-static LRESULT LISTBOX_InsertString( LB_DESCR *descr, INT index,
- LPCWSTR str )
+static LRESULT LISTBOX_InsertString( LB_DESCR *descr, INT index, LPCWSTR str )
{
LPWSTR new_str = NULL;
ULONG_PTR data = 0;
@@ -1815,7 +1809,7 @@
{
INT i;
- for (i = 0; i < descr->nb_items; i++) LISTBOX_DeleteItem( descr, i );
+ for(i = descr->nb_items - 1; i>=0; i--) LISTBOX_DeleteItem( descr, i);
HeapFree( GetProcessHeap(), 0, descr->items );
descr->nb_items = 0;
descr->top_item = 0;
@@ -1873,7 +1867,7 @@
/* scan directory */
if ((handle = FindFirstFileW(filespec, &entry)) == INVALID_HANDLE_VALUE)
{
- int le = GetLastError();
+ int le = GetLastError();
if ((le != ERROR_NO_MORE_FILES) && (le != ERROR_FILE_NOT_FOUND))
return LB_ERR;
}
else
@@ -1913,17 +1907,11 @@
pos = LISTBOX_FindFileStrPos( descr, buffer );
if ((ret = LISTBOX_InsertString( descr, pos, buffer )) < 0)
break;
-
- if (ret <= maxinsert)
- maxinsert++;
- else
- maxinsert = ret;
-
+ if (ret <= maxinsert) maxinsert++; else maxinsert = ret;
} while (FindNextFileW( handle, &entry ));
FindClose( handle );
}
}
-
if (ret >= 0)
{
ret = maxinsert;
@@ -2500,7 +2488,7 @@
LISTBOX_SetSelection( descr, caret, TRUE, FALSE);
if (descr->style & LBS_NOTIFY)
{
- if( descr->lphc )
+ if (descr->lphc && IsWindowVisible( descr->self ))
{
/* make sure that combo parent doesn't hide us */
descr->lphc->wState |= CBF_NOROLLUP;
@@ -2594,7 +2582,7 @@
descr->lphc = lphc;
#ifndef __REACTOS__
- if (is_old_app(hwnd) && ( descr->style & ( WS_VSCROLL | WS_HSCROLL ) )
)
+ if (is_old_app(descr) && ( descr->style & ( WS_VSCROLL | WS_HSCROLL )
) )
{
/* Win95 document "List Box Differences" from MSDN:
If a list box in a version 3.x application has either the
@@ -2607,7 +2595,7 @@
if( lphc )
{
- TRACE_(combo)("[%p]: resetting owner %p -> %p\n", hwnd,
descr->owner, lphc->self );
+ TRACE("[%p]: resetting owner %p -> %p\n", descr->self,
descr->owner, lphc->self );
descr->owner = lphc->self;
}
@@ -2663,8 +2651,8 @@
/***********************************************************************
* ListBoxWndProc_common
*/
-static LRESULT WINAPI ListBoxWndProc_common( HWND hwnd, UINT msg,
- WPARAM wParam, LPARAM lParam, BOOL unicode
)
+static LRESULT ListBoxWndProc_common( HWND hwnd, UINT msg,
+ WPARAM wParam, LPARAM lParam, BOOL unicode )
{
LB_DESCR *descr = (LB_DESCR *)GetWindowLongPtrW( hwnd, 0 );
LPHEADCOMBO lphc = 0;
@@ -2679,7 +2667,7 @@
CREATESTRUCTW *lpcs = (CREATESTRUCTW *)lParam;
if (lpcs->style & LBS_COMBOBOX) lphc = (LPHEADCOMBO)lpcs->lpCreateParams;
if (!LISTBOX_Create( hwnd, lphc )) return -1;
- TRACE("creating wnd=%p descr=%lx\n", hwnd, GetWindowLongPtrW( hwnd,
0 ) );
+ TRACE("creating hwnd %p descr %p\n", hwnd, (void
*)GetWindowLongPtrW( hwnd, 0 ) );
return 0;
}
/* Ignore all other messages before we get a WM_CREATE */
@@ -2689,7 +2677,7 @@
if (descr->style & LBS_COMBOBOX) lphc = descr->lphc;
TRACE("[%p]: msg %s wp %08lx lp %08lx\n",
- hwnd, SPY_GetMsgName(msg, hwnd), wParam, lParam );
+ descr->self, SPY_GetMsgName(msg, descr->self), wParam, lParam );
switch(msg)
{
@@ -2721,6 +2709,8 @@
INT countW = MultiByteToWideChar(CP_ACP, 0, textA, -1, NULL, 0);
if((textW = HeapAlloc(GetProcessHeap(), 0, countW * sizeof(WCHAR))))
MultiByteToWideChar(CP_ACP, 0, textA, -1, textW, countW);
+ else
+ return LB_ERRSPACE;
}
/* in the unicode the version, the string is really overwritten
during the converting case */
@@ -2755,6 +2745,8 @@
INT countW = MultiByteToWideChar(CP_ACP, 0, textA, -1, NULL, 0);
if((textW = HeapAlloc(GetProcessHeap(), 0, countW * sizeof(WCHAR))))
MultiByteToWideChar(CP_ACP, 0, textA, -1, textW, countW);
+ else
+ return LB_ERRSPACE;
}
/* in the unicode the version, the string is really overwritten
during the converting case */
@@ -2785,6 +2777,8 @@
INT countW = MultiByteToWideChar(CP_ACP, 0, textA, -1, NULL, 0);
if((textW = HeapAlloc(GetProcessHeap(), 0, countW * sizeof(WCHAR))))
MultiByteToWideChar(CP_ACP, 0, textA, -1, textW, countW);
+ else
+ return LB_ERRSPACE;
}
wParam = LISTBOX_FindFileStrPos( descr, textW );
ret = LISTBOX_InsertString( descr, wParam, textW );
@@ -2963,8 +2957,12 @@
case LB_GETITEMRECT16:
{
RECT rect;
+ RECT16 *r16 = MapSL(lParam);
ret = LISTBOX_GetItemRect( descr, (INT16)wParam, &rect );
- CONV_RECT32TO16( &rect, MapSL(lParam) );
+ r16->left = rect.left;
+ r16->top = rect.top;
+ r16->right = rect.right;
+ r16->bottom = rect.bottom;
}
return ret;
#endif
@@ -3083,8 +3081,8 @@
if (IS_MULTISELECT(descr)) return LB_ERR;
LISTBOX_SetCaretIndex( descr, wParam, FALSE );
ret = LISTBOX_SetSelection( descr, wParam, TRUE, FALSE );
- if (ret != LB_ERR) ret = descr->selected_item;
- return ret;
+ if (ret != LB_ERR) ret = descr->selected_item;
+ return ret;
#ifndef __REACTOS__
case LB_GETSELCOUNT16:
@@ -3236,7 +3234,7 @@
return LISTBOX_Destroy( descr );
case WM_ENABLE:
- InvalidateRect( hwnd, NULL, TRUE );
+ InvalidateRect( descr->self, NULL, TRUE );
return 0;
case WM_SETREDRAW:
@@ -3252,7 +3250,7 @@
PAINTSTRUCT ps;
HDC hdc = ( wParam ) ? ((HDC)wParam) : BeginPaint( descr->self, &ps
);
ret = LISTBOX_Paint( descr, hdc );
- if( !wParam ) EndPaint( hwnd, &ps );
+ if( !wParam ) EndPaint( descr->self, &ps );
}
return ret;
case WM_SIZE:
@@ -3262,7 +3260,7 @@
return (LRESULT)descr->font;
case WM_SETFONT:
LISTBOX_SetFont( descr, (HFONT)wParam );
- if (lParam) InvalidateRect( hwnd, 0, TRUE );
+ if (lParam) InvalidateRect( descr->self, 0, TRUE );
return 0;
case WM_SETFOCUS:
descr->in_focus = TRUE;
@@ -3283,7 +3281,7 @@
return LISTBOX_HandleVScroll( descr, LOWORD(wParam), HIWORD(wParam) );
case WM_MOUSEWHEEL:
if (wParam & (MK_SHIFT | MK_CONTROL))
- return DefWindowProcW( hwnd, msg, wParam, lParam );
+ return DefWindowProcW( descr->self, msg, wParam, lParam );
return LISTBOX_HandleMouseWheel( descr, (SHORT)HIWORD(wParam) );
case WM_LBUTTONDOWN:
if (lphc)
@@ -3342,7 +3340,7 @@
mousePos.x = (INT16)LOWORD(lParam);
mousePos.y = (INT16)HIWORD(lParam);
- GetClientRect(hwnd, &clientRect);
+ GetClientRect(descr->self, &clientRect);
/*
* When the user clicks outside the combobox and the focus
@@ -3431,7 +3429,6 @@
if ((msg >= WM_USER) && (msg < 0xc000))
WARN("[%p]: unknown msg %04x wp %08lx lp %08lx\n",
hwnd, msg, wParam, lParam );
- break;
}
return unicode ? DefWindowProcW( hwnd, msg, wParam, lParam ) :
@@ -3440,9 +3437,6 @@
/***********************************************************************
* ListBoxWndProcA
- *
- * This is just a wrapper for the real wndproc, it only does window locking
- * and unlocking.
*/
static LRESULT WINAPI ListBoxWndProcA( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam
)
{
@@ -3456,4 +3450,3 @@
{
return ListBoxWndProc_common( hwnd, msg, wParam, lParam, TRUE );
}
-
5783
days inactive
5783
days old
0 comments
1 participants
participants (1)
-
tkreuzer＠svn.reactos.org