Release 980104

Sat Jan  3 17:15:56 1998  Alexandre Julliard  <julliard@lrc.epfl.ch>

	* [debugger/db_disasm.c]
	Added cpuid and cmpxchg instructions.

	* [if1632/builtin.c] [relay32/builtin32.c]
	Fixed broken -dll option with Win32 DLLs.

	* [include/heap.h]
	Added SYSTEM_LOCK/SYSTEM_UNLOCK macros.

	* [configure.in] [misc/lstr.c]
	Added check for wctype.h.
	Commented out --enable-ipc option (IPC code has been broken for a
	long time anyway).

	* [scheduler/critsection.c] [scheduler/event.c]
	  [scheduler/mutex.c] [scheduler/semaphore.c]
	Implemented Win32 synchronization objects.

	* [scheduler/synchro.c]
	Implemented WaitForMultipleObjects and related functions.

	* [scheduler/thread.c]
	If possible, use clone() in CreateThread().

	* [scheduler/thread.c] [scheduler/process.c]
	Made thread and process waitable objects.
	Thread and process id values are now different from the pointers
	they represent.

	* [win32/k32obj.c]
	Moved to scheduler directory.
	Added function table for waiting operations on objects.

	* [files/file.c] [memory/virtual.c]
	Added new K32OBJ function table.

Sun Jan  1 16:48:23 1997  Andreas Mohr <100.30936@germany.net>

	* [files/file.c]
	Fixed my patch for GetTempFileName16() as needed.
	It was ...Name32A() that didn't work properly, not ...Name16().

	* [graphics/x11drv/brush.c]
	Fixed a BadMatch error.

	* [msdos/int21.c]
	Fixed INT21_FindNextFCB() to get correct volume labels e.g.
	in "file open" dialog.

	* [multimedia/joystick.c] [relay32/winmm.spec]
	Stub JoyGetPosEx().

	* [scheduler/process.c] [relay32/kernel32.spec]
	Implemented RegisterServiceProcess().

Wed Dec 31 11:14:43 1997  Lawson Whitney  <lawson_whitney@juno.com>

	* [if1632/kernel.spec] [if1632/relay.c]
	Define CallProcEx32w - Thanks to Marcus Meissner for his excellent
	CallProc32W.

	* [loader/module.c]
	Take a shot at defining FreeLibrary32W.

Sun Dec 28 12:44:04 1997  Kai Morich <kai.morich@rhein-neckar.netsurf.de>

	* [controls/menu.c]
	Menu modification from WM_INITMENUPOPUP message fixed.
	Menu items now can have different wID and hSubMenu (Win95 behavior).

	* [misc/cpu.c]
	Improved IsProcessorFeaturePresent.

Sun Dec 28 03:21:08 1997  Ove Kaaven <ovek@main.arcticnet.no>

	* [include/winsock.h] [misc/winsock.c]
	Fixed WS_SOL_SOCKET for setsockopt(), and made select() return
	empty fd_sets if timeout.

	* [objects/palette.c]
	AnimatePalette() bailed out if entire palette is animated. Fixed.

	* [objects/dib.c]
	Added some code to SetDIBitsToDevice() and its helpers to fix
	some offseting problems.

	* [objects/cursoricon.c]
	Made CreateCursor32() convert the instance handle properly. Made
	DestroyCursor() return correct success status.

Wed Dec 24 17:56:34 1997  Dimitrie O. Paun  <dimi@cs.toronto.edu>

	* [windows/syscolor.c]
	Added definition of GetSysColorPen16/32. This function does not
	exist in the Win32 API but is a very close (and natural) relative
	to GetSysColorBrush function. Moreover, it is *very* much used 
	within Wine since there are a lot of places where we need to draw
	lines with the standard colors.

	* [controls/button.c] [controls/combo.c] [controls/icontitle.c]
	  [controls/menu.c] [controls/progress.c] [controls/scroll.c]
	  [controls/updown.c] [graphics/painting.c] [misc/tweak.c]
	  [windows/defwnd.c] [windows/graphics.c] [windows/nonclient.c]
	Replaced references to sysColorObjects with the appropriate
	call to GetSysColorBrush32/GetSysColorPen32. There is no need to
	expose the implementation of these functions, even within Wine.
	This makes the code easier to understand, debug, maintain.

	* [controls/uitools.c]
	Modified most of the functions in this file to use the now
	standard pens (i.e. GetSysColorPen32). These functions made
	*heavy* use of standard pens so I expect a lot less
	CreatePen/DeleteObject calls can do only good...:)
	Plus some minor modifications (*no* functional changes though).

	* [controls/updown.c]
	Used the new DrawFrameControl32 function to paint the control.
	I also deleted UDDOWN_DrawArrow since it was no longer required.

Tue Dec 23 00:03:33 1997  Steinar Hamre  <steinarh@stud.fim.ntnu.no>

	* [configure.in]
	Added check for -lw.

	* [include/wintypes.h] [tools/build.c]
	Changes to make the assembly understandable for even sun as.
	".ascii" -> ".string", "call %foo" -> "call *%foo",
	"pushw/popw %[cdes]s" written out to ".byte 0x66\npushl/popl %[cdes]s".

	* [memory/ldt.c]
	#ifdef added so <sys/seg.h> will not be included on Solaris.

Mon Dec 22 18:55:19 1997  Marcus Meissner <msmeissn@cip.informatik.uni-erlangen.de>

	* [configure.in]
	Added XF86DGA check.

	* [multimedia/dsound.c][relay32/dsound.spec][include/dsound.h]
	Started DirectSound. Only stubs for now.

	* [graphics/ddraw.c][include/ddraw.h][relay32/ddraw.spec]
	Started to implement DirectDraw. Mostly stubs, some
	testcases work. Requires the XF86DGA extension to XFree86.
	(check demo/blizdemo.exe from the Diablo CD-ROM).

	* [files/drive.c]
	Return correct "CDFS" fsname so Diablo is a bit happier.

Sun Dec 21 21:45:48 1997  Kevin Cozens <kcozens@interlog.com>

	* [misc/registry.c]
	Fixed bugs in the routines which read the Windows '95 registry
	files. Added extra information regarding the format of the Windows
	'95 registry files.
diff --git a/controls/menu.c b/controls/menu.c
index 8c2e984..ee5a098 100644
--- a/controls/menu.c
+++ b/controls/menu.c
@@ -20,7 +20,6 @@
 #include "windows.h"
 #include "bitmap.h"
 #include "gdi.h"
-#include "syscolor.h"
 #include "sysmetrics.h"
 #include "task.h"
 #include "win.h"
@@ -879,7 +878,7 @@
 	r.bottom += MENU_HighlightBottomNudge;
 	r.left += MENU_HighlightLeftNudge;
 	r.right += MENU_HighlightRightNudge;
-	FillRect32( hdc, &r, sysColorObjects.hbrushHighlight );
+	FillRect32( hdc, &r, GetSysColorBrush32(COLOR_HIGHLIGHT) );
     }
     else {
 	RECT32  r = rect;
@@ -887,7 +886,7 @@
 	r.bottom += MENU_HighlightBottomNudge;
 	r.left += MENU_HighlightLeftNudge;
 	r.right += MENU_HighlightRightNudge;
-	FillRect32( hdc, &r, sysColorObjects.hbrushMenu );
+	FillRect32( hdc, &r, GetSysColorBrush32(COLOR_MENU) );
     }
 
     SetBkMode32( hdc, TRANSPARENT );
@@ -899,7 +898,7 @@
 	if(TWEAK_Win95Look)
 	    TWEAK_DrawMenuSeparatorVert95(hdc, rect.left - 1, 3, height - 3);
 	else {
-	    SelectObject32( hdc, sysColorObjects.hpenWindowFrame );
+	    SelectObject32( hdc, GetSysColorPen32(COLOR_WINDOWFRAME) );
 	    MoveTo( hdc, rect.left, 0 );
 	    LineTo32( hdc, rect.left, height );
 	}
@@ -911,7 +910,7 @@
 					   rect.top + SEPARATOR_HEIGHT / 2 + 1,
 					   rect.right - 1);
 	else {
-	    SelectObject32( hdc, sysColorObjects.hpenWindowFrame );
+	    SelectObject32( hdc, GetSysColorPen32(COLOR_WINDOWFRAME) );
 	    MoveTo( hdc, rect.left, rect.top + SEPARATOR_HEIGHT/2 );
 	    LineTo32( hdc, rect.right, rect.top + SEPARATOR_HEIGHT/2 );
 	}
@@ -1067,7 +1066,7 @@
 	rect.right -= POPUP_XSHADE * SYSMETRICS_CXBORDER;
 /*    } */
 
-    if((hPrevBrush = SelectObject32( hdc, sysColorObjects.hbrushMenu )))
+    if((hPrevBrush = SelectObject32( hdc, GetSysColorBrush32(COLOR_MENU) )))
     {
 	HPEN32 hPrevPen;
 	
@@ -1141,10 +1140,10 @@
     if(TWEAK_Win95Look)
 	++lprect->bottom;
 
-    FillRect32(hDC, lprect, sysColorObjects.hbrushMenu );
+    FillRect32(hDC, lprect, GetSysColorBrush32(COLOR_MENU) );
 
     if(!TWEAK_Win95Look) {
-	SelectObject32( hDC, sysColorObjects.hpenWindowFrame );
+	SelectObject32( hDC, GetSysColorPen32(COLOR_WINDOWFRAME) );
 	MoveTo( hDC, lprect->left, lprect->bottom );
 	LineTo32( hDC, lprect->right, lprect->bottom );
     }
@@ -1229,9 +1228,6 @@
 	menu->FocusedItem = NO_SELECTED_ITEM;
     }
 
-    SendMessage16( hwndOwner, WM_INITMENUPOPUP, (WPARAM16)hmenu,
-                   MAKELONG( id, (menu->wFlags & MF_SYSMENU) ? 1 : 0 ));
-
     if( (wndOwner = WIN_FindWndPtr( hwndOwner )) )
     {
 	UINT32	width, height;
@@ -1438,7 +1434,7 @@
     else if (flags & MF_OWNERDRAW) item->text = (LPSTR)str;
     else item->text = NULL;
 
-    if (item->fType & MF_POPUP && item->hSubMenu != id )
+    if ((item->fType & MF_POPUP) && (flags & MF_POPUP) && (item->hSubMenu != id) )
 	DestroyMenu32( item->hSubMenu );   /* ModifyMenu() spec */
 
     if (flags & MF_POPUP)
@@ -1453,12 +1449,18 @@
             item->fState = 0;
 	    return FALSE;
         }
-    }
+    } 
+
+    item->wID = id;
+    if (flags & MF_POPUP)
+      item->hSubMenu = id;
+
+    if ((item->fType & MF_POPUP) && !(flags & MF_POPUP) )
+      flags |= MF_POPUP; /* keep popup */
 
     item->fType = flags & TYPE_MASK;
     item->fState = (flags & STATE_MASK) &
         ~(MF_HILITE | MF_MOUSESELECT | MF_BYPOSITION);
-    item->wID = item->hSubMenu = id;
 
     SetRectEmpty32( &item->rect );
     if (prevText) HeapFree( SystemHeap, 0, prevText );
@@ -1713,6 +1715,7 @@
     POPUPMENU *menu;
     MENUITEM *item;
     WND *wndPtr;
+    HDC32 hdc;
 
     if (!(menu = (POPUPMENU *) USER_HEAP_LIN_ADDR( hmenu ))) return hmenu;
 
@@ -1722,6 +1725,25 @@
     item = &menu->items[menu->FocusedItem];
     if (!(item->fType & MF_POPUP) ||
          (item->fState & (MF_GRAYED | MF_DISABLED))) return hmenu;
+
+    /* message must be send before using item,
+       because nearly everything may by changed by the application ! */
+    rect = item->rect;
+    SendMessage16( hwndOwner, WM_INITMENUPOPUP, (WPARAM16)item->hSubMenu,
+		   MAKELONG( menu->FocusedItem, IS_SYSTEM_MENU(menu) ));
+
+    /* correct item if modified as a reaction to WM_INITMENUPOPUP-message */
+    if (!(item->fState & MF_HILITE)) 
+    {
+        if (menu->wFlags & MF_POPUP) hdc = GetDC32( menu->hWnd );
+        else hdc = GetDCEx32( menu->hWnd, 0, DCX_CACHE | DCX_WINDOW);
+        item->fState |= MF_HILITE;
+        MENU_DrawMenuItem( menu->hWnd, hdc, item, menu->Height, !(menu->wFlags & MF_POPUP) ); 
+	ReleaseDC32( menu->hWnd, hdc );
+    }
+    if (!item->rect.top && !item->rect.left && !item->rect.bottom && !item->rect.right)
+      item->rect = rect;
+
     item->fState |= MF_MOUSESELECT;
 
     if (IS_SYSTEM_MENU(menu))
@@ -2524,6 +2546,7 @@
     BOOL32 ret = FALSE;
 
     HideCaret32(0);
+    SendMessage16( hWnd, WM_INITMENUPOPUP, (WPARAM16)hMenu, 0);
     if (MENU_ShowPopup( hWnd, hMenu, 0, x, y, 0, 0 )) 
 	ret = MENU_TrackMenu( hMenu, wFlags & ~TPM_INTERNAL, 0, 0, hWnd, lpRect );
     ShowCaret32(0);
@@ -2947,8 +2970,6 @@
 
     if (!(menu = (LPPOPUPMENU) USER_HEAP_LIN_ADDR(hMenu))) return -1;
     if ((nPos < 0) || (nPos >= menu->nItems)) return -1;
-    /* FIXME: Now that submenus can have ids, is this still right?  */
-    if (menu->items[nPos].fType & MF_POPUP) return -1;
     return menu->items[nPos].wID;
 }