commit 72d827010cfeef63dc6f59e3e17a2a65d3e30060
author Uwe Bonnes <bon@elektron.ikp.physik.tu-darmstadt.de> Wed Jul 10 23:11:28 2002 +0000
committer Alexandre Julliard <julliard@winehq.org> Wed Jul 10 23:11:28 2002 +0000
tree 93ee9a65df9d3478236e688059eefd68c539289a
parent a80a814234ed760108a87da48bbd25944883596d

CBUpdateEdit: LB_GETTEXTLEN returns LB_ERR on error, not 0.

controls/combo.c

diff --git a/controls/combo.c b/controls/combo.c
index 49215e7..e92a2a7 100644
--- a/controls/combo.c
+++ b/controls/combo.c
@@ -771,6 +771,8 @@
    if( (id = SendMessageW(lphc->hWndLBox, LB_GETCURSEL, 0, 0) ) != LB_ERR )
    {
         size = SendMessageW(lphc->hWndLBox, LB_GETTEXTLEN, id, 0);
+	if (size == LB_ERR)
+	  FIXME("LB_ERR probably not handled yet\n");
         if( (pText = HeapAlloc( GetProcessHeap(), 0, (size + 1) * sizeof(WCHAR))) )
 	{
             /* size from LB_GETTEXTLEN may be too large, from LB_GETTEXT is accurate */
@@ -1113,7 +1115,7 @@
    if( index >= 0 ) /* got an entry */
    {
        length = SendMessageW(lphc->hWndLBox, LB_GETTEXTLEN, (WPARAM)index, 0);
-       if( length )
+       if( length != LB_ERR)
        {
 	   if( (pText = HeapAlloc( GetProcessHeap(), 0, (length + 1) * sizeof(WCHAR))) )
 	   {
@@ -1538,7 +1540,8 @@
 	   INT n = 0;
            INT length = SendMessageW(lphc->hWndLBox, LB_GETTEXTLEN,
 					        (WPARAM)idx, 0 );
-
+	   if(length == LB_ERR)
+	     FIXME("LB_ERR probably not handled yet\n");
 	    if(unicode)
 	    {
 		LPWSTR lpBuffer, lpText = (LPWSTR)lParam;